[3.13] Fix os.posix_spawn() error handling (GH-142532) (#142581)

miss-islington · johnslavik · web-flow · commit 53ca3b7b5ce6 · 2025-12-17T15:43:51.000Z
Fix os.posix_spawn() error handling (GH-142532) Consistently use `goto exit;` in `py_posix_spawn()`. (cherry picked from commit 8cfa351) Co-authored-by: Bartosz Sławecki <bartosz@ilikepython.com>
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
@@ -7352,7 +7352,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
`@@ -7352,7 +7352,7 @@ py_posix_spawn(int use_posix_spawnp, PyObject module, path_t path, PyObject *a`
`7352`	`7352`	`if (argc < 1) {`
`7353`	`7353`	`PyErr_Format(PyExc_ValueError,`
`7354`	`7354`	`"%s: argv must not be empty", func_name);`
`7355`		`- return NULL;`
	`7355`	`+ goto exit;`
`7356`	`7356`	`}`
`7357`	`7357`
`7358`	`7358`	`if (!PyMapping_Check(env) && env != Py_None) {`