[asterisk-bugs] [Asterisk 0015839]: caller id number is empty

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15839 
====================================================================== 
Reported By:                ebroad
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15839
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.2 
SVN Revision (number only!): 216594 
Request Review:              
====================================================================== 
Date Submitted:             2009-09-06 11:59 CDT
Last Modified:              2009-09-06 17:38 CDT
====================================================================== 
Summary:                    caller id number is empty
Description: 
Calling number is empty(defaults to default_callerid). See 
====================================================================== 

---------------------------------------------------------------------- 
 (0110257) ebroad (reporter) - 2009-09-06 17:38
 https://issues.asterisk.org/view.php?id=15839#c110257 
---------------------------------------------------------------------- 
The fix for this is pretty simple. The problem appeared at revision
215522(see
http://lists.digium.com/pipermail/asterisk-commits/2009-September/036703.html).
Essentially the code the parses the scheme in parse_uri fails to stripsoff
the ':' separating the scheme and the user at domain portion of the uri. This
caused some later code that split user:pass to set name to null, because
the name would be prepended with a ':'. The easy fix is to change 'url +=
l;' to 'url += (l + 1);' (which is what my patch does, pending approval),
however, the cleaner way to do this would be to check for a trailing ':'
when called(it is possible that sometime down the line, someone might call
parse_uri with sip:,sips:) and append one if there is none. Additionally,
this fixes '0015830: channel variable ${CALLERID(num)} is empty'. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-09-06 17:38 ebroad         Note Added: 0110257                          
======================================================================