From 1bfd37dc8c8418781c47f37a1e03058ba2c94686 Mon Sep 17 00:00:00 2001 From: sobolevn Date: Wed, 25 Jun 2025 17:17:02 +0300 Subject: [PATCH] [3.13] gh-135839: Fix `module_traverse` and `module_clear` in subinterp modules (GH-135937) (cherry picked from commit bcc2cbaa7f112323939e853ed69fd82f19568ccf) Co-authored-by: sobolevn --- Modules/_interpqueuesmodule.c | 6 ++---- Modules/_interpretersmodule.c | 6 ++---- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/Modules/_interpqueuesmodule.c b/Modules/_interpqueuesmodule.c index 6f050a988b1426..5818066eebe49a 100644 --- a/Modules/_interpqueuesmodule.c +++ b/Modules/_interpqueuesmodule.c @@ -1937,8 +1937,7 @@ static int module_traverse(PyObject *mod, visitproc visit, void *arg) { module_state *state = get_module_state(mod); - traverse_module_state(state, visit, arg); - return 0; + return traverse_module_state(state, visit, arg); } static int @@ -1947,8 +1946,7 @@ module_clear(PyObject *mod) module_state *state = get_module_state(mod); // Now we clear the module state. - clear_module_state(state); - return 0; + return clear_module_state(state); } static void diff --git a/Modules/_interpretersmodule.c b/Modules/_interpretersmodule.c index a8fbf987ab18b3..4ff73d32d2095b 100644 --- a/Modules/_interpretersmodule.c +++ b/Modules/_interpretersmodule.c @@ -1540,8 +1540,7 @@ module_traverse(PyObject *mod, visitproc visit, void *arg) { module_state *state = get_module_state(mod); assert(state != NULL); - traverse_module_state(state, visit, arg); - return 0; + return traverse_module_state(state, visit, arg); } static int @@ -1549,8 +1548,7 @@ module_clear(PyObject *mod) { module_state *state = get_module_state(mod); assert(state != NULL); - clear_module_state(state); - return 0; + return clear_module_state(state); } static void