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:42 -0600 |
commit | 81e2d8aa9a2caa1854757717367b1504c2cd3199 (patch) | |
tree | 10fb09395dfdca0f011cdde73eba17fe3f86088e /res | |
parent | ee3562619c97035a3c30ad04c63db92520e53f49 (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')
-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 d6a0400c8..643cc30a2 100644 --- a/res/ari/resource_bridges.c +++ b/res/ari/resource_bridges.c @@ -957,13 +957,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( |