fixes signed to unsigned int comparision issue for FaxMaxDatagram value.
authorDavid Vossel <dvossel@digium.com>
Wed, 3 Mar 2010 00:18:28 +0000 (00:18 +0000)
committerDavid Vossel <dvossel@digium.com>
Wed, 3 Mar 2010 00:18:28 +0000 (00:18 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@250246 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_sip.c

index 9944c00..cb7fba9 100644 (file)
@@ -7557,7 +7557,7 @@ static int process_sdp_a_image(const char *a, struct sip_pvt *p)
                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;
                }