[asterisk-bugs] [Asterisk 0011402]: Interdigit timeout is half time of the defined time

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11402 
====================================================================== 
Reported By:                eferro
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11402
Category:                   Channels/chan_mgcp
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 89965 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             11-28-2007 12:37 CST
Last Modified:              12-01-2007 01:58 CST
====================================================================== 
Summary:                    Interdigit timeout is half time of the defined time
Description: 
The interdigit timeout at mgcp_ss function of channels/chan_mgcp.c is not
the defined time (at the same file). Is just the half part because the
loop_pause is substract twice for each loop pass.

It very easy to test it because if you change the timeout, allways get a
time out interdigit at twice faster than the second specified.

I test it in 1.4.14, 1.4 branch and trunk, and with the same result.


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

---------------------------------------------------------------------- 
 eferro - 12-01-07 01:58  
---------------------------------------------------------------------- 
I think the correct patch is half_timeout-1.4.v2.patch  because in the loop
if you already subtract from timeout the loop pause, then you should only
check the timeout value in the condition 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
12-01-07 01:58  eferro         Note Added: 0074626                          
======================================================================