[asterisk-commits] russell: branch 1.4 r83121 - /branches/1.4/main/manager.c

[SVN commits to the Asterisk project]

Author: russell
Date: Wed Sep 19 10:10:14 2007
New Revision: 83121

URL: http://svn.digium.com/view/asterisk?view=rev&rev=83121
Log:
Fix up another potential race condition.  Do the loop decrementing use count
on events with the eventq protected from being changed.
(reported on IRC by Ivan)

Modified:
    branches/1.4/main/manager.c

Modified: branches/1.4/main/manager.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/manager.c?view=diff&rev=83121&r1=83120&r2=83121
==============================================================================
--- branches/1.4/main/manager.c (original)
+++ branches/1.4/main/manager.c Wed Sep 19 10:10:14 2007
@@ -684,9 +684,8 @@
 	AST_LIST_LOCK(&sessions);
 	AST_LIST_REMOVE(&sessions, s, list);
 	num_sessions--;
+	free_session(s);
 	AST_LIST_UNLOCK(&sessions);
-
-	free_session(s);
 }
 
 const char *astman_get_header(const struct message *m, char *var)