Bug #72213 [PATCH]: Finally leaks on nested exceptions

From: Date: Mon, 16 May 2016 15:37:41 +0000
Subject: Bug #72213 [PATCH]: Finally leaks on nested exceptions
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-201133@lists.php.net to get a copy of this message 
Edit report at https://bugs.php.net/bug.php?id=72213&edit=1

 ID:                 72213
 Patch added by:     laruence@php.net
 Reported by:        nikic@php.net
 Summary:            Finally leaks on nested exceptions
 Status:             Open
 Type:               Bug
 Package:            Scripting Engine problem
 PHP Version:        master-Git-2016-05-13 (Git)
 Block user comment: N
 Private report:     N

 New Comment:

The following patch has been added/updated:

Patch Name: bug72213.diff
Revision:   1463413056
URL:        https://bugs.php.net/patch-display.php?bug=72213&patch=bug72213.diff&revision=1463413056


Previous Comments:
------------------------------------------------------------------------
[2016-05-13 14:55:30] nikic@php.net

Description:
------------
Attached script causes a leak:

caught b
[Fri May 13 16:51:34 2016]  Script:  '/home/nikic/php-src/t275.php'
/home/nikic/php-src/Zend/zend_objects.c(174) :  Freeing 0x7F6FB46740C0 (152 bytes),
script=/home/nikic/php-src/t275.php
=== Total 1 memory leaks detected ===

The leak is the exception backed up in the outer fast_call, which needs to be discarded.

Test script:
---------------
<?php
function test() {
    try {
        throw new Exception('a');
    } finally {
        try {
            throw new Exception('b');
        } finally {
        }
    }
}

try {
    test();
} catch (Exception $e) {
    echo "caught {$e->getMessage()}\n";
}




------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=72213&edit=1

Thread (8 messages)

« previous php.bugs (#201133) next »