[asterisk-commits] dvossel: branch 1.8 r287645 - /branches/1.8/channels/chan_sip.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Sep 20 16:34:20 CDT 2010


Author: dvossel
Date: Mon Sep 20 16:34:15 2010
New Revision: 287645

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=287645
Log:
Fixes issue with registrations not working properly with pedantic=yes.

(closes issue #18017)
Reported by: schmidts
Patches:
      issues_18017_v1.diff uploaded by dvossel (license 671)
Tested by: schmidts


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=287645&r1=287644&r2=287645
==============================================================================
--- branches/1.8/channels/chan_sip.c (original)
+++ branches/1.8/channels/chan_sip.c Mon Sep 20 16:34:15 2010
@@ -7236,8 +7236,10 @@
 			return SIP_REQ_NOT_MATCH;
 		}
 	} else {
-		/* Verify the fromtag of Request matches the tag they provided earlier. */
-		if (strcmp(arg->fromtag, sip_pvt_ptr->theirtag)) {
+		/* Verify the fromtag of Request matches the tag they provided earlier.
+		 * If this is a Request with authentication credentials, forget their old
+		 * tag as it is not valid after the 401 or 407 response. */
+		if (!arg->authentication_present && strcmp(arg->fromtag, sip_pvt_ptr->theirtag)) {
 			/* their tag does not match the one was have stored for them */
 			return SIP_REQ_NOT_MATCH;
 		}




More information about the asterisk-commits mailing list