[asterisk-bugs] [Asterisk 0011553]: [patch] ChannelRedirect causes the channel executing it to terminate if any form of error occurs.

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11553 
====================================================================== 
Reported By:                johan
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11553
Category:                   Applications/app_channelredirect
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.4.15  
SVN Branch (only for SVN checkouts, not tarball releases): N/A  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             12-14-2007 11:05 CST
Last Modified:              01-12-2008 16:27 CST
====================================================================== 
Summary:                    [patch] ChannelRedirect causes the channel executing
it to terminate if any form of error occurs.
Description: 
When you use ChannelRedirect on a channel that does not exist the call will
be terminated, and the execution of the dial plan will end. Instead only
the error message should be returned, and the execution of the dial plan
can then continue as it should.

The enclosed patch change the behaviour of ChannelRedirect to set the
channel variable CR_STATUS to either SUCCESS, NOARGS, BADARGS, NOCHANNEL,
BADEXTENSION or FAIL, so that your dial plan does not have to crash and
burn in agony.
====================================================================== 

---------------------------------------------------------------------- 
 johan - 01-12-08 16:27  
---------------------------------------------------------------------- 
Finally I've made the changes, please rename app_channelredirect.patch to
reflect that this is a patch for 1.4-trunk, I guess Asterisk version could
be changed to trunk as well.

I have tested the patch the best I could - however - it seems to a problem
with the function ast_parseable_goto that does the actual work. I can't get
either the original ChannelRedirect or my patched version to behave as it
does in 1.4. I thought I was implementing this badly, but the same problem
exists with the original version.

So this will work if we get ast_parseable_goto fixed! 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
01-12-08 16:27  johan          Note Added: 0076825                          
======================================================================