◐ Shell
clean mode source ↗

bpo-43693: Un-revert commit f3fa63e. by ericsnowcurrently · Pull Request #26609 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@ericsnowcurrently

@ericsnowcurrently ericsnowcurrently commented

Jun 8, 2021

edited

Loading

Copy link Copy Markdown

Member

This was reverted in GH-26596 (commit 6d518bb) due to some bad memory accesses.

The memory accesses have been fixed and I verified the address sanitizer is happy now (after duplicating the failure locally).

https://bugs.python.org/issue43693

@bedevere-bot

Copy link Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ericsnowcurrently for commit 5107da7 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots

Test PR w/ buildbots; report in status section

label

Jun 8, 2021

@pablogsal

Copy link Copy Markdown

Member

Thanks for the quick fix, @ericsnowcurrently

@ericsnowcurrently ericsnowcurrently removed the request for review from markshannon

June 8, 2021 20:13

@ericsnowcurrently ericsnowcurrently merged commit 3e1c716 into python:main

Jun 8, 2021

@bedevere-bot

Copy link Copy Markdown

@ericsnowcurrently: Please replace # with GH- in the commit message next time. Thanks!

@ericsnowcurrently ericsnowcurrently deleted the unrevert-add-make-cell branch

June 8, 2021 22:01

ericsnowcurrently added a commit that referenced this pull request

Jun 9, 2021
This is the same fix as for PyFrame_LocalsToFast() in gh-26609, but applied to PyFrame_FastToLocalsWithError(). (It should have been in that PR.)

https://bugs.python.org/issue43693

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

Reviewers

@pablogsal pablogsal pablogsal 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

@ericsnowcurrently @bedevere-bot @pablogsal @the-knights-who-say-ni