Summary:ASTERISK-02518: Has(New)VoiceMail does not assume INBOX if no folder specified
Reporter:kdau (kdau)Labels:
Date Opened:2004-10-02 19:26:14Date Closed:2008-01-15 15:09:00.000-0600
Versions:Frequency of
Environment:Attachments:( 0) 20041003__fix_2559.diff.txt
( 1) app_hasnewvoicemail-error.out
( 2) jamesfix.diff.txt
Description:Contrary to documentation and the apparent intent of the code, the HasVoiceMail and HasNewVoiceMail applications do not assume the INBOX folder if none is specified in their invocation. Reliably repeatable on Gentoo Linux with 1.0.1 and CVS head from today.


Relevant verbose console output from errant invocation attached. Error message suggests folder variable stays null.
Comments:By: kdau (kdau) 2004-10-02 19:28:13

Invocation that caused the error was:
 exten => s,1,HasNewVoicemail(100${c_line})
Where $c_line was set to "3".

By: James Golovich (jamesgolovich) 2004-10-03 01:36:51

Fixed in CVS

By: Tilghman Lesher (tilghman) 2004-10-03 08:52:34

I think there's a better fix to be had (i.e. using strsep correctly).  See attached patch.

By: Mark Spencer (markster) 2004-10-03 09:08:48

James' fix is just fine.  Short and to the point.

By: Mark Spencer (markster) 2004-10-03 09:09:46

Oops need to attach patch for Russell to put in 1.0 though...

By: Russell Bryant (russell) 2004-10-03 12:42:13

added to the 1.0 branch, thanks!

By: Digium Subversion (svnbot) 2008-01-15 15:09:00.000-0600

Repository: asterisk
Revision: 3899

U   branches/v1-0/apps/app_hasnewvoicemail.c

r3899 | russell | 2008-01-15 15:08:59 -0600 (Tue, 15 Jan 2008) | 2 lines

Has(New)VoiceMail does not assume INBOX if no folder specified (bug ASTERISK-2518)