[asterisk-bugs] [Asterisk 0015999]: [patch] Strange nasty sound (Because Asterisk tryes to handle new voicemail, but there is no voicemails, voicemail isn't used)

[Asterisk Bug Tracker]

The following issue is now READY FOR TESTING. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15999 
====================================================================== 
Reported By:                KrisV
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15999
Category:                   Channels/chan_dahdi
Reproducibility:            sometimes
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.6.1.5 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-10-01 04:47 CDT
Last Modified:              2009-10-15 11:11 CDT
====================================================================== 
Summary:                    [patch] Strange nasty sound (Because Asterisk tryes
to handle new voicemail, but there is no voicemails, voicemail isn't used)
Description: 
chan_dahdi.c: 8369
  res = has_voicemail(last);
  if (last->msgstate != res) {
  /* This channel has a new voicemail state,					*  initiate a thread to
send an MWI message */
  ...
  }
But in comments for msgstate:
/*! \brief -1 = unknown, 0 = no messages, 1 = new messages available */
int msgstate;
So, if res=0 and last->msgstate = -1 there is a new message! But it's not
true.


So, more logical to fix like:
  res = has_voicemail(last);
  if (last->msgstate != res && res > 0) {
  /* This channel has a new voicemail state,					* initiate a thread to
send an MWI message */
  ...
  }
In this case the strange sound doesn't appear.


====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-10-15 11:11 ebroad         Status                   needs license => ready
for testing
======================================================================