[3.13] gh-135839: Fix module_traverse and module_clear in _interpchannelsmodule (GH-135840) (#135919)

sobolevn · web-flow · commit 28c5adfca997 · 2025-06-25T12:25:27.000+03:00
(cherry picked from commit dd59c78)
diff --git a/Modules/_interpchannelsmodule.c b/Modules/_interpchannelsmodule.c
@@ -3556,8 +3556,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
@@ -3567,8 +3566,7 @@ module_clear(PyObject *mod)
     assert(state != NULL);
 
     // Now we clear the module state.
-    clear_module_state(state);
-    return 0;
+    return clear_module_state(state);
 }
 
 static void

Original file line number	Diff line number	Diff line change
`@@ -3556,8 +3556,7 @@ module_traverse(PyObject mod, visitproc visit, void arg)`
`3556`	`3556`	`{`
`3557`	`3557`	`module_state *state = get_module_state(mod);`
`3558`	`3558`	`assert(state != NULL);`
`3559`		`- traverse_module_state(state, visit, arg);`
`3560`		`- return 0;`
	`3559`	`+ return traverse_module_state(state, visit, arg);`
`3561`	`3560`	`}`
`3562`	`3561`
`3563`	`3562`	`static int`
`@@ -3567,8 +3566,7 @@ module_clear(PyObject *mod)`
`3567`	`3566`	`assert(state != NULL);`
`3568`	`3567`
`3569`	`3568`	`// Now we clear the module state.`
`3570`		`- clear_module_state(state);`
`3571`		`- return 0;`
	`3569`	`+ return clear_module_state(state);`
`3572`	`3570`	`}`
`3573`	`3571`
`3574`	`3572`	`static void`