[asterisk-dev] [Code Review] 2917: Correct behavior of ARI DELETE bridge command when channels are in the bridge.

[opticron]

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


This sounds like a deficiency in ast_bridge_destroy() since it does not remove channels properly if they are departable.

- opticron


On Oct. 16, 2013, 12:39 p.m., Mark Michelson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2917/
> -----------------------------------------------------------
> 
> (Updated Oct. 16, 2013, 12:39 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22634
>     https://issues.asterisk.org/jira/browse/ASTERISK-22634
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> The ARI command to delete a bridge did not work properly when there were channels in the bridge. The reason is that the channels in the bridge were imparted and marked as departable. In order for the refcounts to work out properly when the bridge is removed, the chaannels need to be departed from the bridge.
> 
> The patch here is pretty simple, but I figured I'd put it up for review since it's my first foray into the land of ARI, and there may be some nuances that I have overlooked.
> 
> 
> Diffs
> -----
> 
>   /branches/12/res/res_stasis.c 401015 
> 
> Diff: https://reviewboard.asterisk.org/r/2917/diff/
> 
> 
> Testing
> -------
> 
> See https://reviewboard.asterisk.org/r/2818 for a testsuite test that exercises this.
> 
> 
> Thanks,
> 
> Mark Michelson
> 
>

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