Bug #74569 [Com]: Return in a finally clause silently ignores an exception thrown in a try clause
Edit report at https://bugs.php.net/bug.php?id=74569&edit=1
ID: 74569
Comment by: olavisau at gmail dot com
Reported by: trianman at gmail dot com
Summary: Return in a finally clause silently ignores an
exception thrown in a try clause
Status: Open
Type: Bug
Package: Scripting Engine problem
Operating System: Any
PHP Version: 7.1.4
Block user comment: N
Private report: N
New Comment:
The correct way of writing this would be to catch the exception and throw it again, however this
also fails. If it is not thrown again, then the behavior is correct.
<?php
class MyException extends Exception
{
public function __construct($message = "", $code = 0, Throwable $previous = null)
{
parent::__construct($message, $code, $previous);
print "2. Exception constructed\n";
}
}
function throw_exception() {
try {
throw new MyException("5. Exception message\n");
} catch(Exception $exception) {
print "3. Exception caught and thrown again\n";
throw $exception;
} finally {
return "4. Return statement\n";
}
}
try {
print "1. Start\n";
print throw_exception();
} catch (Exception $e) {
print $e->getMessage();
}
The flow should be: exception is thrown, caught by the inner block, thrown again, caught by the
outer block and then the finally block should execute, returning the flow to
print throw_exception();
Previous Comments:
------------------------------------------------------------------------
[2017-05-10 16:44:47] trianman at gmail dot com
Description:
------------
Return in a finally clause silently ignores an exception thrown in a try clause.
I understand that there is no way to handle an exception here, but it MUST NOT be silently dropped
out.
The interpreter should trigger an E_NOTICE or something similar here.
Looks like any version of PHP from 5.6 to 7.1.4 behaves in the same manner: https://3v4l.org/C16rP
Here is a similar issue: https://bugs.php.net/bug.php?id=68270
Test script:
---------------
<?php
class MyException extends Exception
{
public function __construct($message = "", $code = 0, Throwable $previous = null)
{
print "2. Exception constructor\n";
parent::__construct($message, $code, $previous);
}
}
function ololo() {
try {
print "1. Start\n";
throw new MyException("4. Exception message\n");
} finally {
return "3. Return statement\n";
}
}
try {
print ololo();
} catch (Exception $e) {
print $e->getMessage();
}
Expected result:
----------------
1. Start
2. Exception constructor
PHP NOTICE: Dropped exception due the return statement in a finally block
3. Return statement
Actual result:
--------------
1. Start
2. Exception constructor
3. Return statement
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=74569&edit=1
Thread (7 messages)