[asterisk-commits] kmoore: branch 11 r374792 - /branches/11/main/manager.c

[SVN commits to the Asterisk project]

Author: kmoore
Date: Wed Oct 10 08:35:53 2012
New Revision: 374792

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=374792
Log:
Fix segfault regression from r370681

Due to usage of ast_hook_send_action, AMI action handling code should
be able to handle a NULL mansession->session.  This would cause a crash
on NULL dereference if action_originate was called from
ast_hook_send_action.

(closes issue ASTERISK-20544)

Modified:
    branches/11/main/manager.c

Modified: branches/11/main/manager.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/main/manager.c?view=diff&rev=374792&r1=374791&r2=374792
==============================================================================
--- branches/11/main/manager.c (original)
+++ branches/11/main/manager.c Wed Oct 10 08:35:53 2012
@@ -4369,7 +4369,7 @@
 
 	/* Allocate requested channel variables */
 	vars = astman_get_variables(m);
-	if (s->session->chanvars) {
+	if (s->session && s->session->chanvars) {
 		struct ast_variable *v, *old;
 		old = vars;
 		vars = NULL;