diff options
author | guirish <root@eocs0030.pok.stglabs.ibm.com> | 2020-04-10 05:45:40 -0400 |
---|---|---|
committer | Nikita Popov <nikita.ppv@gmail.com> | 2020-04-16 11:22:17 +0200 |
commit | a1c1736bfb88d006542dc9bcc10d7101617517a6 (patch) | |
tree | 829376a6901dc791c50fd3c50deb0c7abfbf53f4 | |
parent | 79a36ff7f33194cf08ebe511bd828e1cdaea8e41 (diff) | |
download | php-git-a1c1736bfb88d006542dc9bcc10d7101617517a6.tar.gz |
Fix MySQL local infile / attr handling on big endian systems
Make sure pointer types match what is used by libmysql everywhere.
Closes GH-5380.
-rw-r--r-- | ext/mysqli/mysqli_api.c | 6 | ||||
-rw-r--r-- | ext/mysqli/mysqli_nonapi.c | 4 | ||||
-rw-r--r-- | ext/mysqlnd/mysqlnd_ps.c | 10 | ||||
-rw-r--r-- | ext/pdo_mysql/mysql_driver.c | 4 |
4 files changed, 12 insertions, 12 deletions
diff --git a/ext/mysqli/mysqli_api.c b/ext/mysqli/mysqli_api.c index c4ed1bf543..8632971893 100644 --- a/ext/mysqli/mysqli_api.c +++ b/ext/mysqli/mysqli_api.c @@ -2327,7 +2327,7 @@ PHP_FUNCTION(mysqli_stmt_attr_set) #if MYSQL_VERSION_ID >= 50107 my_bool mode_b; #endif - zend_ulong mode; + unsigned long mode; zend_long attr; void *mode_p; @@ -2370,7 +2370,7 @@ PHP_FUNCTION(mysqli_stmt_attr_get) { MY_STMT *stmt; zval *mysql_stmt; - zend_ulong value = 0; + unsigned long value = 0; zend_long attr; int rc; @@ -2387,7 +2387,7 @@ PHP_FUNCTION(mysqli_stmt_attr_get) if (attr == STMT_ATTR_UPDATE_MAX_LENGTH) value = *((my_bool *)&value); #endif - RETURN_LONG((zend_ulong)value); + RETURN_LONG((unsigned long)value); } /* }}} */ diff --git a/ext/mysqli/mysqli_nonapi.c b/ext/mysqli/mysqli_nonapi.c index 0f5714bf21..4736e4fcb2 100644 --- a/ext/mysqli/mysqli_nonapi.c +++ b/ext/mysqli/mysqli_nonapi.c @@ -305,8 +305,8 @@ void mysqli_common_connect(INTERNAL_FUNCTION_PARAMETERS, zend_bool is_real_conne #if !defined(MYSQLI_USE_MYSQLND) mysql->mysql->reconnect = MyG(reconnect); #endif - - mysql_options(mysql->mysql, MYSQL_OPT_LOCAL_INFILE, (char *)&MyG(allow_local_infile)); + unsigned int allow_local_infile = MyG(allow_local_infile); + mysql_options(mysql->mysql, MYSQL_OPT_LOCAL_INFILE, (char *)&allow_local_infile); end: if (!mysqli_resource) { diff --git a/ext/mysqlnd/mysqlnd_ps.c b/ext/mysqlnd/mysqlnd_ps.c index b2017b9127..d93a5ed7b1 100644 --- a/ext/mysqlnd/mysqlnd_ps.c +++ b/ext/mysqlnd/mysqlnd_ps.c @@ -1899,8 +1899,8 @@ MYSQLND_METHOD(mysqlnd_stmt, attr_set)(MYSQLND_STMT * const s, break; } case STMT_ATTR_CURSOR_TYPE: { - unsigned int ival = *(unsigned int *) value; - if (ival > (zend_ulong) CURSOR_TYPE_READ_ONLY) { + unsigned long ival = *(unsigned long *) value; + if (ival > (unsigned long) CURSOR_TYPE_READ_ONLY) { SET_CLIENT_ERROR(stmt->error_info, CR_NOT_IMPLEMENTED, UNKNOWN_SQLSTATE, "Not implemented"); DBG_INF("FAIL"); DBG_RETURN(FAIL); @@ -1909,7 +1909,7 @@ MYSQLND_METHOD(mysqlnd_stmt, attr_set)(MYSQLND_STMT * const s, break; } case STMT_ATTR_PREFETCH_ROWS: { - unsigned int ival = *(unsigned int *) value; + unsigned long ival = *(unsigned long *) value; if (ival == 0) { ival = MYSQLND_DEFAULT_PREFETCH_ROWS; } else if (ival > 1) { @@ -1948,10 +1948,10 @@ MYSQLND_METHOD(mysqlnd_stmt, attr_get)(const MYSQLND_STMT * const s, *(zend_bool *) value= stmt->update_max_length; break; case STMT_ATTR_CURSOR_TYPE: - *(zend_ulong *) value= stmt->flags; + *(unsigned long *) value= stmt->flags; break; case STMT_ATTR_PREFETCH_ROWS: - *(zend_ulong *) value= stmt->prefetch_rows; + *(unsigned long *) value= stmt->prefetch_rows; break; default: DBG_RETURN(FAIL); diff --git a/ext/pdo_mysql/mysql_driver.c b/ext/pdo_mysql/mysql_driver.c index b7b6770327..6431ccf4e1 100644 --- a/ext/pdo_mysql/mysql_driver.c +++ b/ext/pdo_mysql/mysql_driver.c @@ -628,7 +628,7 @@ static int pdo_mysql_handle_factory(pdo_dbh_t *dbh, zval *driver_options) /* handle MySQL options */ if (driver_options) { zend_long connect_timeout = pdo_attr_lval(driver_options, PDO_ATTR_TIMEOUT, 30); - zend_long local_infile = pdo_attr_lval(driver_options, PDO_MYSQL_ATTR_LOCAL_INFILE, 0); + unsigned int local_infile = (unsigned int) pdo_attr_lval(driver_options, PDO_MYSQL_ATTR_LOCAL_INFILE, 0); zend_string *init_cmd = NULL; #ifndef PDO_USE_MYSQLND zend_string *default_file = NULL, *default_group = NULL; @@ -779,7 +779,7 @@ static int pdo_mysql_handle_factory(pdo_dbh_t *dbh, zval *driver_options) } else { #if defined(MYSQL_OPT_LOCAL_INFILE) || defined(PDO_USE_MYSQLND) // in case there are no driver options disable 'local infile' explicitly - zend_long local_infile = 0; + unsigned int local_infile = 0; if (mysql_options(H->server, MYSQL_OPT_LOCAL_INFILE, (const char *)&local_infile)) { pdo_mysql_error(dbh); goto cleanup; |