mirror of
https://github.com/php/php-src.git
synced 2026-03-24 00:02:20 +01:00
Merge branch 'PHP-8.4'
* PHP-8.4: Fix uaf in SplFixedArray::unset()
This commit is contained in:
@@ -459,8 +459,10 @@ static void spl_fixedarray_object_unset_dimension_helper(spl_fixedarray_object *
|
||||
zend_throw_exception(spl_ce_OutOfBoundsException, "Index invalid or out of range", 0);
|
||||
return;
|
||||
} else {
|
||||
zval_ptr_dtor(&(intern->array.elements[index]));
|
||||
zval garbage;
|
||||
ZVAL_COPY_VALUE(&garbage, &intern->array.elements[index]);
|
||||
ZVAL_NULL(&intern->array.elements[index]);
|
||||
zval_ptr_dtor(&garbage);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
21
ext/spl/tests/gh16478.phpt
Normal file
21
ext/spl/tests/gh16478.phpt
Normal file
@@ -0,0 +1,21 @@
|
||||
--TEST--
|
||||
GH-16478: Use-after-free in SplFixedArray::unset()
|
||||
--FILE--
|
||||
<?php
|
||||
|
||||
class C {
|
||||
function __destruct() {
|
||||
global $arr;
|
||||
$arr->setSize(0);
|
||||
}
|
||||
}
|
||||
|
||||
$arr = new SplFixedArray(2);
|
||||
$arr[0] = new C;
|
||||
unset($arr[0]);
|
||||
var_dump($arr);
|
||||
|
||||
?>
|
||||
--EXPECT--
|
||||
object(SplFixedArray)#1 (0) {
|
||||
}
|
||||
Reference in New Issue
Block a user