[asterisk-bugs] [Asterisk 0015111]: [patch] parsing of sip register lines is broken

[Asterisk Bug Tracker]

The following issue has been RESOLVED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15111 
====================================================================== 
Reported By:                ffs
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   15111
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     resolved
Target Version:             1.6.2.0
Asterisk Version:           1.6.2.0-beta1 
Regression:                 No 
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-05-14 11:03 CDT
Last Modified:              2009-06-18 10:16 CDT
====================================================================== 
Summary:                    [patch] parsing of sip register lines is broken
Description: 
A register line within sip.conf has the following format:

[transport://]user[:secret[:authuser]]@domain[:port][/extension][~expiry]

To extract the extension the parser is searching for '/' in the whole line
(from right to left).
As [/extension] is optional this causes trouble if there is no extension
defined and [transport://] is set.

Suggested solution: parse extension and expiry after the last '@'
character.

A backport to affected older versions would be appreciated.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0014668 register: '/' in username not supported
has duplicate       0015342 parsing of sip register lines is still ...
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-06-18 10:16 svnbot         Status                   assigned => resolved
2009-06-18 10:16 svnbot         Fixed in Version         1.6.2.0 =>          
2009-06-18 10:16 svnbot         Resolution               reopened => fixed   
======================================================================