diff options
author | Andrey Hristov <andrey@php.net> | 2011-01-24 13:58:18 +0000 |
---|---|---|
committer | Andrey Hristov <andrey@php.net> | 2011-01-24 13:58:18 +0000 |
commit | 4439bde5601d9e02c2d187ac5092aba2a197baee (patch) | |
tree | e0ca06b4739d32a440bc0ac696df516ae9038d30 /ext/mysqlnd/mysqlnd_auth.c | |
parent | ce4e6f4db5ce8f7a292bd6479c70f66c509fadb0 (diff) | |
download | php-git-4439bde5601d9e02c2d187ac5092aba2a197baee.tar.gz |
small cleanups
Diffstat (limited to 'ext/mysqlnd/mysqlnd_auth.c')
-rw-r--r-- | ext/mysqlnd/mysqlnd_auth.c | 12 |
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)) { |