[asterisk-bugs] [Asterisk 0013658]: WAITSTATUS will never get set for digitally muted channels

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://bugs.digium.com/view.php?id=13658 
====================================================================== 
Reported By:                explidous
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13658
Category:                   Applications/app_waitforsilence
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.4.22 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-10-09 15:46 CDT
Last Modified:              2008-10-09 15:46 CDT
====================================================================== 
Summary:                    WAITSTATUS will never get set for digitally muted
channels
Description: 
Basically line 110 in app_waitforsilence.c should be:

if (res < 0) { 

and line 116 should be :

if (res == 0) {

On line 110 it is checking to see if the result from ast_waitfor is less
than or equal to 0 if so it exits assuming the channel has hungup. If not
on line 116 it checks to see if the result is not true if so it assumes it
is a digitally muted channel and the wait lasted the appropriate period of
time. It then goes on to set the channel variable WAITSTATUS equal to
SILENCE. WAITSTATUS can never be set to SILENCE for a digitally muted
channel though because line 110 will caused the application to exit. The
interesting way that line 116 current checks to see if the result was 0
probably led to this in the first place.
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-09 15:46 explidous      Asterisk Version          => 1.4.22          
2008-10-09 15:46 explidous      SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================