diff options
| author | dgp <dgp@users.sourceforge.net> | 2016-03-08 15:55:51 (GMT) |
|---|---|---|
| committer | dgp <dgp@users.sourceforge.net> | 2016-03-08 15:55:51 (GMT) |
| commit | 5137357e74c887368ad68c3cb09998b154e68d4f (patch) | |
| tree | 1b49fd42c73cc5d60c6b041d15b35f9c5e04fd7e /generic | |
| parent | 3cd08b4cddcfa292ba21ebbf490fb4b5189e41bb (diff) | |
| parent | c31b2f4c54d596aafee955890527c1b2406093b1 (diff) | |
| download | tcl-5137357e74c887368ad68c3cb09998b154e68d4f.zip tcl-5137357e74c887368ad68c3cb09998b154e68d4f.tar.gz tcl-5137357e74c887368ad68c3cb09998b154e68d4f.tar.bz2 | |
[bbc304f61a] Avoid event handling when reflected channel has a watch change half-completed. (First half in 1 thread, second in another). When this is allowed to happen, false alarm errors from [chan postevent] are the result when timing is unlucky. See test iocmd.tf-24.17.bug3522560 .
Diffstat (limited to 'generic')
| -rw-r--r-- | generic/tclIORChan.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/generic/tclIORChan.c b/generic/tclIORChan.c index 21c766e..f476a1a 100644 --- a/generic/tclIORChan.c +++ b/generic/tclIORChan.c @@ -1609,8 +1609,6 @@ ReflectWatch( return; } - rcPtr->interest = mask; - /* * Are we in the correct thread? */ @@ -1633,6 +1631,7 @@ ReflectWatch( Tcl_Preserve(rcPtr); + rcPtr->interest = mask; maskObj = DecodeEventMask(mask); /* assert maskObj.refCount == 1 */ (void) InvokeTclMethod(rcPtr, METH_WATCH, maskObj, NULL, NULL); @@ -3083,6 +3082,7 @@ ForwardProc( /* assert maskObj.refCount == 1 */ Tcl_Preserve(rcPtr); + rcPtr->interest = paramPtr->watch.mask; (void) InvokeTclMethod(rcPtr, METH_WATCH, maskObj, NULL, NULL); Tcl_DecrRefCount(maskObj); Tcl_Release(rcPtr); |