bpo-30602: Fix lastarg in os.spawnve() (#2287) (#2357)

vstinner · web-flow · commit c472fb6b2744 · 2017-06-23T15:21:24.000+02:00
Fix a regression introduced by myself in the commit 526b226. (cherry picked from commit c8d6ab2)
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
@@ -5189,7 +5189,7 @@ os_spawnve_impl(PyObject *module, int mode, path_t *path, PyObject *argv,
             goto fail_1;
         }
         if (i == 0 && !argvlist[0][0]) {
-            lastarg = i;
+            lastarg = i + 1;
             PyErr_SetString(
                 PyExc_ValueError,
                 "spawnv() arg 2 first element cannot be empty");

Original file line number	Diff line number	Diff line change
`@@ -5189,7 +5189,7 @@ os_spawnve_impl(PyObject module, int mode, path_t path, PyObject *argv,`
`5189`	`5189`	`goto fail_1;`
`5190`	`5190`	`}`
`5191`	`5191`	`if (i == 0 && !argvlist[0][0]) {`
`5192`		`- lastarg = i;`
	`5192`	`+ lastarg = i + 1;`
`5193`	`5193`	`PyErr_SetString(`
`5194`	`5194`	`PyExc_ValueError,`
`5195`	`5195`	`"spawnv() arg 2 first element cannot be empty");`