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_asterisk.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_asterisk.c')
| -rw-r--r-- | res/res_ari_asterisk.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/res/res_ari_asterisk.c b/res/res_ari_asterisk.c index dce634e8a..3f34c7ab6 100644 --- a/res/res_ari_asterisk.c +++ b/res/res_ari_asterisk.c @@ -175,6 +175,7 @@ static void ast_ari_get_global_var_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 400: /* Missing variable parameter. */ is_valid = 1; break; default: @@ -234,6 +235,7 @@ static void ast_ari_set_global_var_cb( break; case 500: /* Internal Server Error */ case 501: /* Not Implemented */ + case 400: /* Missing variable parameter. */ is_valid = 1; break; default: |