[proxy] web.archive.org — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public
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

[3.6] bpo-32374: m_traverse may be called with m_state=NULL (GH-5140) #6129

Merged
merged 1 commit into from
Mar 17, 2018
Merged

[3.6] bpo-32374: m_traverse may be called with m_state=NULL (GH-5140) #6129

merged 1 commit into from
Mar 17, 2018

Conversation

Copy link
Contributor

miss-islington commented Mar 17, 2018 •
edited by bedevere-bot
Loading

Multi-phase initialized modules allow m_traverse to be called while the
module is still being initialized, so module authors may need to account
for that.
(cherry picked from commit c2b0b12)

Co-authored-by: Marcel Plch gmarcel.plch@gmail.com

https://bugs.python.org/issue32374

bpo-32374: m_traverse may be called with m_state=NULL (pythonGH-5140)
ae3980f 
Multi-phase initialized modules allow m_traverse to be called while the
module is still being initialized, so module authors may need to account
for that.
(cherry picked from commit c2b0b12)

Co-authored-by: Marcel Plch <gmarcel.plch@gmail.com>
bedevere-bot mentioned this pull request Mar 17, 2018
Merged
Copy link
Contributor Author

@Traceur759 and @ncoghlan: Backport status check is done, and it's a success ✅ .

miss-islington merged commit 1da0479 into python:3.6 Mar 17, 2018
Copy link
Contributor Author

Thanks!

miss-islington deleted the backport-c2b0b12-3.6 branch March 17, 2018 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

ncoghlan ncoghlan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants