Uh oh!
There was an error while loading. Please reload this page.
gh-151763: Fix dealloc clears the in-flight exception in subtype_dealloc - #153548
Open
Abhi210 wants to merge 1 commit into
Open
gh-151763: Fix dealloc clears the in-flight exception in subtype_dealloc#153548Abhi210 wants to merge 1 commit into
Abhi210 wants to merge 1 commit into
Conversation
Abhi210
marked this pull request as ready for review
July 11, 2026 09:57
Abhi210
requested review from
AA-Turner, corona10, emmatyping, erlend-aasland and itamaro
as code ownersJuly 11, 2026 12:18
Abhi210force-pushed
the
gh-151763-0023
branch
2 times, most recently
from
July 12, 2026 14:53
70c3e1a to
8242da9Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Fixes: #gh-151763OOM-0023
subtype_dealloc()can clear the in-flight exception while tearing down heap-type instances. Unlike slot_tp_finalize(), this deallocation path does not preserve the current exception across operations that may perform arbitrary Py_DECREFs, potentially causingtstate->current_exceptionto be lost.This patch saves the current exception before the destructive teardown (clear_slots(), managed dict clearing, base deallocation, and type decref) and restores it afterward, mirroring the existing exception-preservation pattern used in slot_tp_finalize().