From 28e476f6a2e4c7621f6724c2c3f5764a30623311 Mon Sep 17 00:00:00 2001 From: Kumar Aditya Date: Sun, 30 Mar 2025 16:20:35 +0530 Subject: gh-127945: make initialization of `error_object_name` thread safe in ctypes (#131896) --- Modules/_ctypes/_ctypes.c | 9 +++++++-- Modules/_ctypes/callproc.c | 7 +------ 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/Modules/_ctypes/_ctypes.c b/Modules/_ctypes/_ctypes.c index 803d416..7536d3f 100644 --- a/Modules/_ctypes/_ctypes.c +++ b/Modules/_ctypes/_ctypes.c @@ -6093,14 +6093,19 @@ _ctypes_mod_exec(PyObject *mod) } #ifdef WORDS_BIGENDIAN - st->swapped_suffix = PyUnicode_InternFromString("_le"); + st->swapped_suffix = PyUnicode_InternFromString("_le"); #else - st->swapped_suffix = PyUnicode_InternFromString("_be"); + st->swapped_suffix = PyUnicode_InternFromString("_be"); #endif if (st->swapped_suffix == NULL) { return -1; } + st->error_object_name = PyUnicode_InternFromString("ctypes.error_object"); + if (st->error_object_name == NULL) { + return -1; + } + if (_ctypes_add_types(mod) < 0) { return -1; } diff --git a/Modules/_ctypes/callproc.c b/Modules/_ctypes/callproc.c index 80b66af..158422f 100644 --- a/Modules/_ctypes/callproc.c +++ b/Modules/_ctypes/callproc.c @@ -164,12 +164,7 @@ _ctypes_get_errobj(ctypes_state *st, int **pspace) "cannot get thread state"); return NULL; } - if (st->error_object_name == NULL) { - st->error_object_name = PyUnicode_InternFromString("ctypes.error_object"); - if (st->error_object_name == NULL) { - return NULL; - } - } + assert(st->error_object_name != NULL); if (PyDict_GetItemRef(dict, st->error_object_name, &errobj) < 0) { return NULL; } -- cgit v0.12