[asterisk-bugs] [Asterisk 0015726]: password change for mailboxes without user name

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15726 
====================================================================== 
Reported By:                298
Assigned To:                lmadsen
====================================================================== 
Project:                    Asterisk
Issue ID:                   15726
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.4.26 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-08-16 05:16 CDT
Last Modified:              2009-08-19 18:07 CDT
====================================================================== 
Summary:                    password change for mailboxes without user name
Description: 
Asterisk seems to have no real problem with mail boxes defined without a
name,
for example:
23 => 123,
24 => 456

Both can successfully change their password. However, in the second case
where the comma is missing, an error (see below) is displayed in the
console and the password is updated only in memory, not in voicemail.conf.

WARNING[17638]: app_voicemail.c:843 vm_change_password: variable has bad
format.
====================================================================== 

---------------------------------------------------------------------- 
 (0109313) qwell (administrator) - 2009-08-19 18:07
 https://issues.asterisk.org/view.php?id=15726#c109313 
---------------------------------------------------------------------- 
I see where this is happening in the code, and I see why it gives that
error message.  I'm not sure how to get around it though.  If this is fixed
by setting the password regardless of the return of the strstr, one must
consider the following config line:

6000 => 123 // This comment MUST NOT be eaten. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-08-19 18:07 qwell          Note Added: 0109313                          
======================================================================