[asterisk-commits] qwell: trunk r364844 - in /trunk: ./ main/manager.c

[SVN commits to the Asterisk project]

Author: qwell
Date: Tue May  1 16:49:25 2012
New Revision: 364844

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=364844
Log:
Prevent a potential crash when using manager hooks.

Found by me while poking at DPMA-127.
........

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

Merged revisions 364842 from http://svn.asterisk.org/svn/asterisk/branches/10

Modified:
    trunk/   (props changed)
    trunk/main/manager.c

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

Modified: trunk/main/manager.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/manager.c?view=diff&rev=364844&r1=364843&r2=364844
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Tue May  1 16:49:25 2012
@@ -4226,7 +4226,7 @@
 		ast_parse_allow_disallow(NULL, cap, codecs, 1);
 	}
 
-	if (!ast_strlen_zero(app)) {
+	if (!ast_strlen_zero(app) && s->session) {
 		int bad_appdata = 0;
 		/* To run the System application (or anything else that goes to
 		 * shell), you must have the additional System privilege */