[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:              02-19-2008 10:35 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 - 02-19-08 10:35  
---------------------------------------------------------------------- 
That code isn't necessary, reading the code shows that
ast_async_parseable_goto always returns 0, so I simplified the patch.

Please delete the two older patches: app_channelredirect.c.trunk.patch and
app_channelredirect.patch

I also removed the NOARGS, BADARGS, BADEXTENSION an FAIL as results
leaving only
NOCHANNEL and SUCCESS. This is because everything else could be done in
dialplan.

That means a minimum of changes, but still the relevant and important
one.
Catch the possibility that the channel has gone away... 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
02-19-08 10:35  johan          Note Added: 0082567                          
======================================================================