[Process] Fix for #8754 (Timed-out processes are successful)

romainneutron · fabpot · commit fa01e6b4d39a · 2013-08-15T00:22:46.000+02:00
diff --git a/src/Symfony/Component/Process/Process.php b/src/Symfony/Component/Process/Process.php
@@ -646,9 +646,9 @@ public function stop($timeout = 10, $signal = null)
         $timeoutMicro = microtime(true) + $timeout;
         if ($this->isRunning()) {
             proc_terminate($this->process);
-            while ($this->isRunning() && microtime(true) < $timeoutMicro) {
+            do {
                 usleep(1000);
-            }
+            } while ($this->isRunning() && microtime(true) < $timeoutMicro);
 
             if ($this->isRunning() && !$this->isSigchildEnabled()) {
                 if (null !== $signal || defined('SIGKILL')) {
diff --git a/src/Symfony/Component/Process/Tests/AbstractProcessTest.php b/src/Symfony/Component/Process/Tests/AbstractProcessTest.php
@@ -444,6 +444,7 @@ public function testCheckTimeoutOnStartedProcess()
         $duration = microtime(true) - $start;
 
         $this->assertLessThan($timeout + $precision, $duration);
+        $this->assertFalse($process->isSuccessful());
     }
 
     public function testGetPid()
diff --git a/src/Symfony/Component/Process/Tests/SigchildDisabledProcessTest.php b/src/Symfony/Component/Process/Tests/SigchildDisabledProcessTest.php
@@ -61,6 +61,14 @@ public function testProcessWithoutTermSignal()
         parent::testProcessWithoutTermSignal();
     }
 
+    /**
+     * @expectedException \Symfony\Component\Process\Exception\RuntimeException
+     */
+    public function testCheckTimeoutOnStartedProcess()
+    {
+        parent::testCheckTimeoutOnStartedProcess();
+    }
+
     /**
      * @expectedException \Symfony\Component\Process\Exception\RuntimeException
      */

Original file line number	Diff line number	Diff line change
`@@ -444,6 +444,7 @@ public function testCheckTimeoutOnStartedProcess()`
`444`	`444`	`$duration = microtime(true) - $start;`
`445`	`445`
`446`	`446`	`$this->assertLessThan($timeout + $precision, $duration);`
	`447`	`+ $this->assertFalse($process->isSuccessful());`
`447`	`448`	`}`
`448`	`449`
`449`	`450`	`public function testGetPid()`