From: Terry Wilson <twilson@digium.com>
Date: Thu, 22 Dec 2011 18:38:46 +0000 (+0000)
Subject: Allow packetization vaules > 127
X-Git-Tag: 1.8.9.0-rc1~17
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=05078f24e1c45b9bdc61177668a8356edf95b176;p=thirdparty%2Fasterisk.git

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/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@348833 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/include/asterisk/frame.h b/include/asterisk/frame.h
index 07bad8ed88..6dcabda1f5 100644
--- a/include/asterisk/frame.h
+++ b/include/asterisk/frame.h
@@ -37,7 +37,7 @@ extern "C" {
 
 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) */
 };
 
 /*!