From 2eef53c465460609cd66770b5dcdcb17a20fc33e Mon Sep 17 00:00:00 2001
From: Joshua Colp <jcolp@digium.com>
Date: Sat, 27 Sep 2014 17:29:05 +0000
Subject: [PATCH] res_pjsip_session: Reduce SDP size by removing duplicate
connection lines.
Due to the architecture of how media streams are handled each individual
handler adds connection details (IP address) for it. The first media stream
is then used as the top level SDP connection line. In practice each
line ends up being the same so to reduce the SDP size stream-level connection
information is also added to the SDP if it differs from the top level SDP
connection line.
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@424077 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
res/res_pjsip_session.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c
index c31e4d252f..3b924ac2f9 100644
--- a/res/res_pjsip_session.c
+++ b/res/res_pjsip_session.c
@@ -2132,10 +2132,27 @@ static struct pjmedia_sdp_session *create_local_sdp(pjsip_inv_session *inv, stru
/* Use the connection details of the first media stream if possible for SDP level */
if (local->media_count) {
+ int stream;
+
+ /* Since we are using the first media stream as the SDP level we can get rid of it
+ * from the stream itself
+ */
local->conn = local->media[0]->conn;
+ local->media[0]->conn = NULL;
pj_strassign(&local->origin.net_type, &local->conn->net_type);
pj_strassign(&local->origin.addr_type, &local->conn->addr_type);
pj_strassign(&local->origin.addr, &local->conn->addr);
+
+ /* Go through each media stream seeing if the connection details actually differ,
+ * if not just use SDP level and reduce the SDP size
+ */
+ for (stream = 1; stream < local->media_count; stream++) {
+ if (!pj_strcmp(&local->conn->net_type, &local->media[stream]->conn->net_type) &&
+ !pj_strcmp(&local->conn->addr_type, &local->media[stream]->conn->addr_type) &&
+ !pj_strcmp(&local->conn->addr, &local->media[stream]->conn->addr)) {
+ local->media[stream]->conn = NULL;
+ }
+ }
} else {
local->origin.net_type = STR_IN;
local->origin.addr_type = session->endpoint->media.rtp.ipv6 ? STR_IP6 : STR_IP4;
--
GitLab