[asterisk-commits] mmichelson: branch 1.6.0 r123335 - in /branches/1.6.0: ./ channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Tue Jun 17 13:10:30 2008
New Revision: 123335

URL: http://svn.digium.com/view/asterisk?view=rev&rev=123335
Log:
Merged revisions 123334 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r123334 | mmichelson | 2008-06-17 13:09:54 -0500 (Tue, 17 Jun 2008) | 19 lines

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

........
r123333 | mmichelson | 2008-06-17 13:09:16 -0500 (Tue, 17 Jun 2008) | 11 lines

Cisco BTS sends SIP responses with a tab between the Cseq number and
SIP request method in the Cseq: header. Asterisk did not handle this
properly, but with this patch, all is well.

(closes issue #12834)
Reported by: tobias_e
Patches:
      12834.patch uploaded by putnopvut (license 60)
Tested by: tobias_e


........

................

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

Propchange: branches/1.6.0/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.0/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/branches/1.6.0/channels/chan_sip.c?view=diff&rev=123335&r1=123334&r2=123335
==============================================================================
--- branches/1.6.0/channels/chan_sip.c (original)
+++ branches/1.6.0/channels/chan_sip.c Tue Jun 17 13:10:30 2008
@@ -15150,12 +15150,12 @@
 	int sipmethod;
 	int res = 1;
 	const char *c = get_header(req, "Cseq");
-	const char *msg = strchr(c, ' ');
+	/* Skip the Cseq and its subsequent spaces */
+	const char *msg = ast_skip_blanks(ast_skip_nonblanks((char *)c));
 
 	if (!msg)
 		msg = "";
-	else
-		msg++;
+
 	sipmethod = find_sip_method(msg);
 
 	owner = p->owner;