Fix os.posix_spawn() error handling (#142532)

johnslavik · web-flow · commit 8cfa35156048 · 2025-12-11T18:02:26.000+01:00
Consistently use `goto exit;` in `py_posix_spawn()`.
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
@@ -7903,7 +7903,7 @@ py_posix_spawn(int use_posix_spawnp, PyObject *module, path_t *path, PyObject *a
     if (argc < 1) {
         PyErr_Format(PyExc_ValueError,
                      "%s: argv must not be empty", func_name);
-        return NULL;
+        goto exit;
     }
 
     if (!PyMapping_Check(env) && env != Py_None) {

Original file line number	Diff line number	Diff line change
`@@ -7903,7 +7903,7 @@ py_posix_spawn(int use_posix_spawnp, PyObject module, path_t path, PyObject *a`
`7903`	`7903`	`if (argc < 1) {`
`7904`	`7904`	`PyErr_Format(PyExc_ValueError,`
`7905`	`7905`	`"%s: argv must not be empty", func_name);`
`7906`		`- return NULL;`
	`7906`	`+ goto exit;`
`7907`	`7907`	`}`
`7908`	`7908`
`7909`	`7909`	`if (!PyMapping_Check(env) && env != Py_None) {`