Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:93417 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 85404 invoked from network); 20 May 2016 13:41:36 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 May 2016 13:41:36 -0000 Authentication-Results: pb1.pair.com smtp.mail=nikita.ppv@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=nikita.ppv@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.161.173 as permitted sender) X-PHP-List-Original-Sender: nikita.ppv@gmail.com X-Host-Fingerprint: 209.85.161.173 mail-yw0-f173.google.com Received: from [209.85.161.173] ([209.85.161.173:34913] helo=mail-yw0-f173.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 35/A1-08692-F041F375 for ; Fri, 20 May 2016 09:41:36 -0400 Received: by mail-yw0-f173.google.com with SMTP id g133so109243055ywb.2 for ; Fri, 20 May 2016 06:41:35 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=pxkwxGFUt+8j75PjoCpB3xaEMlpBLZL3A8MBJU6El10=; b=Yd15u1AVeJSfYruyTYxJnlpm8Cddo4iUcyHJ0AC+QxWO21MOyFfZARR9aDn992wbUg Eb1ZaqMPwA/XXfICVwXe+bt2S6511eDFAK0bUSpS2MtJWvlfrJXj8KFik3De7dXujxXX CjYXk7QAjiGMvObifqZAPB3FrhQbSC0gBSrmvTXZqnbZZSz1Ba5RR3kvjbQk2rZIsq4p Ea4dBoQF34huo3S3J/+xmPRcNWiKX6UMfy2S7BD0Xhy6qNCwspz2V8weaWPLUc0Nsmx7 0BLY9GiHhpA6Xa3omCyvK7HeozEMrOnrQq9WWTd3IKRJBNrDGpICR93XPRZyldkCJgtN gHLw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=pxkwxGFUt+8j75PjoCpB3xaEMlpBLZL3A8MBJU6El10=; b=NSK7rVHWd1VEd933sv7ZIq5vknEfU+qJDU8oAOGuKtul5T6c+WbB+NNtUincTMncyx vmNHvA7qhhY5FJ9lcxtTGx/38kKP7bxqy/yW6fBC9NkQpeg1IuKJXoyquEwsvgMw6p6s DJtgBUdoUjVkebHG3qweJ4u0yydyPPN6oIHjp5sXVohovecvRWuobMq2IGz4vztufA4w MDFykmMp6shoJzGams/ljpbEXGU/STc5bvHCSQRhNUuiBfMzM1sCkmV1ZAsDvQ2UlltS KtOuMsLOquYNlH/syFzTicAk+luvRlGVY9GhD/lsjuZkpDq6CgDOwuFvRsoxmccuR6KO MURQ== X-Gm-Message-State: AOPr4FUR15kz4805h8QrAe7ewpYJqny0dg/7M5dzQbc0qbkYnQAhj/GIWUBlb+aCri9MaZD6XiTEKjYgXrZUpw== MIME-Version: 1.0 X-Received: by 10.129.71.135 with SMTP id u129mr1927134ywa.162.1463751692582; Fri, 20 May 2016 06:41:32 -0700 (PDT) Received: by 10.13.239.3 with HTTP; Fri, 20 May 2016 06:41:32 -0700 (PDT) In-Reply-To: References: Date: Fri, 20 May 2016 15:41:32 +0200 Message-ID: To: Dmitry Stogov Cc: Xinchen Hui , internals Content-Type: multipart/alternative; boundary=001a114d7490ad0a200533463f1f Subject: Re: "finally" handling refactoring (Bug #72213) From: nikita.ppv@gmail.com (Nikita Popov) --001a114d7490ad0a200533463f1f Content-Type: text/plain; charset=UTF-8 On Fri, May 20, 2016 at 2:07 PM, Dmitry Stogov wrote: > hi, > > > Please review the path > https://gist.github.com/dstogov/0a809891c6a3ac3fac4bd0d9711dd330 > > > I hope, it should completely fix https://bugs.php.net/bug.php?id=72213 > > > I'm going to commit this on Monday. > > > Thanks. Dmitry. > USE_ZEND_ALLOC=0 valgrind sapi/cli/php Zend/tests/bug65784.php gives me ==27339== Invalid read of size 8 ==27339== at 0xB93F76: ZEND_CATCH_SPEC_CONST_CV_HANDLER (zend_vm_execute.h:9362) ==27339== by 0xB7E872: execute_ex (zend_vm_execute.h:426) ==27339== by 0xB7EA74: zend_execute (zend_vm_execute.h:471) ==27339== by 0xB161EA: zend_execute_scripts (zend.c:1427) ==27339== by 0xA5319D: php_execute_script (main.c:2492) ==27339== by 0xC058D5: do_cli (php_cli.c:982) ==27339== by 0xC06D65: main (php_cli.c:1352) ==27339== Address 0xf5e0880 is 16 bytes inside a block of size 152 free'd ==27339== at 0x4C2BDEC: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) ==27339== by 0xAD5B2C: _efree (zend_alloc.c:2423) ==27339== by 0xB6C4A6: zend_objects_store_del (zend_objects_API.c:187) ==27339== by 0xB10FF6: _zval_dtor_func (zend_variables.c:56) ==27339== by 0xB6445E: i_zval_ptr_dtor (zend_variables.h:48) ==27339== by 0xB64636: zend_object_std_dtor (zend_objects.c:68) ==27339== by 0xB6C38D: zend_objects_store_del (zend_objects_API.c:178) ==27339== by 0xB10FF6: _zval_dtor_func (zend_variables.c:56) ==27339== by 0xB7BF6E: i_free_compiled_variables (zend_execute.c:2101) ==27339== by 0xB7EABD: zend_leave_helper_SPEC (zend_vm_execute.h:481) ==27339== by 0xB83F83: ZEND_FAST_RET_SPEC_HANDLER (zend_vm_execute.h:1894) ==27339== by 0xB7E872: execute_ex (zend_vm_execute.h:426) and so on. Nikita --001a114d7490ad0a200533463f1f--