-
Notifications
You must be signed in to change notification settings - Fork 7.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pdo lazy object crash #18114
Labels
Comments
Reduced: <?php
$db = new PDO('sqlite::memory:');
$x = $db->query('select 1 as queryString');
foreach ($x->fetch(PDO::FETCH_LAZY) as $entry) {
} |
Fun fun, so it leaks on 8.3-8.4 and it crashes on master |
Caused by 0537968, somehow... |
The problem is that the |
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Mar 19, 2025
Since 0537968, the properties are no longer initialized. So we call object_properties_init to handle that correctly. Lower branches have a memory leak, but that requires a separate fix.
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Mar 19, 2025
Since 0537968, the properties are no longer initialized. So we call object_properties_init to handle that correctly. Lower branches have a memory leak, but that requires a separate fix.
Reopening as a reminder to also solve the leak problem on lower branches |
nielsdos
added a commit
to nielsdos/php-src
that referenced
this issue
Mar 20, 2025
This should call zend_object_std_dtor() to clean the property table etc. This also has a semantic influence because previously weak refs were not notified for example. This fixes the final issue in phpGH-18114 (the crash was master-only and fixed already). Closes phpGH-18114.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The following code:
Resulted in this output:
To reproduce:
Commit:
Configurations:
Operating System:
This report is automatically generated by FlowFusion
PHP Version
cd58662
Operating System
No response
The text was updated successfully, but these errors were encountered: