Suppress warnings when open_basedir is non-empty

cbj4074 · web-flow · commit a6d452f15ccd · 2018-05-03T12:50:56.000-04:00
If PHP is configured *with a non-empty open_basedir* value that does not permit access to the target location, these calls to is_executable() throw warnings. While Symfony may not raise exceptions for warnings in production environments, other frameworks (such as Laravel) do, in which case any of these checks causes a show-stopping 500 error. We fixed a similar issue in the ExecutableFinder class via symfony/symfony#16182 . This has always been an issue, but 709e15e made it more likely that a warning is triggered.
diff --git a/PhpExecutableFinder.php b/PhpExecutableFinder.php
@@ -49,20 +49,20 @@ public function find($includeArgs = true)
         }
 
         if ($php = getenv('PHP_PATH')) {
-            if (!is_executable($php)) {
+            if (@!is_executable($php)) {
                 return false;
             }
 
             return $php;
         }
 
         if ($php = getenv('PHP_PEAR_PHP_BIN')) {
-            if (is_executable($php)) {
+            if (@is_executable($php)) {
                 return $php;
             }
         }
 
-        if (is_executable($php = PHP_BINDIR.('\\' === DIRECTORY_SEPARATOR ? '\\php.exe' : '/php'))) {
+        if (@is_executable($php = PHP_BINDIR.('\\' === DIRECTORY_SEPARATOR ? '\\php.exe' : '/php'))) {
             return $php;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -49,20 +49,20 @@ public function find($includeArgs = true)`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`if ($php = getenv('PHP_PATH')) {`
`52`		`- if (!is_executable($php)) {`
	`52`	`+ if (@!is_executable($php)) {`
`53`	`53`	`return false;`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`return $php;`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`if ($php = getenv('PHP_PEAR_PHP_BIN')) {`
`60`		`- if (is_executable($php)) {`
	`60`	`+ if (@is_executable($php)) {`
`61`	`61`	`return $php;`
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`
`65`		`- if (is_executable($php = PHP_BINDIR.('\\' === DIRECTORY_SEPARATOR ? '\\php.exe' : '/php'))) {`
	`65`	`+ if (@is_executable($php = PHP_BINDIR.('\\' === DIRECTORY_SEPARATOR ? '\\php.exe' : '/php'))) {`
`66`	`66`	`return $php;`
`67`	`67`	`}`
`68`	`68`