Show error in child process if exec fails (in proc_open)

alexdowad · alexdowad · commit a30aa5a9e92d · 2020-05-03T20:07:34.000+02:00
diff --git a/ext/standard/proc_open.c b/ext/standard/proc_open.c
@@ -1095,7 +1095,9 @@ PHP_FUNCTION(proc_open)
 			}
 		}
 
-		/* Show errors from exec!! */
+		/* If execvp/execle/execl are successful, we will never reach here
+		 * Display error and exit with non-zero (error) status code */
+		php_error_docref(NULL, E_WARNING, "Exec failed - %s", strerror(errno));
 		_exit(127);
 	} else if (child < 0) {
 		/* failed to fork() */

Original file line number	Diff line number	Diff line change
`@@ -1095,7 +1095,9 @@ PHP_FUNCTION(proc_open)`
`1095`	`1095`	`}`
`1096`	`1096`	`}`
`1097`	`1097`
`1098`		`- /* Show errors from exec!! */`
	`1098`	`+ /* If execvp/execle/execl are successful, we will never reach here`
	`1099`	`+ * Display error and exit with non-zero (error) status code */`
	`1100`	`+ php_error_docref(NULL, E_WARNING, "Exec failed - %s", strerror(errno));`
`1099`	`1101`	`_exit(127);`
`1100`	`1102`	`} else if (child < 0) {`
`1101`	`1103`	`/* failed to fork() */`