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

[Mark Michelson]

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

Ship it!


Ship It!

- Mark Michelson


On Oct. 17, 2013, 11:22 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2928/
> -----------------------------------------------------------
> 
> (Updated Oct. 17, 2013, 11:22 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> 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/c6d1b52e/attachment.html>