Edit report at https://bugs.php.net/bug.php?id=61253&edit=1
ID: 61253 Comment by: mattfic...@php.net Reported by: cataphr...@php.net Summary: Wrappers opened with errors concurrency problem Status: Assigned Type: Bug Package: Streams related Operating System: Any (ZTS) PHP Version: 5.4SVN-2012-03-02 (SVN) Assigned To: mattficken Block user comment: N Private report: N New Comment: As of 5_4 rev324027, I can no longer reproduce this bug using Apache 2.2 on Windows 2008r2. I think the fix works. Previous Comments: ------------------------------------------------------------------------ [2012-03-08 12:39:39] cataphr...@php.net Automatic comment from SVN on behalf of cataphract Revision: http://svn.php.net/viewvc/?view=revision&revision=324021 Log: - Oops committed old version of the patch in r324020. Fixed. See bug #61253. ------------------------------------------------------------------------ [2012-03-08 12:32:14] cataphr...@php.net I've committed the patch. Testing would still be useful. ------------------------------------------------------------------------ [2012-03-08 12:30:50] cataphr...@php.net Automatic comment from SVN on behalf of cataphract Revision: http://svn.php.net/viewvc/?view=revision&revision=324020 Log: - Fixed bug #61253: Wrappers opened with errors concurrency problem #NOTE: There is a very small possibility that this will further break #extensions that access wrapper->{err_stack, err_count}. On PECL SVN, rar is the #only one and it may leak memory after this. I say "further break" because #extensions that do that are already broken (will segfault) under ZTS, which is #why this patch is necessary. #There was what I deem as tacit acceptance from 5.3/5.4 RMs on this. ------------------------------------------------------------------------ [2012-03-07 14:05:35] cataphr...@php.net The following patch has been added/updated: Patch Name: bug61253.diff Revision: 1331129135 URL: https://bugs.php.net/patch-display.php?bug=61253&patch=bug61253.diff&revision=1331129135 ------------------------------------------------------------------------ [2012-03-03 20:42:12] paj...@php.net Matt, can you test this patch please? and add the test to our suites while being at it :) ------------------------------------------------------------------------ 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=61253 -- Edit this bug report at https://bugs.php.net/bug.php?id=61253&edit=1
