[asterisk-bugs] [Asterisk 0010502]: Check for callerid being present is incomplete.

[noreply at bugs.digium.com]

The following issue has been ASSIGNED. 
====================================================================== 
http://bugs.digium.com/view.php?id=10502 
====================================================================== 
Reported By:                dswartz
Assigned To:                dwaynemh
====================================================================== 
Project:                    Asterisk
Issue ID:                   10502
Category:                   Applications/app_zapateller
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.4.10.1  
SVN Branch (only for SVN checkouts, not tarball releases): N/A  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             08-20-2007 16:42 CDT
Last Modified:              08-24-2007 17:31 CDT
====================================================================== 
Summary:                    Check for callerid being present is incomplete.
Description: 

The check currently is:

if (chan->cid.cid_num && nocallerid) {

This is fine if there never was a CID number.  If there was a string
present, but it was nulled out by an AGI script (or some context code),
this pointer will still be non-null, but will point (correctly) to a null
string.  Check should be more like:

if (chan->cid.cid_num && strlen(chan->cid.cid_num) && nocallerid) {

p.s. I have submitted a license and am waiting for approval.
====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
08-24-07 17:31  dwaynemh       Status                   new => assigned     
08-24-07 17:31  dwaynemh       Assigned To               => dwaynemh        
======================================================================