From e539042615457f25e32cc481db82a48146b76495 Mon Sep 17 00:00:00 2001 From: vasiljevic Date: Fri, 1 Jul 2005 10:57:56 +0000 Subject: Protect against spurious wake-ups while waiting on the condition variable when tearing down the notifier thread [Bug# 1222872]. --- ChangeLog | 6 ++++++ unix/tclUnixNotfy.c | 14 ++++++++------ 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 9a160a7..6cc9f05 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2005-07-01 Zoran Vasiljevic + + * unix/tclUnixNotfy.c: protect against spurious wake-ups while + waiting on the condition variable when tearing down the notifier + thread [Bug# 1222872]. + 2005-06-27 Don Porter *** 8.4.11 TAGGED FOR RELEASE *** diff --git a/unix/tclUnixNotfy.c b/unix/tclUnixNotfy.c index 1f1615d..24d552f 100644 --- a/unix/tclUnixNotfy.c +++ b/unix/tclUnixNotfy.c @@ -11,7 +11,7 @@ * See the file "license.terms" for information on usage and redistribution * of this file, and for a DISCLAIMER OF ALL WARRANTIES. * - * RCS: @(#) $Id: tclUnixNotfy.c,v 1.11.2.12 2005/06/07 10:26:58 dkf Exp $ + * RCS: @(#) $Id: tclUnixNotfy.c,v 1.11.2.13 2005/07/01 10:57:59 vasiljevic Exp $ */ #ifndef HAVE_COREFOUNDATION /* Darwin/Mac OS X CoreFoundation notifier @@ -275,7 +275,7 @@ Tcl_FinalizeNotifier(clientData) */ if (notifierCount == 0) { - int result, dummy; + int result; if (triggerPipe < 0) { panic("Tcl_FinalizeNotifier: notifier pipe not initialized"); } @@ -288,13 +288,15 @@ Tcl_FinalizeNotifier(clientData) * not just close the pipe and check for EOF in the notifier * thread because if a background child process was created with * exec, select() would not register the EOF on the pipe until the - * child processes had terminated. [Bug: 4139] + * child processes had terminated. [Bug: 4139] [Bug: 1222872] */ + write(triggerPipe, "q", 1); close(triggerPipe); - - Tcl_ConditionWait(¬ifierCV, ¬ifierMutex, NULL); - result = Tcl_JoinThread(notifierThread, &dummy); + while(triggerPipe >= 0) { + Tcl_ConditionWait(¬ifierCV, ¬ifierMutex, NULL); + } + result = Tcl_JoinThread(notifierThread, NULL); if (result) { Tcl_Panic("Tcl_FinalizeNotifier: unable to join notifier thread"); } -- cgit v0.12