Message182897
Thanks, it's surprising this was never noticed before.
Your patch has two issues:
- it doesn't clear the thread state before deleting it, which would leak the frame, thread-specific dict, etc
- it only clears the thread states after the current thread state: if the current thread is not at the head of the linked list - if it's not the most recently created thread - some thread states won't get cleared
I'm attaching a patch doing the cleanup in PyEval_ReInitThreads(), with test. |
|
Date |
User |
Action |
Args |
2013-02-24 21:46:09 | neologix | set | recipients:
+ neologix, pitrou, Ringding |
2013-02-24 21:46:09 | neologix | set | messageid: <1361742369.91.0.828071909678.issue17094@psf.upfronthosting.co.za> |
2013-02-24 21:46:09 | neologix | link | issue17094 messages |
2013-02-24 21:46:09 | neologix | create | |
|