Skip to content

bpo-36356: Fix leak when updating search path in config #12444

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
Closed
wants to merge 1 commit into from
Closed

bpo-36356: Fix leak when updating search path in config #12444

wants to merge 1 commit into from

Conversation

@btharper
Copy link
Contributor

@btharper btharper commented Mar 19, 2019 •
edited by bedevere-bot
Loading

@btharper
Copy link
Contributor Author

btharper commented Mar 19, 2019 •
edited by bedevere-bot
Loading

Can someone add a the skip news label for this? This is a further part of bpo-36356 and previously mentioned in Misc/NEWS.d/next/Build/2019-03-18-23-49-15.bpo-36356.WNrwYI.rst
@vstinner
Copy link
Member

vstinner commented Mar 20, 2019

Your PR fix maybe the memory leak, but your PR makes me realize that it's a bug. I wrote PR #12454 to fix the bug (and a few other similar bugs). My PR also fix the memory leak.
@vstinner vstinner closed this Mar 20, 2019
@btharper btharper deleted the env_free branch March 20, 2019 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review

4 participants

@btharper @vstinner @the-knights-who-say-ni @bedevere-bot