[asterisk-commits] dvossel: trunk r250246 - /trunk/channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Mar 2 18:18:32 CST 2010


Author: dvossel
Date: Tue Mar  2 18:18:28 2010
New Revision: 250246

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=250246
Log:
fixes signed to unsigned int comparision issue for FaxMaxDatagram value.

Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_sip.c?view=diff&rev=250246&r1=250245&r2=250246
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Tue Mar  2 18:18:28 2010
@@ -7557,7 +7557,7 @@
 		found = TRUE;
 	} else if ((sscanf(a, "T38FaxMaxDatagram:%30u", &x) == 1) || (sscanf(a, "T38MaxDatagram:%30u", &x) == 1)) {
 		/* override the supplied value if the configuration requests it */
-		if (p->t38_maxdatagram > x) {
+		if (((signed int) p->t38_maxdatagram >= 0) && ((unsigned int) p->t38_maxdatagram > x)) {
 			ast_debug(1, "Overriding T38FaxMaxDatagram '%d' with '%d'\n", x, p->t38_maxdatagram);
 			x = p->t38_maxdatagram;
 		}




More information about the asterisk-commits mailing list