Implemented fix for incorrect filedescriptor closing

2 files changed, 52 insertions, 7 deletions

diff --git a/ext/standard/proc_open.c b/ext/standard/proc_open.c
index ff38e75d3f..d9508dfecc 100644
--- a/ext/standard/proc_open.c
+++ b/ext/standard/proc_open.c
@@ -854,6 +854,12 @@ PHP_FUNCTION(proc_open)
 		}
 #endif
 
+#if PHP_CAN_DO_PTS
+		if (dev_ptmx >= 0) {
+			close(dev_ptmx);
+			close(slave_pty);
+		}
+#endif
 		/* close those descriptors that we just opened for the parent stuff,
 		 * dup new descriptors into required descriptors and close the original
 		 * cruft */
@@ -869,13 +875,6 @@ PHP_FUNCTION(proc_open)
 				close(descriptors[i].childend);
 		}
 
-#if PHP_CAN_DO_PTS
-		if (dev_ptmx >= 0) {
-			close(dev_ptmx);
-			close(slave_pty);
-		}
-#endif
-
 		if (cwd) {
 			php_ignore_value(chdir(cwd));
 		}
diff --git a/ext/standard/tests/file/bug69442.phpt b/ext/standard/tests/file/bug69442.phpt
new file mode 100644
index 0000000000..e5255acb37
--- /dev/null
+++ b/ext/standard/tests/file/bug69442.phpt
@@ -0,0 +1,46 @@
+--TEST--
+proc_open with PTY closes incorrect file descriptor
+--SKIPIF--
+<?php
+
+$code = <<< 'EOC'
+    <?php
+    $descriptors = array(array("pty"), array("pty"), array("pty"), array("pipe", "w"));
+    $pipes = array();
+    $process = proc_open('echo "foo";', $descriptors, $pipes);
+EOC;
+
+    $tmpFile = tempnam(sys_get_temp_dir(), "bug69442");
+    file_put_contents($tmpFile, $code);
+
+    exec($_SERVER['TEST_PHP_EXECUTABLE']." ".$tmpFile." 2>&1", $output);
+    $output = join("\n", $output);
+    unlink($tmpFile);
+
+    if (strstr($output, "pty pseudo terminal not supported on this system") !== false) {
+        die("skip PTY pseudo terminals are not supported");
+    }
+--FILE--
+<?php
+$cmd = '(echo "foo" ; exit 42;) 3>/dev/null; code=$?; echo $code >&3; exit $code';
+$descriptors = array(array("pty"), array("pty"), array("pty"), array("pipe", "w"));
+$pipes = array();
+
+$process = proc_open($cmd, $descriptors, $pipes);
+
+foreach ($pipes as $type => $pipe) {
+    $data = fread($pipe, 999);
+    echo 'type ' . $type . ' ';
+    var_dump($data);
+    fclose($pipe);
+}
+proc_close($process);
+--EXPECT--
+type 0 string(5) "foo
+"
+type 1 string(0) ""
+type 2 string(0) ""
+type 3 string(3) "42
+"
+
+