[asterisk-bugs] [Asterisk 0015672]: ? in register= string breaks registration

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15672 
====================================================================== 
Reported By:                test011
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   15672
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.0
Asterisk Version:           1.6.2.0-beta4 
Regression:                 Yes 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-08-07 00:42 CDT
Last Modified:              2009-08-19 16:22 CDT
====================================================================== 
Summary:                    ? in register= string breaks registration
Description: 
If you put '?' (w/o quotation) in register= string in sip.conf, asterisk
does not parse it properly.

One of my register= string has '?' in its password section, and asterisk
can't parse it properly. This does not affect other lines in sip.conf

The string I used is as bellow,

register =>xxx at xxx.com:abc!5j5?!9!:xxx at hostip/123

Does '?' have been used as some token? Maybe I did not get that. I am
pretty sure I haven't seen it in change.log or sip.conf.sample.

The registration strings have worked fine with version 1.4.2, 1.6.0, and
1.6.1.0.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0014344 [patch] Outbound proxy not used for reg...
related to          0015008 [patch] Introduce better parsing for th...
====================================================================== 

---------------------------------------------------------------------- 
 (0109297) svnbot (reporter) - 2009-08-19 16:22
 https://issues.asterisk.org/view.php?id=15672#c109297 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 213117

_U  branches/1.6.2/
U   branches/1.6.2/channels/chan_sip.c
U   branches/1.6.2/configs/sip.conf.sample

------------------------------------------------------------------------
r213117 | tilghman | 2009-08-19 16:22:36 -0500 (Wed, 19 Aug 2009) | 16
lines

Merged revisions 213098 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r213098 | tilghman | 2009-08-19 16:05:17 -0500 (Wed, 19 Aug 2009) | 9
lines
  
  Better parsing for the "register" line
  Allows characters that are otherwise used as delimiters to be used
within
  certain fields (like the secret).
  (closes issue https://issues.asterisk.org/view.php?id=15008, closes issue
https://issues.asterisk.org/view.php?id=15672)
   Reported by: tilghman
   Patches: 
         20090818__issue15008.diff.txt uploaded by tilghman (license 14)
   Tested by: lmadsen, tilghman
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=213117 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-08-19 16:22 svnbot         Checkin                                      
2009-08-19 16:22 svnbot         Note Added: 0109297                          
======================================================================