[Asterisk-code-review] res pjsip sdp rtp: Properly set packetization on the codec (asterisk[13])

[Joshua Colp]

Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/10495 )

Change subject: res_pjsip_sdp_rtp: Properly set packetization on the codec
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.asterisk.org/#/c/10495/2/res/res_pjsip_sdp_rtp.c
File res/res_pjsip_sdp_rtp.c:

https://gerrit.asterisk.org/#/c/10495/2/res/res_pjsip_sdp_rtp.c@396
PS2, Line 396: 		if (codecs.framing) {
             : 			/* Copy the codecs framing ms even though there is no match */
             : 			ast_rtp_codecs_set_framing(ast_rtp_instance_get_codecs(session_media->rtp), codecs.framing);
             : 		}
> This is the transcoding case and if the framing is not copied then the transcoded payload ends up wi […]
This code is for SDP negotiation though, it has no concept of transcoding here and this code path is if the configuration and the actual SDP results in no formats and the stream is rejected - so I'm not sure how this change helps in the scenario you mention.



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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic64b8fc6a140a93fbdb2f97550a40d0ff334e607
Gerrit-Change-Number: 10495
Gerrit-PatchSet: 2
Gerrit-Owner: Robert Cripps <rcripps at voxbone.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Robert Cripps <rcripps at voxbone.com>
Gerrit-Comment-Date: Thu, 25 Oct 2018 14:22:06 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181025/7332b8eb/attachment.html>