Bug #61253 [Asn->Csd]: Wrappers opened with errors concurrency problem
Edit report at https://bugs.php.net/bug.php?id=61253&edit=1
ID: 61253
Updated by: mattficken@php.net
Reported by: cataphract@php.net
Summary: Wrappers opened with errors concurrency problem
-Status: Assigned
+Status: Closed
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:
Closing bug.
Previous Comments:
------------------------------------------------------------------------
[2012-03-09 18:29:35] mattficken@php.net
As of 5_4 rev324027, I can no longer reproduce this bug using Apache 2.2 on Windows 2008r2.
I think the fix works.
------------------------------------------------------------------------
[2012-03-08 12:39:39] cataphract@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] cataphract@php.net
I've committed the patch. Testing would still be useful.
------------------------------------------------------------------------
[2012-03-08 12:30:50] cataphract@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] cataphract@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
------------------------------------------------------------------------
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
Thread (8 messages)