diff options
author | Mark Michelson <mmichelson@digium.com> | 2013-08-29 15:43:23 +0000 |
---|---|---|
committer | Mark Michelson <mmichelson@digium.com> | 2013-08-29 15:43:23 +0000 |
commit | 0bc2a773657fd2d3d8841e2421ca99b486311204 (patch) | |
tree | f085737c3b114fea03b337ddbe9c043811a0e929 /main | |
parent | 72cf2779e8a35c1bfc5cc63a7991c8900a712d5c (diff) |
Multiple revisions 397921-397922
........
r397921 | mmichelson | 2013-08-29 10:42:10 -0500 (Thu, 29 Aug 2013) | 6 lines
Resolve assumptions that bridge snapshots would be non-NULL for transfer stasis events.
Attempting to transfer an unbridged call would result in crashes in either CEL code or
in the conversion to AMI messages.
........
r397922 | mmichelson | 2013-08-29 10:42:29 -0500 (Thu, 29 Aug 2013) | 3 lines
Remove extra debug message.
........
Merged revisions 397921-397922 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@397923 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main')
-rw-r--r-- | main/cel.c | 19 | ||||
-rw-r--r-- | main/stasis_bridges.c | 12 |
2 files changed, 26 insertions, 5 deletions
diff --git a/main/cel.c b/main/cel.c index e063fab9b..c4b80b5c1 100644 --- a/main/cel.c +++ b/main/cel.c @@ -1197,10 +1197,24 @@ static void cel_blind_transfer_cb( struct ast_channel_snapshot *chan_snapshot = obj->channel; struct ast_bridge_snapshot *bridge_snapshot = obj->bridge; struct ast_json *blob = obj->blob; - struct ast_json *json_exten = ast_json_object_get(blob, "exten"); - struct ast_json *json_context = ast_json_object_get(blob, "context"); + struct ast_json *json_result = ast_json_object_get(blob, "result"); + struct ast_json *json_exten; + struct ast_json *json_context; RAII_VAR(struct ast_json *, extra, NULL, ast_json_unref); const char *exten, *context; + enum ast_transfer_result result; + + if (!json_result) { + return; + } + + result = ast_json_integer_get(json_result); + if (result != AST_BRIDGE_TRANSFER_SUCCESS) { + return; + } + + json_exten = ast_json_object_get(blob, "exten"); + json_context = ast_json_object_get(blob, "context"); if (!json_exten || !json_context) { return; @@ -1211,6 +1225,7 @@ static void cel_blind_transfer_cb( if (!exten || !context) { return; } + extra = ast_json_pack("{s: s, s: s, s: s}", "extension", exten, "context", context, diff --git a/main/stasis_bridges.c b/main/stasis_bridges.c index be1294ad0..0ff9ccaab 100644 --- a/main/stasis_bridges.c +++ b/main/stasis_bridges.c @@ -585,11 +585,17 @@ static struct ast_manager_event_blob *blind_transfer_to_ami(struct stasis_messag } channel_state = ast_manager_build_channel_state_string_prefix(blob->channel, "Transferer"); - bridge_state = ast_manager_build_bridge_state_string(blob->bridge); - if (!channel_state || !bridge_state) { + if (!channel_state) { return NULL; } + if (blob->bridge) { + bridge_state = ast_manager_build_bridge_state_string(blob->bridge); + if (!bridge_state) { + return NULL; + } + } + exten = ast_json_string_get(ast_json_object_get(blob->blob, "exten")); context = ast_json_string_get(ast_json_object_get(blob->blob, "context")); result = ast_json_integer_get(ast_json_object_get(blob->blob, "result")); @@ -604,7 +610,7 @@ static struct ast_manager_event_blob *blind_transfer_to_ami(struct stasis_messag "Extension: %s\r\n", result_strs[result], ast_str_buffer(channel_state), - ast_str_buffer(bridge_state), + bridge_state ? ast_str_buffer(bridge_state) : "", is_external ? "Yes" : "No", context, exten); |