[asterisk-commits] russell: trunk r83126 - in /trunk: ./ main/manager.c

[SVN commits to the Asterisk project]

Author: russell
Date: Wed Sep 19 10:11:12 2007
New Revision: 83126

URL: http://svn.digium.com/view/asterisk?view=rev&rev=83126
Log:
Merged revisions 83121 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r83121 | russell | 2007-09-19 10:10:14 -0500 (Wed, 19 Sep 2007) | 4 lines

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:
    trunk/   (props changed)
    trunk/main/manager.c

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

Modified: trunk/main/manager.c
URL: http://svn.digium.com/view/asterisk/trunk/main/manager.c?view=diff&rev=83126&r1=83125&r2=83126
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Wed Sep 19 10:11:12 2007
@@ -761,9 +761,8 @@
 	AST_LIST_LOCK(&sessions);
 	AST_LIST_REMOVE(&sessions, s, list);
 	ast_atomic_fetchadd_int(&num_sessions, -1);
+	free_session(s);
 	AST_LIST_UNLOCK(&sessions);
-
-	free_session(s);
 }
 
 const char *astman_get_header(const struct message *m, char *var)