Skip to content

Fix GH-18494: PDO OCI segfault in statement GC #18495

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

Closed
wants to merge 1 commit into from

Conversation

nielsdos
Copy link
Member

@nielsdos nielsdos commented May 4, 2025

This is the same issue that was fixed in 2ae897f, but now for OCI.

This is the same issue that was fixed in 2ae897f, but now for OCI.
@nielsdos nielsdos linked an issue May 4, 2025 that may be closed by this pull request
Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MSTM, you might need to make this PR "upstream" since this has been unbundled.

@nielsdos nielsdos closed this in dcf9d8f May 5, 2025
@nielsdos
Copy link
Member Author

nielsdos commented May 5, 2025

Annoying but yeah I'll upstream...

nielsdos added a commit to nielsdos/pecl-database-pdo_oci that referenced this pull request May 5, 2025
This is the same issue that was fixed in 2ae897fff7af3a794a31a8aeeeeb4f21f6a41393, but now for OCI.

Fixed at php-src on 8.3 in php/php-src#18495
mvorisek pushed a commit to php/pecl-database-pdo_oci that referenced this pull request May 5, 2025
This is the same issue that was fixed in php/php-src@2ae897f, but now for PDO OCI.

Fixed at php-src on 8.3 in php/php-src#18495.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PDO OCI segfault in statement GC
2 participants