diff options
author | Richard Mudgett <rmudgett@digium.com> | 2017-11-26 09:44:51 -0600 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2017-11-26 09:51:59 -0600 |
commit | 55c4d8e00875cf1eb24f42f24379ad48c374b3c4 (patch) | |
tree | 16943b8e622d6e68009a2a8c8cda6f508d0d1c87 /res/ari | |
parent | 16381e54e4490a0a452bd77ec162ca5e21adf73e (diff) |
res_ari: Fix inverted test giving wrong error message.
The patch for ASTERISK_24560 inverted a test checking if the bridge name
is being updated to a different name.
* Fix the test to return "Changing bridge name is not implemented" when
someone attempts to change the bridge name.
ASTERISK-27445
Change-Id: I4b70bf08b0e02e016108b077ff75b345dec12fc9
Diffstat (limited to 'res/ari')
-rw-r--r-- | res/ari/resource_bridges.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/res/ari/resource_bridges.c b/res/ari/resource_bridges.c index cd289aa12..a320c2987 100644 --- a/res/ari/resource_bridges.c +++ b/res/ari/resource_bridges.c @@ -968,13 +968,12 @@ void ast_ari_bridges_create_with_id(struct ast_variable *headers, if (bridge) { /* update */ - if (!ast_strlen_zero(args->name)) { - if (!strcmp(args->name, bridge->name)) { - ast_ari_response_error( - response, 500, "Internal Error", - "Changing bridge name is not implemented"); - return; - } + if (!ast_strlen_zero(args->name) + && strcmp(args->name, bridge->name)) { + ast_ari_response_error( + response, 500, "Internal Error", + "Changing bridge name is not implemented"); + return; } if (!ast_strlen_zero(args->type)) { ast_ari_response_error( |