[asterisk-commits] oej: trunk r171528 - in /trunk: ./ channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jan 27 09:00:19 CST 2009


Author: oej
Date: Tue Jan 27 09:00:19 2009
New Revision: 171528

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=171528
Log:
Solving the same issue, but a bit different in trunk...

Merged revisions 171527 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r171527 | oej | 2009-01-27 15:33:20 +0100 (Tis, 27 Jan 2009) | 13 lines

Use the same branch tag in CANCEL as in INVITE

Originally putnopvut implemented some changes in revision 142079 that according to the bug report seemed to have worked then, but somehow fails now.
I guess code, as humans, get old and forget stuff. Anyway, this bug caused CANCEL not to work with picky systems. 

Thanks Fredrik for pointing out where the bug in the SIP messaging was.

(closes issue #14346)
Reported by: oej
Patches: 
      bug14346.diff uploaded by oej (license 306)
Tested by: oej

........

Modified:
    trunk/   (props changed)
    trunk/channels/chan_sip.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/svn-view/asterisk/trunk/channels/chan_sip.c?view=diff&rev=171528&r1=171527&r2=171528
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Tue Jan 27 09:00:19 2009
@@ -9817,7 +9817,8 @@
 	if (init > 1)
 		initreqprep(&req, p, sipmethod);
 	else
-		reqprep(&req, p, sipmethod, 0, 1);
+		/* If init=1, we should not generate a new branch. If it's 0, we need a new branch. */
+		reqprep(&req, p, sipmethod, 0, init ? 0 : 1);
 		
 	if (p->options && p->options->auth)
 		add_header(&req, p->options->authheader, p->options->auth);




More information about the asterisk-commits mailing list