[asterisk-commits] elguero: branch 10 r375601 - in /branches/10: ./ channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Nov 2 12:19:37 CDT 2012


Author: elguero
Date: Fri Nov  2 12:19:33 2012
New Revision: 375601

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

Merged revisions 375594 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/channels/chan_sip.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/channels/chan_sip.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/channels/chan_sip.c?view=diff&rev=375601&r1=375600&r2=375601
==============================================================================
--- branches/10/channels/chan_sip.c (original)
+++ branches/10/channels/chan_sip.c Fri Nov  2 12:19:33 2012
@@ -9426,6 +9426,7 @@
 				res = (p->session_modify == FALSE) ? 0 : -1;
 				goto process_sdp_cleanup;
 			}
+			processed = TRUE;
 			break;
 		case 'c':
 			if (process_sdp_c(value, &sessionsa)) {




More information about the asterisk-commits mailing list