Bug #74878 [Com]: Data race in ZTS builds

Edit report at https://bugs.php.net/bug.php?id=74878&edit=1 ID: 74878 Comment by: maroszek at gmx dot net Reported by: maroszek at gmx dot net Summary: Data race in ZTS builds Status: Re-Opened Type: Bug Package: *General Issues Operating System: Linux/MacOS PHP Version: 7.2.0alpha3 Block user comment: N Private report: N New Comment: Hi, i'd love to bump this one. I think this is the only remaining blocker issue for ZTS usage on PHP 7.2. Using the incomplete patch from nikic fixes all crash related issues for my use case. (I know the memory leak problems) It would be awesome if we could get this properly fixed and tested until the 7.2 release. I would love to help - but i have no idea where to start. Would it help if i could reproduce the issue with the pthreads extension? (https://github.com/krakjoe/pthreads)? It should be affected aswell. Thanks a lot in advance! Previous Comments: ------------------------------------------------------------------------ [2017-07-18 09:29:27] maroszek at gmx dot net Hi! I just wanted to kindly ask if this is still on your radar as the issue is marked as closed? Thanks! ------------------------------------------------------------------------ [2017-07-10 14:50:41] maroszek at gmx dot net Thanks! Didn't knew where to look :) If i can somehow assist in fully solving the problem, i'll gladly do that! ------------------------------------------------------------------------ [2017-07-10 14:22:42] spam2 at rhsoft dot net Related To: Bug #74878 ------------------------------------------------------------------------ [2017-07-10 14:22:36] spam2 at rhsoft dot net because it created more problems than it solves -------- Weitergeleitete Nachricht -------- Betreff: Re: [PHP-CVS] com php-src: Fixed bug #74878: NEWS Zend/zend_API.c Datum: Mon, 10 Jul 2017 08:08:22 +0000 Von: Dmitry Stogov <dmitry@zend.com> An: Nikita Popov <nikic@php.net>, php-cvs@lists.php.net <php-cvs@lists.php.net> Kopie (CC): Sara Golemon <pollita@php.net> This commit introduced a lot of memory leaks. Probably, you meant string interning only for internal classes. Even for internal classes only, this may require additional changes in opcache, that copies interned strings into shared memory. It's better to revert this, before complete fix ------------------------------------------------------------------------ [2017-07-10 14:19:44] maroszek at gmx dot net Hi, i have seen the commit got reverted. Any hints why? Can we reopen the issue then? The previous commit successfully fixed the crashing issues i was experiencing. Thanks! ------------------------------------------------------------------------ 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=74878 -- Edit this bug report at https://bugs.php.net/bug.php?id=74878&edit=1