[Home]

Summary:ASTERISK-05292: [patch] app_disa does not properly jump to the "i" extension if an invalid extension was received.
Reporter:Robert Christian (thetatag)Labels:
Date Opened:2005-10-13 19:49:21Date Closed:2008-01-15 15:52:02.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_disa
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_disa.c.diff
( 1) app_disa.c.patch.txt
Description:app_disa does not have any support for intercepting an invalid extension.  It doesn't continue on in the dialplan at the next priority from where it was called, it doesn't jump +101, and it doesn't do what I believe is the most reasonable behavior: go to the "i" extension (if any) in the target context.  For app_disa to properly replicate the behavior of dialing from a physical extension into Asterisk, it must support the "i" extension rather that just dropping to a congestion indication.

****** ADDITIONAL INFORMATION ******

My proposed patch will add the functionality to app_disa to jump to an "i" extension in the target context if the received extension is invalid.  If the "i" extension does not exist, app_disa will give the congestion indication as it does currently.
Comments:By: Kevin P. Fleming (kpfleming) 2005-10-13 19:58:20

Committed to CVS HEAD with minor mods, thanks!
By: Robert Christian (thetatag) 2005-10-14 18:50:44

Oops...Forgot to set ${INVALID_EXTEN} in my original patch.  A new patch has been uploaded that should correct this problem.


By: Mark Spencer (markster) 2005-10-22 21:46:36

Added to CVS head, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:51:01.000-0600

Repository: asterisk
Revision: 6779

U   trunk/apps/app_disa.c

------------------------------------------------------------------------
r6779 | kpfleming | 2008-01-15 15:51:00 -0600 (Tue, 15 Jan 2008) | 2 lines

make DISA() use 'i' extension if it exists (issue ASTERISK-5292 with mods)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=6779
By: Digium Subversion (svnbot) 2008-01-15 15:52:02.000-0600

Repository: asterisk
Revision: 6845

U   trunk/apps/app_disa.c

------------------------------------------------------------------------
r6845 | markster | 2008-01-15 15:52:01 -0600 (Tue, 15 Jan 2008) | 2 lines

Fix app_disa to set the proper variable before goign to invalid (bug ASTERISK-5292)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=6845