-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
Closed
Labels
3.14bugs and security fixesbugs and security fixes3.15new features, bugs and security fixesnew features, bugs and security fixesextension-modulesC modules in the Modules dirC modules in the Modules dirtype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error
Description
Bug report
Refs #133265
Right here PyLong_FromLong
can in theory (unlikely in practice) return NULL
. It is missing this check. I understand that this is a test code, but I also don't see a reason why we should not be explicit about it.
cpython/Modules/_testinternalcapi.c
Line 1045 in 13cb8ca
#define SET_COUNT(DICT, STRUCT, NAME) \ |
I have a PR ready.
Linked PRs
Metadata
Metadata
Assignees
Labels
3.14bugs and security fixesbugs and security fixes3.15new features, bugs and security fixesnew features, bugs and security fixesextension-modulesC modules in the Modules dirC modules in the Modules dirtype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error