Skip to content

Fixed GH-17383 - pdo_firebird: PDOException has wrong code and message since PHP 8.4 #18072

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.

Sign up for GitHub

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 9 commits into from
Closed

Fixed GH-17383 - pdo_firebird: PDOException has wrong code and message since PHP 8.4 #18072

Changes from 2 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
6558bda
Added NULL check for `H->db` in `firebird_handle_closer`
SakiTakamachi Mar 15, 2025
22cfe1c
Fixed "H->db != NULL" to "H->db".
SakiTakamachi Mar 16, 2025
740322a
Added a test
SakiTakamachi Apr 18, 2025
2d6fb0c
Merge branch 'PHP-8.4' into fix/gh17383
SakiTakamachi Apr 18, 2025
5d180f0
debug
SakiTakamachi Apr 18, 2025
0d44b4c
rm debug code
SakiTakamachi Apr 18, 2025
a1f34c0
Added a host name string to the DB specification to disable trusted auth
SakiTakamachi Apr 18, 2025
eb8d99e
Added sql to create user
SakiTakamachi Apr 18, 2025
08c5f0f
[skip ci] Added parentheses
SakiTakamachi Apr 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion ext/pdo_firebird/firebird_driver.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -594,7 +594,8 @@ static void firebird_handle_closer(pdo_dbh_t *dbh) /* {{{ */
}
H->in_manually_txn = 0;

if (isc_detach_database(H->isc_status, &H->db)) {
/* isc_detach_database returns 0 on success, 1 on failure. */
if (H->db && isc_detach_database(H->isc_status, &H->db)) {
php_firebird_error(dbh);
}

Expand Down
Loading