[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