[asterisk-commits] tilghman: branch 1.6.1 r211234 - in /branches/1.6.1: ./ apps/app_stack.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Sun Aug  9 02:12:51 2009
New Revision: 211234

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=211234
Log:
Merged revisions 211232 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r211232 | tilghman | 2009-08-09 02:11:22 -0500 (Sun, 09 Aug 2009) | 4 lines
  
  Check for NULL frame, before dereferencing pointer.
  (closes issue #15617)
   Reported by: rain
........

Modified:
    branches/1.6.1/   (props changed)
    branches/1.6.1/apps/app_stack.c

Propchange: branches/1.6.1/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.1/apps/app_stack.c
URL: http://svn.asterisk.org/svn-view/asterisk/branches/1.6.1/apps/app_stack.c?view=diff&rev=211234&r1=211233&r2=211234
==============================================================================
--- branches/1.6.1/apps/app_stack.c (original)
+++ branches/1.6.1/apps/app_stack.c Sun Aug  9 02:12:51 2009
@@ -356,7 +356,12 @@
 
 	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;