Skip to content

Commit ff84598

Browse files
committed
Merge branch 'PHP-8.2'
* PHP-8.2: [ci skip] NEWS Fix GH-10437: Set active fiber to null on bailout (#10443)
2 parents 306a72a + 5e1b966 commit ff84598

File tree

2 files changed

+19
-0
lines changed

2 files changed

+19
-0
lines changed

Zend/tests/fibers/gh10437.phpt

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
--TEST--
2+
GH-10437 (Segfault/assertion when using fibers in shutdown function after bailout)
3+
--FILE--
4+
<?php
5+
6+
register_shutdown_function(function (): void {
7+
var_dump(Fiber::getCurrent());
8+
});
9+
10+
$fiber = new Fiber(function (): never {
11+
trigger_error('Bailout in fiber', E_USER_ERROR);
12+
});
13+
$fiber->start();
14+
15+
?>
16+
--EXPECTF--
17+
Fatal error: Bailout in fiber in %sgh10437.php on line %d
18+
NULL

Zend/zend_fibers.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -645,6 +645,7 @@ static zend_always_inline zend_fiber_transfer zend_fiber_switch_to(
645645

646646
/* Forward bailout into current fiber. */
647647
if (UNEXPECTED(transfer.flags & ZEND_FIBER_TRANSFER_FLAG_BAILOUT)) {
648+
EG(active_fiber) = NULL;
648649
zend_bailout();
649650
}
650651

0 commit comments

Comments
 (0)