[asterisk-bugs] [Asterisk 0016749]: [patch] DSP progress detection unable to detect SIT

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16749 
====================================================================== 
Reported By:                dant
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16749
Category:                   Core/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-02-02 00:34 CST
Last Modified:              2010-02-02 00:43 CST
====================================================================== 
Summary:                    [patch] DSP progress detection unable to detect SIT
Description: 
The DSP should be able to detect the SIT. The existing code checks for the
first tone followed by the second tone, followed by the third tone in
sequence and on detection of the third tone allows firing a CONGESTION
control frame.

This unfortunately doesn't work due to the fact that entire SIT is about 1
second long, the DSP is working with smaller samples and the code will not
stay in a second or third tone state beyond one pass.

This issue is being raised against trunk, but, the problem appears to be
present even in 1.2.
====================================================================== 

---------------------------------------------------------------------- 
 (0117496) dant (reporter) - 2010-02-02 00:43
 https://issues.asterisk.org/view.php?id=16749#c117496 
---------------------------------------------------------------------- 
Patch attached that allows the second and third tones to be present across
multiple dsp calls allowing the sequence to complete...

New debug output:
[Feb  2 14:10:20] DEBUG[11515] dsp.c: Stop state 0 with duration 0
[Feb  2 14:10:20] DEBUG[11515] dsp.c: Start state 5
[Feb  2 14:10:21] DEBUG[11515] dsp.c: Stop state 5 with duration 15
[Feb  2 14:10:21] DEBUG[11515] dsp.c: Start state 6
[Feb  2 14:10:21] DEBUG[11515] dsp.c: Stop state 6 with duration 15
[Feb  2 14:10:21] DEBUG[11515] dsp.c: Start state 7 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-02-02 00:43 dant           Note Added: 0117496                          
======================================================================