]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Fix Wrong Result In Debug Message For SDP Origin Processing
authorMichael L. Young <elgueromexicano@gmail.com>
Fri, 2 Nov 2012 16:58:14 +0000 (16:58 +0000)
committerMichael L. Young <elgueromexicano@gmail.com>
Fri, 2 Nov 2012 16:58:14 +0000 (16:58 +0000)
While looking at some debug logs, I noticed that it was being reported that the
SDP origin line was unsupported or failed.  Upon looking into this on my local
machine, I found that I too was getting this debug message yet everything seemed
to be getting processed properly.  What was discovered is, that, the variable to
determine what is displayed in the debug message for the SDP line that was
processed, was not being set for the origin line when the result was successful.

This patch fixes this and was tested on local machine.

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

channels/chan_sip.c

index ba7231ae25baa86d15f29d170b999f4e3847ba4e..5f68ff2d163481639dadacaf6528bfc29298aae3 100644 (file)
@@ -9216,8 +9216,10 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action
                        /* If we end up receiving SDP that doesn't actually modify the session we don't want to treat this as a fatal
                         * error. We just want to ignore the SDP and let the rest of the packet be handled as normal.
                         */
-                       if (!process_sdp_o(value, p))
+                       if (!process_sdp_o(value, p)) {
                                return (p->session_modify == FALSE) ? 0 : -1;
+                       }
+                       processed = TRUE;
                        break;
                case 'c':
                        if (process_sdp_c(value, &sessionsa)) {