diff options
author | Christoph M. Becker <cmbecker69@gmx.de> | 2021-02-08 19:01:42 +0100 |
---|---|---|
committer | Christoph M. Becker <cmbecker69@gmx.de> | 2021-02-08 19:01:42 +0100 |
commit | 882b418a014890b9b6763825ce9766f21da38a6f (patch) | |
tree | 4e7be0b2d35f8142cb56b4bbae641e3d75f798b9 /ext/pgsql/pgsql.c | |
parent | 050214b1f1497bfebe031e32a40c197080511844 (diff) | |
parent | 92f71d9d42e620c3a08219728a2bfe1abb85a214 (diff) | |
download | php-git-882b418a014890b9b6763825ce9766f21da38a6f.tar.gz |
Merge branch 'PHP-8.0'
* PHP-8.0:
Fix locale dependent parsing of PostgreSQL version number
Diffstat (limited to 'ext/pgsql/pgsql.c')
-rw-r--r-- | ext/pgsql/pgsql.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/ext/pgsql/pgsql.c b/ext/pgsql/pgsql.c index 821af14efa..5565d8fc85 100644 --- a/ext/pgsql/pgsql.c +++ b/ext/pgsql/pgsql.c @@ -598,7 +598,8 @@ static void php_pgsql_do_connect(INTERNAL_FUNCTION_PARAMETERS, int persistent) } } pgsql = (PGconn *) le->ptr; - if (PQprotocolVersion(pgsql) >= 3 && atof(PQparameterStatus(pgsql, "server_version")) >= 7.2) { + /* consider to use php_version_compare() here */ + if (PQprotocolVersion(pgsql) >= 3 && zend_strtod(PQparameterStatus(pgsql, "server_version"), NULL) >= 7.2) { pg_result = PQexec(pgsql, "RESET ALL;"); PQclear(pg_result); } @@ -4063,14 +4064,16 @@ PHP_FUNCTION(pg_get_notify) if (result_type & PGSQL_NUM) { add_index_string(return_value, 0, pgsql_notify->relname); add_index_long(return_value, 1, pgsql_notify->be_pid); - if (PQprotocolVersion(pgsql) >= 3 && atof(PQparameterStatus(pgsql, "server_version")) >= 9.0) { + /* consider to use php_version_compare() here */ + if (PQprotocolVersion(pgsql) >= 3 && zend_strtod(PQparameterStatus(pgsql, "server_version"), NULL) >= 9.0) { add_index_string(return_value, 2, pgsql_notify->extra); } } if (result_type & PGSQL_ASSOC) { add_assoc_string(return_value, "message", pgsql_notify->relname); add_assoc_long(return_value, "pid", pgsql_notify->be_pid); - if (PQprotocolVersion(pgsql) >= 3 && atof(PQparameterStatus(pgsql, "server_version")) >= 9.0) { + /* consider to use php_version_compare() here */ + if (PQprotocolVersion(pgsql) >= 3 && zend_strtod(PQparameterStatus(pgsql, "server_version"), NULL) >= 9.0) { add_assoc_string(return_value, "payload", pgsql_notify->extra); } } |