[asterisk-bugs] [Asterisk 0018146]: ast_app_dtget inconsistency

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18146 
====================================================================== 
Reported By:                dimas
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18146
Category:                   Core/General
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.2.13 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-10-16 09:13 CDT
Last Modified:              2010-10-16 09:13 CDT
====================================================================== 
Summary:                    ast_app_dtget inconsistency
Description: 
I noticed strange thing in the code decided to share. 1.6.2.13, main/app.c,
ast_app_dtget function at the very beginning it tries to configure the
timeout unless it is specified:

        if (!timeout && chan->pbx) {
                timeout = chan->pbx->dtimeoutms / 1000.0;
        } else if (!timeout) {
                timeout = 5;
        }

Both branches indicate  the final value for timeout is in seconds. However
I believe ast_waitfordigit accepts timeout in milliseconds.
And as I see when ast_app_dtget is called by builtin_atxfer, the timeout
passed is also in ms. So I guess the code above may be wrong.

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-10-16 09:13 dimas          Regression                => No              
2010-10-16 09:13 dimas          SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================