[asterisk-commits] twilson: branch 1.8 r348833 - /branches/1.8/include/asterisk/frame.h

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Dec 22 12:38:53 CST 2011


Author: twilson
Date: Thu Dec 22 12:38:46 2011
New Revision: 348833

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=348833
Log:
Allow packetization vaules > 127

According to the RTP packetization documentation, and the maximum values
listed in AST_FORMAT_LIST, we should support values > that the signed
char array that ast_codec_pref makes available to store the value. All
places in the code treat the framing field as though it were an int
array instaead of a char array anyway, so this just fixes the type of
the array.

(closes issue ASTERISK-18876)
Review: https://reviewboard.asterisk.org/r/1639/

Modified:
    branches/1.8/include/asterisk/frame.h

Modified: branches/1.8/include/asterisk/frame.h
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/include/asterisk/frame.h?view=diff&rev=348833&r1=348832&r2=348833
==============================================================================
--- branches/1.8/include/asterisk/frame.h (original)
+++ branches/1.8/include/asterisk/frame.h Thu Dec 22 12:38:46 2011
@@ -37,7 +37,7 @@
 
 struct ast_codec_pref {
 	char order[sizeof(format_t) * 8];
-	char framing[sizeof(format_t) * 8];
+	int framing[64]; /*!< Magic numbers are bad, but this just needs to be bigger than ARRAY_LEN(AST_FORMAT_LIST) */
 };
 
 /*!




More information about the asterisk-commits mailing list