Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Jan 8, 2021

Fix a reference leak in the getcodec() function of CJK codecs.

https://bugs.python.org/issue42866

Fix a reference leak in the getcodec() function of CJK codecs.
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @vstinner, I had trouble checking out the 3.8 backport branch.
Please backport using cherry_picker on command line.
cherry_picker e542d417b96077d04aec089505eacb990c9799ae 3.8

@bedevere-bot
Copy link

GH-24167 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 8, 2021
Fix a reference leak in the getcodec() function of CJK codecs.
(cherry picked from commit e542d41)

Co-authored-by: Victor Stinner <[email protected]>
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 8, 2021
Fix a reference leak in the getcodec() function of CJK codecs.
(cherry picked from commit e542d41)

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

GH-24168 is a backport of this pull request to the 3.8 branch.

adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
Fix a reference leak in the getcodec() function of CJK codecs.
@tongxiaoge1001
Copy link

tongxiaoge1001 commented Jun 16, 2021

Is python3.7 involved?

@vstinner
Copy link
Member Author

Sadly, right now, Python 3.7 and 3.8 no longer accept bugfixes: you must upgrade Python, or backport the fix yourself.
https://devguide.python.org/#status-of-python-branches

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants