Merge pull request voryx#70 from valepu/patch-2

davidwdan · web-flow · commit 4ed1b90c9ee5 · 2018-01-02T14:53:17.000-05:00
Add escapeshellarg to PHP_BINARY
diff --git a/src/Voryx/ThruwayBundle/Command/ThruwayProcessCommand.php b/src/Voryx/ThruwayBundle/Command/ThruwayProcessCommand.php
@@ -263,7 +263,7 @@ protected function add()
      */
     protected function addSymfonyCmdWorkers($env)
     {
-        $phpBinary = PHP_BINARY;
+        $phpBinary = escapeshellarg(PHP_BINARY);
         if (!$this->input->getOption('no-exec')) {
             $phpBinary = 'exec ' . $phpBinary;
         }
@@ -320,7 +320,7 @@ protected function addShellCmdWorkers()
      */
     protected function addWorkers($env)
     {
-        $phpBinary = PHP_BINARY;
+        $phpBinary = escapeshellarg(PHP_BINARY);
         if (!$this->input->getOption('no-exec')) {
             $phpBinary = 'exec ' . $phpBinary;
         }

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@ protected function add()`
`263`	`263`	`*/`
`264`	`264`	`protected function addSymfonyCmdWorkers($env)`
`265`	`265`	`{`
`266`		`- $phpBinary = PHP_BINARY;`
	`266`	`+ $phpBinary = escapeshellarg(PHP_BINARY);`
`267`	`267`	`if (!$this->input->getOption('no-exec')) {`
`268`	`268`	`$phpBinary = 'exec ' . $phpBinary;`
`269`	`269`	`}`
`@@ -320,7 +320,7 @@ protected function addShellCmdWorkers()`
`320`	`320`	`*/`
`321`	`321`	`protected function addWorkers($env)`
`322`	`322`	`{`
`323`		`- $phpBinary = PHP_BINARY;`
	`323`	`+ $phpBinary = escapeshellarg(PHP_BINARY);`
`324`	`324`	`if (!$this->input->getOption('no-exec')) {`
`325`	`325`	`$phpBinary = 'exec ' . $phpBinary;`
`326`	`326`	`}`