[asterisk-commits] rmudgett: trunk r412228 - in /trunk: ./ apps/app_stack.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Apr 11 16:43:34 CDT 2014


Author: rmudgett
Date: Fri Apr 11 16:43:30 2014
New Revision: 412228

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=412228
Log:
app_stack: Add missing unlock in off-nominal path of STACK_PEEK function.

ASTERISK-23620 #close
Reported by: Bradley Watkins
Patches:
      ASTERISK-23620_unlock_oldlist.patch (license #5021) patch uploaded by Bradley Watkins
........

Merged revisions 412225 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 412226 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 412227 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/apps/app_stack.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: trunk/apps/app_stack.c
URL: http://svnview.digium.com/svn/asterisk/trunk/apps/app_stack.c?view=diff&rev=412228&r1=412227&r2=412228
==============================================================================
--- trunk/apps/app_stack.c (original)
+++ trunk/apps/app_stack.c Fri Apr 11 16:43:30 2014
@@ -873,6 +873,7 @@
 		if (!ast_true(args.suppress)) {
 			ast_log(LOG_ERROR, "Stack peek of '%s' is more stack frames than I have\n", args.n);
 		}
+		AST_LIST_UNLOCK(oldlist);
 		ast_channel_unlock(chan);
 		return -1;
 	}




More information about the asterisk-commits mailing list