[Asterisk-code-review] res pjsip sdp rtp: Allow only single ssrc attribute. (asterisk[16])

[Joshua C. Colp]

Joshua C. Colp has uploaded this change for review. ( https://gerrit.asterisk.org/11030


Change subject: res_pjsip_sdp_rtp: Allow only single ssrc attribute.
......................................................................

res_pjsip_sdp_rtp: Allow only single ssrc attribute.

When processing SSRC attributes we were iterating through
all of them, even though we only need to know the remote
SSRC once. This was problematic because some browsers group
SSRCs together on a stream, and due to our negotiation only
end up using the first one. Since we set the second one as
the remote SSRC we would drop the received media from them
instead of allowing it through.

In the future this may be extended to allow SSRC groups
and to use information from the attributes.

Change-Id: I4dc87087dbe56a83aa65f0f897bbd4ca75ec1270
---
M res/res_pjsip_sdp_rtp.c
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/30/11030/1

diff --git a/res/res_pjsip_sdp_rtp.c b/res/res_pjsip_sdp_rtp.c
index c2b7d67..9686960 100644
--- a/res/res_pjsip_sdp_rtp.c
+++ b/res/res_pjsip_sdp_rtp.c
@@ -1127,6 +1127,7 @@
 		}
 
 		ast_rtp_instance_set_remote_ssrc(session_media->rtp, ssrc);
+		break;
 	}
 }
 

-- 
To view, visit https://gerrit.asterisk.org/11030
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4dc87087dbe56a83aa65f0f897bbd4ca75ec1270
Gerrit-Change-Number: 11030
Gerrit-PatchSet: 1
Gerrit-Owner: Joshua C. Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190221/907f4b92/attachment.html>