diff options
author | dkf <donal.k.fellows@manchester.ac.uk> | 2012-10-04 08:25:52 (GMT) |
---|---|---|
committer | dkf <donal.k.fellows@manchester.ac.uk> | 2012-10-04 08:25:52 (GMT) |
commit | d9ec510026ac166ce0732f7a77573654e7ab2410 (patch) | |
tree | 7ebd338853b7207ceb8b25bda783dcceadf6fe12 | |
parent | 26529fbec2cc37660e2f376993a1098b4d95404a (diff) | |
parent | bfd2f2fbf7a0c7fb68ded3c44b169cb5ce97c24f (diff) | |
download | tcl-d9ec510026ac166ce0732f7a77573654e7ab2410.zip tcl-d9ec510026ac166ce0732f7a77573654e7ab2410.tar.gz tcl-d9ec510026ac166ce0732f7a77573654e7ab2410.tar.bz2 |
merge trunk
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | generic/tclIO.c | 12 |
2 files changed, 15 insertions, 3 deletions
@@ -1,3 +1,9 @@ +2012-10-03 Don Porter <dgp@users.sourceforge.net> + + * generic/tclIO.c: When checking for std channels being closed, + compare the channel state, not the channel itself so that stacked + channels do not cause trouble. + 2012-09-26 Reinhard Max <max@suse.de> * generic/tclIOSock.c (TclCreateSocketAddress): Work around a bug diff --git a/generic/tclIO.c b/generic/tclIO.c index 4e24533..0cb9fa9 100644 --- a/generic/tclIO.c +++ b/generic/tclIO.c @@ -879,19 +879,25 @@ CheckForStdChannelsBeingClosed( ChannelState *statePtr = ((Channel *) chan)->state; ThreadSpecificData *tsdPtr = TCL_TSD_INIT(&dataKey); - if ((chan == tsdPtr->stdinChannel) && tsdPtr->stdinInitialized) { + if (tsdPtr->stdinInitialized + && tsdPtr->stdinChannel != NULL + && statePtr == ((Channel *)tsdPtr->stdinChannel)->state) { if (statePtr->refCount < 2) { statePtr->refCount = 0; tsdPtr->stdinChannel = NULL; return; } - } else if ((chan == tsdPtr->stdoutChannel) && tsdPtr->stdoutInitialized) { + } else if (tsdPtr->stdoutInitialized + && tsdPtr->stdoutChannel != NULL + && statePtr == ((Channel *)tsdPtr->stdoutChannel)->state) { if (statePtr->refCount < 2) { statePtr->refCount = 0; tsdPtr->stdoutChannel = NULL; return; } - } else if ((chan == tsdPtr->stderrChannel) && tsdPtr->stderrInitialized) { + } else if (tsdPtr->stderrInitialized + && tsdPtr->stderrChannel != NULL + && statePtr == ((Channel *)tsdPtr->stderrChannel)->state) { if (statePtr->refCount < 2) { statePtr->refCount = 0; tsdPtr->stderrChannel = NULL; |