Skip to content

Commit

Permalink
Merge branch 'PHP-8.3'
Browse files Browse the repository at this point in the history
  • Loading branch information
bwoebi committed Jun 1, 2024
2 parents 37f683a + d98586b commit 5fe799a
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 0 deletions.
35 changes: 35 additions & 0 deletions Zend/tests/generators/gh14387.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
--TEST--
GH-14387 (Crash when stack walking in destructor of yielded from values during Generator->throw())
--FILE--
<?php

function prime(Generator $generator) {
$generator->valid();
}

$g = (function () {
yield from [null, new class {
function __destruct() {
// Trigger a stack walk, hitting a bad frame.
throw new Exception;
}
}];
})();

prime($g);

$g->throw(new Error);

?>
--EXPECTF--
Fatal error: Uncaught Error in %s:%d
Stack trace:
#0 {main}

Next Exception in %s:%d
Stack trace:
#0 %s(%d): class@anonymous->__destruct()
#1 [internal function]: {%s}()
#2 %s(%d): Generator->throw(Object(Error))
#3 {main}
thrown in %s on line %d
1 change: 1 addition & 0 deletions Zend/zend_generators.c
Original file line number Diff line number Diff line change
Expand Up @@ -466,6 +466,7 @@ static void zend_generator_throw_exception(zend_generator *generator, zval *exce
* to pretend the exception happened during the YIELD opcode. */
EG(current_execute_data) = generator->execute_data;
generator->execute_data->opline--;
generator->execute_data->prev_execute_data = original_execute_data;

if (exception) {
zend_throw_exception_object(exception);
Expand Down

0 comments on commit 5fe799a

Please sign in to comment.