kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-12 15:45:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_ari: Fix inverted test giving wrong error message.

Browse Source 
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
This commit is contained in:
Richard Mudgett
2017-11-26 09:44:51 -06:00
parent ee3562619c
commit 81e2d8aa9a
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
res/ari/resource_bridges.c
View File

@@ -957,13 +957,12 @@ void ast_ari_bridges_create_with_id(struct ast_variable *headers,
if (bridge) { if (bridge) {
/* update */ /* update */
if (!ast_strlen_zero(args->name)) { if (!ast_strlen_zero(args->name)
if (!strcmp(args->name, bridge->name)) { && strcmp(args->name, bridge->name)) {
ast_ari_response_error( ast_ari_response_error(
response, 500, "Internal Error", response, 500, "Internal Error",
"Changing bridge name is not implemented"); "Changing bridge name is not implemented");
return; return;
}
} }
if (!ast_strlen_zero(args->type)) { if (!ast_strlen_zero(args->type)) {
ast_ari_response_error( ast_ari_response_error(