[asterisk-commits] sgriepentrog: trunk r430164 - /trunk/main/rtp_engine.c

[SVN commits to the Asterisk project]

Author: sgriepentrog
Date: Wed Dec 31 12:54:37 2014
New Revision: 430164

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=430164
Log:
rtp_engine: keep payload types in correct range

In r428708 additional codecs were added including
a payload type of 128 which is outside of nominal
range of 0-127.  This change moves changes 128 to
96 to avoid causing a pjsip assertion when making
a call to an endpoint configured with allow=all.

ASTERISK-24367 #close
Review: https://reviewboard.asterisk.org/r/4286/


Modified:
    trunk/main/rtp_engine.c

Modified: trunk/main/rtp_engine.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/rtp_engine.c?view=diff&rev=430164&r1=430163&r2=430164
==============================================================================
--- trunk/main/rtp_engine.c (original)
+++ trunk/main/rtp_engine.c Wed Dec 31 12:54:37 2014
@@ -2189,7 +2189,8 @@
 	add_static_payload(125, ast_format_slin44, 0);
 	add_static_payload(126, ast_format_slin48, 0);
 	add_static_payload(127, ast_format_slin96, 0);
-	add_static_payload(128, ast_format_slin192, 0);
+	/* payload types above 127 are not valid */
+	add_static_payload(96, ast_format_slin192, 0);
 	/* Opus and VP8 */
 	add_static_payload(100, ast_format_vp8, 0);
 	add_static_payload(107, ast_format_opus, 0);