[Process] Fix potential race condition leading to transient tests

nicolas-grekas · nicolas-grekas · commit 99fc428bdcbb · 2015-12-30T09:46:06.000+01:00
diff --git a/src/Symfony/Component/Process/Process.php b/src/Symfony/Component/Process/Process.php
@@ -1072,13 +1072,10 @@ private function readPipes($blocking, $close)
 
         $callback = $this->callback;
         foreach ($result as $type => $data) {
-            if (3 === $type) {
-                $this->fallbackStatus['running'] = false;
-                if (!isset($this->fallbackStatus['signaled'])) {
-                    $this->fallbackStatus['exitcode'] = (int) $data;
-                }
-            } else {
+            if (3 !== $type) {
                 $callback($type === self::STDOUT ? self::OUT : self::ERR, $data);
+            } elseif (!isset($this->fallbackStatus['signaled'])) {
+                $this->fallbackStatus['exitcode'] = (int) $data;
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1072,13 +1072,10 @@ private function readPipes($blocking, $close)`
`1072`	`1072`
`1073`	`1073`	`$callback = $t 8A99 his->callback;`
`1074`	`1074`	`foreach ($result as $type => $data) {`
`1075`		`- if (3 === $type) {`
`1076`		`- $this->fallbackStatus['running'] = false;`
`1077`		`- if (!isset($this->fallbackStatus['signaled'])) {`
`1078`		`- $this->fallbackStatus['exitcode'] = (int) $data;`
`1079`		`- }`
`1080`		`- } else {`
	`1075`	`+ if (3 !== $type) {`
`1081`	`1076`	`$callback($type === self::STDOUT ? self::OUT : self::ERR, $data);`
	`1077`	`+ } elseif (!isset($this->fallbackStatus['signaled'])) {`
	`1078`	`+ $this->fallbackStatus['exitcode'] = (int) $data;`
`1082`	`1079`	`}`
`1083`	`1080`	`}`
`1084`	`1081`	`}`