Merged revisions 94808 via svnmerge from
authorTilghman Lesher <tilghman@meg.abyt.es>
Wed, 26 Dec 2007 20:49:35 +0000 (20:49 +0000)
committerTilghman Lesher <tilghman@meg.abyt.es>
Wed, 26 Dec 2007 20:49:35 +0000 (20:49 +0000)
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)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@94809 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/manager.c

index e229a78..56584fa 100644 (file)
@@ -2589,6 +2589,14 @@ static void *session_do(void *data)
                        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: