summaryrefslogtreecommitdiff
path: root/ext/mysqlnd/mysqlnd_auth.c
diff options
context:
space:
mode:
authorAndrey Hristov <andrey@php.net>2011-01-24 13:58:18 +0000
committerAndrey Hristov <andrey@php.net>2011-01-24 13:58:18 +0000
commit4439bde5601d9e02c2d187ac5092aba2a197baee (patch)
treee0ca06b4739d32a440bc0ac696df516ae9038d30 /ext/mysqlnd/mysqlnd_auth.c
parentce4e6f4db5ce8f7a292bd6479c70f66c509fadb0 (diff)
downloadphp-git-4439bde5601d9e02c2d187ac5092aba2a197baee.tar.gz
small cleanups
Diffstat (limited to 'ext/mysqlnd/mysqlnd_auth.c')
-rw-r--r--ext/mysqlnd/mysqlnd_auth.c12
1 files changed, 2 insertions, 10 deletions
diff --git a/ext/mysqlnd/mysqlnd_auth.c b/ext/mysqlnd/mysqlnd_auth.c
index 184238f893..b1ad42348a 100644
--- a/ext/mysqlnd/mysqlnd_auth.c
+++ b/ext/mysqlnd/mysqlnd_auth.c
@@ -34,9 +34,9 @@ enum_func_status
mysqlnd_auth_handshake(MYSQLND * conn,
const char * const user,
const char * const passwd,
+ const size_t passwd_len,
const char * const db,
const size_t db_len,
- const size_t passwd_len,
const MYSQLND_OPTIONS * const options,
unsigned long mysql_flags,
unsigned int server_charset_no,
@@ -158,9 +158,9 @@ mysqlnd_auth_change_user(MYSQLND * const conn,
const char * const user,
const size_t user_len,
const char * const passwd,
+ const size_t passwd_len,
const char * const db,
const size_t db_len,
- const size_t passwd_len,
const zend_bool silent,
zend_bool use_full_blown_auth_packet,
const char * const auth_protocol,
@@ -172,14 +172,7 @@ mysqlnd_auth_change_user(MYSQLND * const conn,
size_t * switch_to_auth_protocol_data_len
TSRMLS_DC)
{
- /*
- User could be max 16 * 3 (utf8), pass is 20 usually, db is up to 64*3
- Stack space is not that expensive, so use a bit more to be protected against
- buffer overflows.
- */
enum_func_status ret = FAIL;
- char buffer[MYSQLND_MAX_ALLOWED_USER_LEN + 1 + SCRAMBLE_LENGTH + MYSQLND_MAX_ALLOWED_DB_LEN + 1 + 2 /* charset*/ ];
- char *p = buffer;
const MYSQLND_CHARSET * old_cs = conn->charset;
MYSQLND_PACKET_CHANGE_AUTH_RESPONSE * change_auth_resp_packet = NULL;
MYSQLND_PACKET_CHG_USER_RESPONSE * chg_user_resp = NULL;
@@ -230,7 +223,6 @@ mysqlnd_auth_change_user(MYSQLND * const conn,
if (mysqlnd_get_server_version(conn) >= 50123) {
auth_packet->charset_no = conn->charset->nr;
- p+=2;
}
if (!PACKET_WRITE(auth_packet, conn)) {