[Process] Properly close pipes after a Process::stop call

romainneutron · romainneutron · commit d84df4c2c5c7 · 2013-09-16T13:47:05.000+02:00
diff --git a/src/Symfony/Component/Process/Process.php b/src/Symfony/Component/Process/Process.php
@@ -626,9 +626,13 @@ public function stop($timeout = 10, $signal = null)
                     $this->signal($signal ?: SIGKILL);
                 }
             }
+        }
 
-            $this->updateStatus(false);
+        $this->updateStatus(false);
+        if ($this->processInformation['running']) {
+            $this->close();
         }
+
         $this->status = self::STATUS_TERMINATED;
 
         return $this->exitcode;
diff --git a/src/Symfony/Component/Process/Tests/AbstractProcessTest.php b/src/Symfony/Component/Process/Tests/AbstractProcessTest.php
@@ -477,6 +477,21 @@ public function testCheckTimeoutOnStartedProcess()
         $this->assertFalse($process->isSuccessful());
     }
 
+    public function testStartAfterATimeout()
+    {
+        $process = $this->getProcess('php -r "while(true) {echo \'\'; usleep(1000); }"');
+        $process->setTimeout(0.1);
+        try {
+            $process->run();
+            $this->fail('An exception should have been raised.');
+        } catch (\Exception $e) {
+
+        }
+        $process->start();
+        usleep(10000);
+        $process->stop();
+    }
+
     public function testGetPid()
     {
         $process = $this->getProcess('php -r "sleep(1);"');

Original file line number	Diff line number	Diff line change
`@@ -626,9 +626,13 @@ public function stop($timeout = 10, $signal = null)`
`626`	`626`	`$this->signal($signal ?: SIGKILL);`
`627`	`627`	`}`
`628`	`628`	`}`
	`629`	`+ }`
`629`	`630`
`630`		`- $this->updateStatus(false);`
	`631`	`+ $this->updateStatus(false);`
	`632`	`+ if ($this->processInformation['running']) {`
	`633`	`+ $this->close();`
`631`	`634`	`}`
	`635`	`+`
`632`	`636`	`$this->status = self::STATUS_TERMINATED;`
`633`	`637`
`634`	`638`	`return $this->exitcode;`