diff --git a/NEWS b/NEWS index a872ba7cc69..89dc3f9d124 100644 --- a/NEWS +++ b/NEWS @@ -35,6 +35,10 @@ PHP NEWS . Fixed bug #79917 (File cache segfault with a static variable in inherited method). (Nikita) +- PDO: + . Fixed bug #64705 (errorInfo property of PDOException is null when + PDO::__construct() fails). (Ahmed Abdou) + - Standard: . Fixed bug #79930 (array_merge_recursive() crashes when called with array with single reference). (Nikita) diff --git a/ext/pdo/pdo_dbh.c b/ext/pdo/pdo_dbh.c index aa0edbbb472..1396816fe90 100644 --- a/ext/pdo/pdo_dbh.c +++ b/ext/pdo/pdo_dbh.c @@ -37,6 +37,33 @@ static int pdo_dbh_attribute_set(pdo_dbh_t *dbh, zend_long attr, zval *value); +void pdo_throw_exception(unsigned int driver_errcode, char *driver_errmsg, pdo_error_type *pdo_error) +{ + zval error_info,pdo_exception; + char *pdo_exception_message; + + object_init_ex(&pdo_exception, php_pdo_get_exception()); + array_init(&error_info); + + add_next_index_string(&error_info, *pdo_error); + add_next_index_long(&error_info, driver_errcode); + add_next_index_string(&error_info, driver_errmsg); + + spprintf(&pdo_exception_message, 0,"SQLSTATE[%s] [%d] %s",*pdo_error, driver_errcode, driver_errmsg); + zend_update_property(php_pdo_get_exception(), &pdo_exception, "errorInfo", sizeof("errorInfo")-1, &error_info); + zend_update_property_long(php_pdo_get_exception(), &pdo_exception, "code", sizeof("code")-1, driver_errcode); + zend_update_property_string( + php_pdo_get_exception(), + &pdo_exception, + "message", + sizeof("message")-1, + pdo_exception_message + ); + efree(pdo_exception_message); + zval_ptr_dtor(&error_info); + zend_throw_exception_object(&pdo_exception); +} + void pdo_raise_impl_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *sqlstate, const char *supp) /* {{{ */ { pdo_error_type *pdo_err = &dbh->error_code; diff --git a/ext/pdo/php_pdo_driver.h b/ext/pdo/php_pdo_driver.h index d057857fa63..e56a6c47347 100644 --- a/ext/pdo/php_pdo_driver.h +++ b/ext/pdo/php_pdo_driver.h @@ -691,4 +691,5 @@ PDO_API void php_pdo_dbh_delref(pdo_dbh_t *dbh); PDO_API void php_pdo_free_statement(pdo_stmt_t *stmt); +PDO_API void pdo_throw_exception(unsigned int driver_errcode, char *driver_errmsg, pdo_error_type *pdo_error); #endif /* PHP_PDO_DRIVER_H */ diff --git a/ext/pdo_mysql/mysql_driver.c b/ext/pdo_mysql/mysql_driver.c index cceb587b4e4..5b57f84816b 100644 --- a/ext/pdo_mysql/mysql_driver.c +++ b/ext/pdo_mysql/mysql_driver.c @@ -103,8 +103,7 @@ int _pdo_mysql_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int lin if (!dbh->methods) { PDO_DBG_INF("Throwing exception"); - zend_throw_exception_ex(php_pdo_get_exception(), einfo->errcode, "SQLSTATE[%s] [%d] %s", - *pdo_err, einfo->errcode, einfo->errmsg); + pdo_throw_exception(einfo->errcode, einfo->errmsg, pdo_err); } PDO_DBG_RETURN(einfo->errcode); diff --git a/ext/pdo_mysql/tests/bug_64705.phpt b/ext/pdo_mysql/tests/bug_64705.phpt new file mode 100644 index 00000000000..53e79757df2 --- /dev/null +++ b/ext/pdo_mysql/tests/bug_64705.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #64705 errorInfo property of PDOException is null when PDO::__construct() fails +--SKIPIF-- + +--FILE-- +errorInfo) && is_array($e->errorInfo)); +} +?> +--EXPECTF-- +bool(true) \ No newline at end of file diff --git a/ext/pdo_pgsql/pgsql_driver.c b/ext/pdo_pgsql/pgsql_driver.c index 403bfd611ae..74c7024a88f 100644 --- a/ext/pdo_pgsql/pgsql_driver.c +++ b/ext/pdo_pgsql/pgsql_driver.c @@ -96,8 +96,7 @@ int _pdo_pgsql_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, int errcode, const char * } if (!dbh->methods) { - zend_throw_exception_ex(php_pdo_get_exception(), einfo->errcode, "SQLSTATE[%s] [%d] %s", - *pdo_err, einfo->errcode, einfo->errmsg); + pdo_throw_exception(einfo->errcode, einfo->errmsg, pdo_err); } return errcode; diff --git a/ext/pdo_pgsql/tests/bug_64705.phpt b/ext/pdo_pgsql/tests/bug_64705.phpt new file mode 100644 index 00000000000..464b4d0769a --- /dev/null +++ b/ext/pdo_pgsql/tests/bug_64705.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #64705 errorInfo property of PDOException is null when PDO::__construct() fails +--SKIPIF-- + +--FILE-- +errorInfo) && is_array($e->errorInfo)); +} +?> +--EXPECTF-- +bool(true) \ No newline at end of file diff --git a/ext/pdo_sqlite/sqlite_driver.c b/ext/pdo_sqlite/sqlite_driver.c index 120488e2296..83ea2d6083d 100644 --- a/ext/pdo_sqlite/sqlite_driver.c +++ b/ext/pdo_sqlite/sqlite_driver.c @@ -76,8 +76,7 @@ int _pdo_sqlite_error(pdo_dbh_t *dbh, pdo_stmt_t *stmt, const char *file, int li } if (!dbh->methods) { - zend_throw_exception_ex(php_pdo_get_exception(), einfo->errcode, "SQLSTATE[%s] [%d] %s", - *pdo_err, einfo->errcode, einfo->errmsg); + pdo_throw_exception(einfo->errcode, einfo->errmsg, pdo_err); } return einfo->errcode; diff --git a/ext/pdo_sqlite/tests/bug_64705.phpt b/ext/pdo_sqlite/tests/bug_64705.phpt new file mode 100644 index 00000000000..861a99fbd9b --- /dev/null +++ b/ext/pdo_sqlite/tests/bug_64705.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #64705 errorInfo property of PDOException is null when PDO::__construct() fails +--SKIPIF-- + +--FILE-- +errorInfo) && is_array($e->errorInfo)); +} +?> +--EXPECTF-- +bool(true) \ No newline at end of file