Thanks to visit codestin.com
Credit goes to github.com

Skip to content

bpo-35470: Fix a reference counting bug in _PyImport_FindExtensionObjectEx() #11128

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

Merged

Conversation

ZackerySpytz
Copy link
Contributor

@ZackerySpytz ZackerySpytz commented Dec 12, 2018

Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

That's what we get for using borrowed references. :/

@ericsnowcurrently
Copy link
Member

@ZackerySpytz, are you already in Misc/ACKS? If not, be sure to add yourself in this PR.

Also, this change (small as it is) should have a NEWS entry.

@vstinner
Copy link
Member

I removed the " needs backport to 3.6" label, the 3.6 branch no longer accept bugfixes (only security fixes are accepted): https://devguide.python.org/#status-of-python-branches

@serhiy-storchaka serhiy-storchaka merged commit 89c4f90 into python:master Jan 10, 2019
@miss-islington
Copy link
Contributor

Thanks @ZackerySpytz for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-11506 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 10, 2019
…ectEx(). (pythonGH-11128)

(cherry picked from commit 89c4f90)

Co-authored-by: Zackery Spytz <[email protected]>
miss-islington added a commit that referenced this pull request Jan 10, 2019
…ectEx(). (GH-11128)

(cherry picked from commit 89c4f90)

Co-authored-by: Zackery Spytz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants