[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 has been CLOSED 
====================================================================== 
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:                     closed
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:              
Resolution:                 open
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-10-01 04:47 CDT
Last Modified:              2010-07-07 08:22 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.


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

---------------------------------------------------------------------- 
 (0124296) lmadsen (administrator) - 2010-07-07 08:22
 https://issues.asterisk.org/view.php?id=15999#c124296 
---------------------------------------------------------------------- 
Based on the comments in this issue, it appears you are working on
third-party software which cannot be supported here. For additional help
with code modifications, please utilize the asterisk-dev mailing list. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-07-07 08:22 lmadsen        Note Added: 0124296                          
2010-07-07 08:22 lmadsen        Status                   ready for testing =>
closed
======================================================================