diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-09 07:11:22 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-09 07:11:22 +0000 |
commit | 1a4692a9fadb781fa3a31875ade990e391a48ff0 (patch) | |
tree | 4f1c18318ed226d0b21ae8a653c7b7f03c170cfc /apps/app_stack.c | |
parent | 673ecef518e2dda20531e3618b48aa9abb49f3ec (diff) |
Check for NULL frame, before dereferencing pointer.
(closes issue #15617)
Reported by: rain
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@211232 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps/app_stack.c')
-rw-r--r-- | apps/app_stack.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/apps/app_stack.c b/apps/app_stack.c index 1a2523122..d46f0481c 100644 --- a/apps/app_stack.c +++ b/apps/app_stack.c @@ -474,7 +474,12 @@ static int local_read(struct ast_channel *chan, const char *cmd, char *data, cha oldlist = stack_store->data; AST_LIST_LOCK(oldlist); - frame = AST_LIST_FIRST(oldlist); + if (!(frame = AST_LIST_FIRST(oldlist))) { + /* Not within a Gosub routine */ + AST_LIST_UNLOCK(oldlist); + return -1; + } + AST_LIST_TRAVERSE(&frame->varshead, variables, entries) { if (!strcmp(data, ast_var_name(variables))) { const char *tmp; |