[asterisk-dev] [Code Review] 2928: Fix orphaned bridge if neither of the joining channels can join.

[svnbot]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2928/
-----------------------------------------------------------

(Updated Oct. 22, 2013, 12:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 401424


Bugs: ASTERISK-22629
    https://issues.asterisk.org/jira/browse/ASTERISK-22629


Repository: Asterisk


Description
-------

The original issue noted that the bridge is orphaned when res_parking.so is not loaded and a call uses the dial kK flags.

A similar issue happens when only one of the park flags is used.  In this case you have the bridge with one or the other channel left in it.  The channel and bridge will stay around until the channel hangs up.

* Fixed the initial bridge channel push failure to act as if the channel were kicked out of the bridge.  The bridge then decides if it needs to be dissolved.


Diffs
-----

  /branches/12/include/asterisk/bridge_channel_internal.h 401177 
  /branches/12/main/bridge.c 401177 
  /branches/12/main/bridge_channel.c 401177 

Diff: https://reviewboard.asterisk.org/r/2928/diff/


Testing
-------

Tested with res_parking.so unloaded with Caller dialing Callee
and dial kK flags:
  The bridge is no longer left lying around and saw the three case of the channels attempting to join and being bounced out.
  1) Caller attempts to join first
  2) Callee attempts to join first
  3) Caller and Callee join at the same time
and dial K flag:
  The bridge and channels are no longer left in the following cases
  1) Caller attempts to join first
  2) Callee joins first while Caller fails to join
and dial k flag:
  The bridge and channels are no longer left in the following cases
  1) Caller joins first while Callee fails to join
  2) Callee attempts to join first


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131022/81078947/attachment.html>