[asterisk-bugs] [Asterisk 0019136]: [patch] 'e' special extension fails to trigger in at least two cases

[Asterisk Bug Tracker]

The following issue is now READY FOR REVIEW. 
====================================================================== 
https://issues.asterisk.org/view.php?id=19136 
====================================================================== 
Reported By:                kshumard
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   19136
Category:                   PBX/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for review
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 314115 
Request Review:              
====================================================================== 
Date Submitted:             2011-04-18 14:45 CDT
Last Modified:              2011-04-18 16:01 CDT
====================================================================== 
Summary:                    [patch] 'e' special extension fails to trigger in at
least two cases
Description: 
https://wiki.asterisk.org/wiki/display/AST/The+Asterisk+Dialplan says, "e -
This extension will substitute as a catchall for any of the 'i', 't', or
'T' extensions, if any of them do not exist and catching the error in a
single routine is desired. The function EXCEPTION may be used to query the
type of exception or the location where it occurred."

The attached patch attempts to make this statement more true. In testing,
neither 'i' nor 't' was present but when the appropriate conditions were
met the channel didn't move it to the 'e' extension. The patch throws an
exception if WaitExten results in a timeout, and explicitly jumps to e,1
when an invalid extension is entered and there's no 'i' (though the
proposed patch might not be the proper way to accomplish this).

pbx_builtin_background should also throw an exception if it times out, but
currently doesn't. I just didn't know the appropriate logic to use at the
end of the function to add it in.
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-18 16:01 lmadsen        Status                   new => ready for review
======================================================================