Skip to content

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

Merged
serhiy-storchaka merged 1 commit intopython:masterfrom
ZackerySpytz:bpo-35470-ref-counting-bug-import
Jan 10, 2019
Merged

bpo-35470: Fix a reference counting bug in _PyImport_FindExtensionObjectEx()#11128
serhiy-storchaka merged 1 commit intopython:masterfrom
ZackerySpytz:bpo-35470-ref-counting-bug-import

Conversation

@ZackerySpytz
Copy link
Copy Markdown
Contributor

@ZackerySpytz ZackerySpytz commented Dec 12, 2018

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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 <zspytz@gmail.com>
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 <zspytz@gmail.com>
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