[asterisk-commits] qwell: branch 10 r364842 - in /branches/10: ./ main/manager.c

[SVN commits to the Asterisk project]

Author: qwell
Date: Tue May  1 16:44:13 2012
New Revision: 364842

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=364842
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

Modified:
    branches/10/   (props changed)
    branches/10/main/manager.c

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

Modified: branches/10/main/manager.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/main/manager.c?view=diff&rev=364842&r1=364841&r2=364842
==============================================================================
--- branches/10/main/manager.c (original)
+++ branches/10/main/manager.c Tue May  1 16:44:13 2012
@@ -4125,7 +4125,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 */