Skip to content

gh-138756: Fix memory leak in PyInitConfig_Free() #138759

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Sep 11, 2025
Merged

gh-138756: Fix memory leak in PyInitConfig_Free() #138759

merged 3 commits into from
Sep 11, 2025

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Sep 10, 2025 •
edited by bedevere-app bot
Loading

@vstinner vstinner added skip news needs backport to 3.14 bugs and security fixes labels Sep 10, 2025
@bedevere-app bedevere-app bot mentioned this pull request Sep 10, 2025
Closed
@vstinner vstinner mentioned this pull request Sep 10, 2025
Closed
encukou
encukou reviewed Sep 11, 2025
View reviewed changes
@vstinner vstinner enabled auto-merge (squash) September 11, 2025 10:21
@vstinner vstinner merged commit 96dee64 into python:main Sep 11, 2025
43 checks passed
@vstinner vstinner deleted the free_config branch September 11, 2025 10:46
@miss-islington-app
Copy link

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 11, 2025
@vstinner @miss-islington
pythongh-138756: Fix memory leak in PyInitConfig_Free() (pythonGH-138759
d30142e 
)

Clear also memory of PyConfig members.
(cherry picked from commit 96dee64)

Co-authored-by: Victor Stinner <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Sep 11, 2025

GH-138785 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Sep 11, 2025
@vstinner
Copy link
Member Author

Oh, there is another leak: inittab. I wrote #138792 to release inittab as well.

vstinner added a commit that referenced this pull request Oct 7, 2025
@miss-islington @vstinner
[3.14] gh-138756: Fix memory leak in PyInitConfig_Free() (GH-138759) (#…
0accda7 
…138785)

gh-138756: Fix memory leak in PyInitConfig_Free() (GH-138759)

Clear also memory of PyConfig members.
(cherry picked from commit 96dee64)

Co-authored-by: Victor Stinner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@encukou encukou encukou left review comments

@ericsnowcurrently ericsnowcurrently Awaiting requested review from ericsnowcurrently ericsnowcurrently is a code owner

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner @encukou