Skip to content

bpo-42576: Raise TypeError when passing in keyword arguments to GenericAlias#23656

Merged
miss-islington merged 2 commits intopython:masterfrom
Fidget-Spinner:ga-crash
Dec 5, 2020
Merged

bpo-42576: Raise TypeError when passing in keyword arguments to GenericAlias#23656
miss-islington merged 2 commits intopython:masterfrom
Fidget-Spinner:ga-crash

Conversation

@Fidget-Spinner
Copy link
Copy Markdown
Member

@Fidget-Spinner Fidget-Spinner commented Dec 5, 2020

Use _PyArg_NoKeywords instead of _PyArg_NoKwnames when checking the kwds tuple when creating GenericAlias. This fixes an interpreter crash when passing in keyword arguments to GenericAlias's constructor.

Needs backport to 3.9.

https://bugs.python.org/issue42576

Automerge-Triggered-By: GH:gvanrossum

@miss-islington miss-islington merged commit 804d689 into python:master Dec 5, 2020
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @Fidget-Spinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@bedevere-bot
Copy link
Copy Markdown

GH-23659 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2020
…icAlias (pythonGH-23656)

Use `_PyArg_NoKeywords` instead of `_PyArg_NoKwnames` when checking the `kwds` tuple when creating `GenericAlias`. This fixes an interpreter crash when passing in keyword arguments to `GenericAlias`'s constructor.

Needs backport to 3.9.

Automerge-Triggered-By: GH:gvanrossum
(cherry picked from commit 804d689)

Co-authored-by: kj <28750310+Fidget-Spinner@users.noreply.github.com>
@Fidget-Spinner Fidget-Spinner deleted the ga-crash branch December 5, 2020 16:09
miss-islington added a commit that referenced this pull request Dec 5, 2020
…icAlias (GH-23656)

Use `_PyArg_NoKeywords` instead of `_PyArg_NoKwnames` when checking the `kwds` tuple when creating `GenericAlias`. This fixes an interpreter crash when passing in keyword arguments to `GenericAlias`'s constructor.

Needs backport to 3.9.

Automerge-Triggered-By: GH:gvanrossum
(cherry picked from commit 804d689)

Co-authored-by: kj <28750310+Fidget-Spinner@users.noreply.github.com>
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 13, 2021
…icAlias (pythonGH-23656)

Use `_PyArg_NoKeywords` instead of `_PyArg_NoKwnames` when checking the `kwds` tuple when creating `GenericAlias`. This fixes an interpreter crash when passing in keyword arguments to `GenericAlias`'s constructor.

Needs backport to 3.9.

Automerge-Triggered-By: GH:gvanrossum
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.

5 participants