Skip to content

[3.9] bpo-42576: Raise TypeError when passing in keyword arguments to GenericAlias (GH-23656)#23659

Merged
miss-islington merged 1 commit into
python:3.9from
miss-islington:backport-804d689-3.9
Dec 5, 2020
Merged

[3.9] bpo-42576: Raise TypeError when passing in keyword arguments to GenericAlias (GH-23656)#23659
miss-islington merged 1 commit into
python:3.9from
miss-islington:backport-804d689-3.9

Conversation

@miss-islington

@miss-islington miss-islington commented Dec 5, 2020

Copy link
Copy Markdown
Contributor

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

https://bugs.python.org/issue42576

Automerge-Triggered-By: GH:gvanrossum

…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>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@Fidget-Spinner: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@Fidget-Spinner: Status check is done, and it's a success ✅ .

@miss-islington

Copy link
Copy Markdown
Contributor Author

@Fidget-Spinner: Status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit d5c029b into python:3.9 Dec 5, 2020
@miss-islington
miss-islington deleted the backport-804d689-3.9 branch December 5, 2020 16:24
@miss-islington

Copy link
Copy Markdown
Contributor Author

@Fidget-Spinner: Status check is done, and it's a success ✅ .

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