diff options
author | Richard Mudgett <rmudgett@digium.com> | 2014-08-14 16:33:27 +0000 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2014-08-14 16:33:27 +0000 |
commit | 79c5c08db9ff3f640e6464c385a0cc9baf13e656 (patch) | |
tree | f73f75dfc174256c3ff2cc9f65de77731e6b2473 /res/ari/resource_channels.c | |
parent | e4b32731b910cdbcc1f9de49dbc8b5fe34d938a5 (diff) |
ARI: Originate to app local channel subscription code optimization.
Reduce the scope of local_peer and only get it if the ARI originate is
subscribing to the channels.
Review: https://reviewboard.asterisk.org/r/3905/
........
Merged revisions 421009 from http://svn.asterisk.org/svn/asterisk/branches/12
........
Merged revisions 421010 from http://svn.asterisk.org/svn/asterisk/branches/13
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@421012 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/ari/resource_channels.c')
-rw-r--r-- | res/ari/resource_channels.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/res/ari/resource_channels.c b/res/ari/resource_channels.c index d0a1be32e..23f7fa0f6 100644 --- a/res/ari/resource_channels.c +++ b/res/ari/resource_channels.c @@ -746,7 +746,6 @@ static void ari_channels_handle_originate_with_id(const char *args_endpoint, ast_format_cap_alloc(AST_FORMAT_CAP_FLAG_DEFAULT), ao2_cleanup); char *stuff; struct ast_channel *chan; - struct ast_channel *local_peer; RAII_VAR(struct ast_channel_snapshot *, snapshot, NULL, ao2_cleanup); struct ast_assigned_ids assignedids = { .uniqueid = args_channel_id, @@ -831,13 +830,16 @@ static void ari_channels_handle_originate_with_id(const char *args_endpoint, return; } - /* See if this is a Local channel and if so, get the peer */ - local_peer = ast_local_get_peer(chan); - if (!ast_strlen_zero(args_app)) { + struct ast_channel *local_peer; + stasis_app_subscribe_channel(args_app, chan); + + /* Subscribe to the Local channel peer also. */ + local_peer = ast_local_get_peer(chan); if (local_peer) { stasis_app_subscribe_channel(args_app, local_peer); + ast_channel_unref(local_peer); } } @@ -846,9 +848,6 @@ static void ari_channels_handle_originate_with_id(const char *args_endpoint, ast_ari_response_ok(response, ast_channel_snapshot_to_json(snapshot, NULL)); ast_channel_unref(chan); - if (local_peer) { - ast_channel_unref(local_peer); - } } void ast_ari_channels_originate_with_id(struct ast_variable *headers, |