[asterisk-bugs] [Asterisk 0014599]: searchcontexts=yes causes voicemail boxes to be setup wrong

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14599 
====================================================================== 
Reported By:                lmadsen
Assigned To:                mmichelson
====================================================================== 
Project:                    Asterisk
Issue ID:                   14599
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Target Version:             1.4.24
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 179056 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-03-03 17:36 CST
Last Modified:              2009-03-05 13:00 CST
====================================================================== 
Summary:                    searchcontexts=yes causes voicemail boxes to be
setup wrong
Description: 
In issue 13853 I thought the following note was an issue with IMAP, but it
seems it is an issue regardless of whether you're using IMAP. The issue
comes from the searchcontext=yes option being enabled. The text in the note
I posted to issue http://bugs.digium.com/view.php?id=13853 is still relevant.
====================================================================== 

---------------------------------------------------------------------- 
 (0101273) svnbot (reporter) - 2009-03-05 13:00
 http://bugs.digium.com/view.php?id=14599#c101273 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 180380

U   branches/1.4/apps/app_voicemail.c
U   branches/1.4/configs/voicemail.conf.sample

------------------------------------------------------------------------
r180380 | mmichelson | 2009-03-05 12:58:48 -0600 (Thu, 05 Mar 2009) | 25
lines

Fix broken mailbox parsing when searchcontexts option is enabled.

When using the searchcontexts option in voicemail.conf, the code
made the assumption that all mailbox names defined were unique across
all contexts. However, the code did nothing to actually enforce this
assumption, nor did it do anything to alert a user that he may have
created an ambiguity in his voicemail.conf file by defining the same
mailbox name in multiple contexts.

With this change, we now will issue a nice long warning if searchcontexts
is on and we encounter the same mailbox name in multiple contexts and
ignore
any duplicates after the first box. Whether searchcontexts is enabled or
not,
if we come across a duplicate mailbox in the same context, then we will
issue
a warning and ignore the duplicated mailbox. I have also added a small
note
to voicemail.conf.sample in the explanation for searchcontexts explaining
that you cannot define the same mailbox in multiple contexts if you have
enabled the option.

(closes issue http://bugs.digium.com/view.php?id=14599)
Reported by: lmadsen
Patches:
      14599.patch uploaded by mmichelson (license 60) (with slight
modification)
Tested by: lmadsen


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=180380 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-03-05 13:00 svnbot         Checkin                                      
2009-03-05 13:00 svnbot         Note Added: 0101273                          
======================================================================