diff options
author | David M. Lee <dlee@digium.com> | 2013-08-21 16:23:59 +0000 |
---|---|---|
committer | David M. Lee <dlee@digium.com> | 2013-08-21 16:23:59 +0000 |
commit | 5762c1b4ac5c1519463b31ab7078fcd9cb55a393 (patch) | |
tree | e4b1dd24f8908070538a5eeec1bbc8379a0ed934 /res/res_ari_channels.c | |
parent | a6da087716478b4a54defb17d56a9574548a8d2a (diff) |
ARI: Correct segfault with /variable calls are missing ?variable parameter.
Both /asterisk/variable and /channel/{channelId}/variable requires a
?variable parameter to be passed into the query. But we weren't checking
for the parameter being missing, which caused a segfault.
All calls now properly return 400 Bad Request errors when the parameter
is missing. The Swagger api-docs were updated accordingly.
(closes issue ASTERISK-22273)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@397306 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_ari_channels.c')
-rw-r--r-- | res/res_ari_channels.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/res/res_ari_channels.c b/res/res_ari_channels.c index 8a6687bc2..063e766f1 100644 --- a/res/res_ari_channels.c +++ b/res/res_ari_channels.c @@ -1055,6 +1055,7 @@ static void ast_ari_get_channel_var_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 400: /* Missing variable parameter. */ case 404: /* Channel not found */ case 409: /* Channel not in a Stasis application */ is_valid = 1; @@ -1122,6 +1123,7 @@ static void ast_ari_set_channel_var_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 400: /* Missing variable parameter. */ case 404: /* Channel not found */ case 409: /* Channel not in a Stasis application */ is_valid = 1; |