[Asterisk-code-review] ARI: Ensure announcer channels are destroyed. (asterisk[13])

Richard Mudgett asteriskteam at digium.com
Fri Jun 17 17:03:46 CDT 2016


Richard Mudgett has posted comments on this change.

Change subject: ARI: Ensure announcer channels are destroyed.
......................................................................


Patch Set 3: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/3023/3/res/ari/resource_bridges.c
File res/ari/resource_bridges.c:

PS3, Line 471: 	if (!thread_data) {
             : 		ast_ari_response_alloc_failed(response);
             : 		return;
             : 	}
Oops here is another place where stasis_app_bridge_playback_channel_remove() needs to be called to clean up.


-- 
To view, visit https://gerrit.asterisk.org/3023
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0ddc0f64484ea0016245800b409b567dfe85cfb4
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list