Bug #66437 [Fbk->NoF]: proc_close misbehaves if proc_get_status was used

Edit report at https://bugs.php.net/bug.php?id=66437&edit=1 ID: 66437 Updated by: php-bugs@lists.php.net Reported by: phpbugs2012 at joern dot heissler dot de Summary: proc_close misbehaves if proc_get_status was used -Status: Feedback +Status: No Feedback Type: Bug Package: Program Execution Operating System: Linux PHP Version: 5.5.7 Private report: N New Comment: No feedback was provided. The bug is being suspended because we assume that you are no longer experiencing the problem. If this is not the case and you are able to provide the information that was requested earlier, please do so and change the status of the bug back to "Re-Opened". Thank you. Previous Comments: ------------------------------------------------------------------------ [2014-01-08 07:17:18] krakjoe@php.net ok, fixed the patch so it will compile ... I forgot about popen ... there's not really a tidy way to give that the same logic as proc_open though, not really happy with that ... Why shouldn't waitpid be in strace ? waitpid is still called on process that have gone away unless the last call to get_status sets proc->closed ... ------------------------------------------------------------------------ [2014-01-08 07:15:03] krakjoe@php.net The following patch has been added/updated: Patch Name: proc-open-stuff.patch Revision: 1389165303 URL: https://bugs.php.net/patch-display.php?bug=66437&patch=proc-open-stuff.patch&revision=1389165303 ------------------------------------------------------------------------ [2014-01-08 00:22:55] phpbugs2012 at joern dot heissler dot de Thanks for the really quick patch! ext/standard/file.c can't be compiled, popen seems to use the same global vars. Same for main/streams/streams.c I commented out the noncompiling code so that I can test the proc_… functions. I still see wait4 (i.e. waitpid) in the strace for proc_get_status after the child terminated. ------------------------------------------------------------------------ [2014-01-07 20:19:28] krakjoe@php.net Please test attached patch. ------------------------------------------------------------------------ [2014-01-07 20:18:49] krakjoe@php.net The following patch has been added/updated: Patch Name: proc-open-stuff.patch Revision: 1389125929 URL: https://bugs.php.net/patch-display.php?bug=66437&patch=proc-open-stuff.patch&revision=1389125929 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=66437 -- Edit this bug report at https://bugs.php.net/bug.php?id=66437&edit=1