Repository navigation
Memory leaks in SPL constructors #16604
Description
Description
The following code:
<?php
$traversable = new RecursiveArrayIterator( [] );
$obj = new RecursiveIteratorIterator( $traversable );
$obj->__construct( $traversable );Resulted in this output:
[Fri Oct 25 19:55:20 2024] Script: '/var/www/html/doubleConstruct.php'
/usr/src/php/ext/spl/spl_iterators.c(597) : Freeing 0x00007f8883202b40 (56 bytes), script=/var/www/html/doubleConstruct.php
[Fri Oct 25 19:55:20 2024] Script: '/var/www/html/doubleConstruct.php'
/usr/src/php/Zend/zend_objects_API.h(94) : Freeing 0x00007f888324f480 (144 bytes), script=/var/www/html/doubleConstruct.php
[Fri Oct 25 19:55:20 2024] Script: '/var/www/html/doubleConstruct.php'
/usr/src/php/ext/spl/spl_array.c(1645) : Freeing 0x00007f888325f200 (96 bytes), script=/var/www/html/doubleConstruct.php
=== Total 3 memory leaks detected ===
But I expected this output instead:
[none]
Other SPL classes with this issue
- RecursiveTreeIterator
- SplFileObject
- SplTempFileObject
PHP Version
8.5-dev
Operating System
No response
Activity
Girgias commentedon Oct 25, 2024
Those would be fixed by #16480
Fix GH-16477 (Segmentation fault when calling __debugInfo() after fai…
Fix GH-16477 (Segmentation fault when calling __debugInfo() after fai…
Fix GH-16477 (Segmentation fault when calling __debugInfo() after fai…
Girgias commentedon Oct 25, 2024
Please double-check this is still happening now that the PR has been merged.
DanielEScherzer commentedon Oct 26, 2024
Still an issue for RecursiveIteratorIterator, RecursiveTreeIterator, and SplFileObject
SplTempFileObject now throws an error for calling the constructor a second time, so that is fixed
Fix phpGH-16604: Memory leaks in SPL constructors
Fix GH-16604: Memory leaks in SPL constructors
Merge branch 'PHP-8.2' into PHP-8.3
Merge branch 'PHP-8.3' into PHP-8.4
Merge branch 'PHP-8.4'