-
-
Notifications
You must be signed in to change notification settings - Fork 33.1k
[3.14] gh-140067: Fix memory leak in sub-interpreter creation (GH-140111) #140118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3.14] gh-140067: Fix memory leak in sub-interpreter creation (GH-140111) #140118
Conversation
…ythonGH-140111) Fix memory leak in sub-interpreter creation caused by overwriting of the previously used `_malloced` field. Now the pointer is stored in the first word of the memory block to avoid it being overwritten accidentally. (cherry picked from commit 59547a2) Co-authored-by: Shamil <[email protected]> Co-authored-by: Kumar Aditya <[email protected]>
There's a somewhat surprising failing test on gil-enabled tsan CI, will investigate later marking draft for now |
I think that's a known failure. |
Oh, nevermind, #140111 was what caused that known failure. |
This reverts commit c0dee26.
61a20ac
to
d7e44f7
Compare
🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit 150ae29 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F140118%2Fmerge If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again. |
Fix memory leak in sub-interpreter creation caused by overwriting of the previously used
_malloced
field. Now the pointer is stored in the first word of the memory block to avoid it being overwritten accidentally.(cherry picked from commit 59547a2)