[asterisk-commits] tilghman: trunk r94809 - in /trunk: ./ main/manager.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Wed Dec 26 14:49:35 2007
New Revision: 94809

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

........
r94808 | tilghman | 2007-12-26 14:43:38 -0600 (Wed, 26 Dec 2007) | 6 lines

Workaround for what is probably a glibc bug (but we'll see this crop up again
and again, if we don't add the workaround).
Reported by: rolek
Patch by: tilghman
(Closes issue #11601, closes issue #11426)

........

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=94809&r1=94808&r2=94809
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Wed Dec 26 14:49:35 2007
@@ -2589,6 +2589,14 @@
 			ast_verb(2, "Connect attempt from '%s' unable to authenticate\n", ast_inet_ntoa(s->sin.sin_addr));
 		ast_log(LOG_EVENT, "Failed attempt from %s\n", ast_inet_ntoa(s->sin.sin_addr));
 	}
+
+	/* If the entire session occurs in a single context switch, then it's
+	 * possible to get an unsafe memory condition by free()ing the memory
+	 * before letting other threads run at least once.  This actually seems
+	 * like a workaround for a glibc bug.
+	 */
+	usleep(1);
+
 	destroy_session(s);
 
 done: