Fix deadlock on shutdown if test_current_{exception,frames} fails #102019
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These tests spawn a thread that waits on a
threading.Event
. If the test fails any of its assertions, theEvent
won't be signaled and the thread will wait indefinitely, causing a deadlock whenthreading._shutdown()
tries to join all outstanding threads.The solution is just to put the code in a
try-finally
block, with the existing clean-up in thefinally
.@swtaarrs found and fixed this originally while working on Cinder.
Hopefully this is trivial enough to not need an issue or a NEWS blurb.