Skip to content

bpo-42814: Fix undefined behavior in Objects/genericaliasobject.c #24073

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 Jan 3, 2021

In is_typing_name(), va_end() is not always called before the
function returns. It is undefined behavior to call va_start()
without also calling va_end().

https://bugs.python.org/issue42814

In is_typing_name(), va_end() is not always called before the
function returns.  It is undefined behavior to call va_start()
without also calling va_end().
@miss-islington
Copy link
Contributor

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

@miss-islington
Copy link
Contributor

Sorry, @ZackerySpytz and @vstinner, I could not cleanly backport this to 3.9 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 5d3553b0a8959e7505bbec4de03077dbf135ee4b 3.9

@vstinner vstinner removed the needs backport to 3.9 only security fixes label Jan 3, 2021
@vstinner
Copy link
Member

vstinner commented Jan 3, 2021

Ah, 3.9 doesn't use va_start() in Objects/genericaliasobject.c.

@Fidget-Spinner
Copy link
Member

Sorry, I didn't notice that in my original PR, thanks for catching this and fixing it Zackery!

adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
…thonGH-24073)

In is_typing_name(), va_end() is not always called before the
function returns.  It is undefined behavior to call va_start()
without also calling va_end().
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants