From: Michael L. Young <elgueromexicano@gmail.com>
Date: Fri, 2 Nov 2012 16:58:14 +0000 (+0000)
Subject: Fix Wrong Result In Debug Message For SDP Origin Processing
X-Git-Tag: 1.8.19.0-tc1~13
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=e949d5297ca8663ffffe6708c9e422c25b13b0db;p=thirdparty%2Fasterisk.git

Fix Wrong Result In Debug Message For SDP Origin Processing

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
---

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index ba7231ae25..5f68ff2d16 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -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)) {