[asterisk-bugs] [Asterisk 0018211]: Channel hangs up when redirected through CLI or AMI

Mon Nov 22 13:28:34 CST 2010

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18211 
====================================================================== 
Reported By:                zahir_koradia
Assigned To:                rmudgett
====================================================================== 
Project:                    Asterisk
Issue ID:                   18211
Category:                   Channels/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 293045 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-10-26 01:58 CDT
Last Modified:              2010-11-22 13:28 CST
====================================================================== 
Summary:                    Channel hangs up when redirected through CLI or AMI
Description: 
SVN revision 293045:

When a channel is redirected using either CLI or AMI, the channel hangs
up. This has been tested for GTalk and SIP channels. Redirecting any of
these two channels to console channel, musiconhold, or SIP channel triggers
the above bug. The bug has not been tested for other channel combinations.

The same bug was reported once as issue
https://issues.asterisk.org/view.php?id=18171 (by SantaFox), which is no
longer accessible on the tracker. Google cache has a copy of it at 

http://webcache.googleusercontent.com/search?q=cache:tcBAw2AF69EJ:https://issues.asterisk.org/print_bug_page.php%3Fbug_id%3D18171+asterisk+issue+18171&cd=1&hl=en&ct=clnk
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0018230 [regression] Redirect function (over co...
====================================================================== 

---------------------------------------------------------------------- 
 (0129039) svnbot (reporter) - 2010-11-22 13:28
 https://issues.asterisk.org/view.php?id=18211#c129039 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 295843

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_macro.c
U   branches/1.6.2/include/asterisk/channel.h
U   branches/1.6.2/include/asterisk/frame.h
U   branches/1.6.2/main/channel.c
U   branches/1.6.2/main/pbx.c

------------------------------------------------------------------------
r295843 | rmudgett | 2010-11-22 13:28:27 -0600 (Mon, 22 Nov 2010) | 53
lines

Merged revisions 295790 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r295790 | rmudgett | 2010-11-22 12:46:26 -0600 (Mon, 22 Nov 2010) | 46
lines

  The channel redirect function (CLI or AMI) hangs up the call instead of
redirecting the call.

  To recreate the problem:
  1) Party A calls Party B
  2) Invoke CLI "channel redirect" command to redirect channel call leg
  associated with A.
  3) All associated channels are hung up.

  Note that if the CLI command were done on the channel call leg
associated
  with B it works.

  This regression was a result of the fix for issue
https://issues.asterisk.org/view.php?id=16946
  (https://reviewboard.asterisk.org/r/740/).

  The regression affects all features that use an async goto to execute
the
  dialplan because of an external event: Channel redirect, AMI redirect,
SIP
  REFER, and FAX detection.

  The struct ast_channel._softhangup code is a mess.  The variable is used
  for several purposes that do not necessarily result in the call being
hung
  up.  I have added doxygen comments to describe how the various
_softhangup
  bits are used.  I have corrected all the places where the variable was
  tested in a non-bit oriented manner.

  The primary fix is the new AST_CONTROL_END_OF_Q frame.  It acts as a
weak
  hangup request so the soft hangup requests that do not normally result
in
  a hangup do not hangup.

  JIRA SWP-2470
  JIRA SWP-2489

  (closes issue https://issues.asterisk.org/view.php?id=18171)
  Reported by: SantaFox
  (closes issue https://issues.asterisk.org/view.php?id=18185)
  Reported by: kwemheuer
  (closes issue https://issues.asterisk.org/view.php?id=18211)
  Reported by: zahir_koradia
  (closes issue https://issues.asterisk.org/view.php?id=18230)
  Reported by: vmarrone
  (closes issue https://issues.asterisk.org/view.php?id=18299)
  Reported by: mbrevda
  (closes issue https://issues.asterisk.org/view.php?id=18322)
  Reported by: nerbos

  Review:	https://reviewboard.asterisk.org/r/1013/
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-11-22 13:28 svnbot         Checkin                                      
2010-11-22 13:28 svnbot         Note Added: 0129039                          
======================================================================