Understand Python Threading Bug
Solution 1:
The bug occurs because of a bad interaction between dummy thread objects created by the threading API when one calls threading.currentThread() on a foreign thread, and the threading._after_fork function, called to clean up resources after a call to os.fork().
To work around the bug without modifying Python's source, monkey-patch threading._DummyThread with a no-op implementation of __stop:
import threading
threading._DummyThread._Thread__stop = lambda x: 42The cause of the bug is best narrowed down in comments by Richard Oudkerk and cooyeah. What happens is the following:
The
threadingmodule allowsthreading.currentThread()to be called from a thread not created by thethreadingAPI calls. It then returns a "dummy thread" instance which supports a very limited subset of theThreadAPI, but is still useful for identifying the current thread.threading._DummyThreadis implemented as a subclass ofThread.Threadinstances normally contain an internal callable (self.__block) that keeps reference to an OS-level lock allocated for the instance. Since publicThreadmethods that might end up usingself.__blockare all overridden by_DummyThread,_DummyThread's constructor intentionally releases the OS-level lock by deletingself.__block.threading._after_forkbreaks the encapsulation and calls the privateThread.__stopmethod on all registered threads, including the dummy ones, where__stopwas never meant to be invoked. (They weren't started by Python, so their stopping is not managed by Python either.) As the dummy threads don't know about__stop, they inherit it fromThread, and that implementation happily accesses the private__blockattribute that doesn't exist in_DummyThreadinstances. This access finally causes the error.
The bug is fixed in the 2.7 branch by modifying Thread.__stop not to break when __block is deleted. The 3.x branch, where __stop is spelled as _stop and therefore protected, fixes it by overriding _DummyThread's _stop to do nothing.
Post a Comment for "Understand Python Threading Bug"