[asterisk-commits] elguero: branch 1.8 r375594 - /branches/1.8/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: elguero
Date: Fri Nov  2 11:58:14 2012
New Revision: 375594

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=375594
Log:
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.

Modified:
    branches/1.8/channels/chan_sip.c

Modified: branches/1.8/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/channels/chan_sip.c?view=diff&rev=375594&r1=375593&r2=375594
==============================================================================
--- branches/1.8/channels/chan_sip.c (original)
+++ branches/1.8/channels/chan_sip.c Fri Nov  2 11:58:14 2012
@@ -9216,8 +9216,10 @@
 			/* 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)) {