[PROCESS] make sure /dev/tty is readable

staabm · fabpot · commit 935afe650ac5 · 2015-02-24T12:07:24.000+01:00
When using Process from Web-SAPI it is likely that the webserver user
doesn't has rights to use /dev/tty
diff --git a/src/Symfony/Component/Process/Process.php b/src/Symfony/Component/Process/Process.php
@@ -792,6 +792,9 @@ public function setTty($tty)
         if ('\\' === DIRECTORY_SEPARATOR && $tty) {
             throw new RuntimeException('TTY mode is not supported on Windows platform.');
         }
+        if ($tty && (!file_exists('/dev/tty') || !is_readable('/dev/tty'))) {
+            throw new RuntimeException('TTY mode requires /dev/tty to be readable.');
+        }
 
         $this->tty = (bool) $tty;
 

Original file line number	Diff line number	Diff line change
`@@ -792,6 +792,9 @@ public function setTty($tty)`
`792`	`792`	`if ('\\' === DIRECTORY_SEPARATOR && $tty) {`
`793`	`793`	`throw new RuntimeException('TTY mode is not supported on Windows platform.');`
`794`	`794`	`}`
	`795`	`+ if ($tty && (!file_exists('/dev/tty') \|\| !is_readable('/dev/tty'))) {`
	`796`	`+ throw new RuntimeException('TTY mode requires /dev/tty to be readable.');`
	`797`	`+ }`
`795`	`798`
`796`	`799`	`$this->tty = (bool) $tty;`
`797`	`800`