[asterisk-dev] [Code Review] 4286: rtp_engine: avoid payload types above 127

Scott Griepentrog reviewboard at asterisk.org
Wed Dec 31 12:54:44 CST 2014

This is an automatically generated e-mail. To reply, visit:

(Updated Dec. 31, 2014, 12:54 p.m.)


This change has been marked as submitted.

Review request for Asterisk Developers.


Committed in revision 430164

Bugs: ASTERISK-24367

Repository: Asterisk


Valid payload type codes are between 0 and 127 to allow for being stored in 7 bits.  During call setup, pjsip validates the SDP and will assert if it encounters an invalid payload type code (see pjmedia_sdp_validate() in pjmedia/src/pjmedia/sdp.c).  This assert will be hit if a call is placed to a pjsip endpoint with allow=all set.

To avoid this, the previous use 128 for the slin192 format has been changed to 95.


  /trunk/main/rtp_engine.c 429845 

Diff: https://reviewboard.asterisk.org/r/4286/diff/


Tested with pjsip calls to allow=all configured extensions.


Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20141231/0b25ef3f/attachment.html>

More information about the asterisk-dev mailing list