Return false in isTtySupported() when open_basedir restrictions prevent access to /dev/tty.

mjauvin · web-flow · commit 1393de6f0688 · 2024-05-07T13:49:07.000-04:00
If open_basedir restrictions are in effect, checking if the file /dev/tty is writable will prevent setting tty mode on the process, and avoid failing to create a Process.
diff --git a/Process.php b/Process.php
@@ -1211,7 +1211,7 @@ public static function isTtySupported(): bool
     {
         static $isTtySupported;
 
-        return $isTtySupported ??= ('/' === \DIRECTORY_SEPARATOR && stream_isatty(\STDOUT));
+        return $isTtySupported ??= ('/' === \DIRECTORY_SEPARATOR && stream_isatty(\STDOUT) && @is_writable('/dev/tty'));
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -1211,7 +1211,7 @@ public static function isTtySupported(): bool`
`1211`	`1211`	`{`
`1212`	`1212`	`static $isTtySupported;`
`1213`	`1213`
`1214`		`- return $isTtySupported ??= ('/' === \DIRECTORY_SEPARATOR && stream_isatty(\STDOUT));`
	`1214`	`+ return $isTtySupported ??= ('/' === \DIRECTORY_SEPARATOR && stream_isatty(\STDOUT) && @is_writable('/dev/tty'));`
`1215`	`1215`	`}`
`1216`	`1216`
`1217`	`1217`	`/**`