[3.6] bpo-9194: Fix the bounds checking in winreg.c's fixupMultiSZ() (GH-12687) by miss-islington · Pull Request #12910 · 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

@miss-islington

Copy link Copy Markdown

Contributor

@miss-islington miss-islington commented

Apr 22, 2019

edited by bedevere-bot

Loading

(cherry picked from commit 56ed864)

Co-authored-by: Zackery Spytz zspytz@gmail.com

https://bugs.python.org/issue9194

…nGH-12687)

(cherry picked from commit 56ed864)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>

@miss-islington

Copy link Copy Markdown

Contributor Author

@ZackerySpytz and @zooba: Status check is done, and it's a success ✅ .

@miss-islington

Copy link Copy Markdown

Contributor Author

Sorry, I can't merge this PR. Reason: You're not authorized to push to this branch. Visit https://help.github.com/articles/about-protected-branches/ for more information..

@miss-islington

Copy link Copy Markdown

Contributor Author

@ZackerySpytz and @zooba: Status check is done, and it's a success ✅ .

@miss-islington

Copy link Copy Markdown

Contributor Author

Sorry, I can't merge this PR. Reason: You're not authorized to push to this branch. Visit https://help.github.com/articles/about-protected-branches/ for more information..

@zooba

Copy link Copy Markdown

Member

zooba commented

Apr 22, 2019

edited

Loading

@ned-deily Would you like a fix for a potential (read) buffer overrun? Feel free to just close if you don't.

@ned-deily ned-deily merged commit dadc347 into python:3.6

May 2, 2019

@miss-islington miss-islington deleted the backport-56ed864-3.6 branch

May 2, 2019 16:00

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

Reviewers

@zooba zooba zooba approved these changes

@ned-deily ned-deily ned-deily approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@miss-islington @zooba @ned-deily @the-knights-who-say-ni @bedevere-bot @ZackerySpytz