List:Commits« Previous MessageNext Message »
From:Georgi Kodinov Date:January 14 2011 3:48pm
Subject:bzr commit into mysql-5.5 branch (Georgi.Kodinov:3247) Bug#59453
View as plain text  
#At file:///Users/kgeorge/mysql/work/B59453-5.5/ based on revid:georgi.kodinov@stripped

 3247 Georgi Kodinov	2011-01-14
      Bug #59453: Non-ASCIIZ string kills protocol extensibility in MySQL 5.5
      When the server sends the name of the plugin it's using in the handshake 
      packet it was null terminating it in it's buffer, but was sending a length of 
      the packet 1 byte short.
      Fixed to send the terminating 0 as well by increasing the length of the 
      packet to include it. 
      In this way the handshake packet becomes similar to the change user packet
      where the plugin name is null terminated.
      No test suite added as the fix can only be observed by analyzing the bytes
      sent over the wire.

=== modified file 'sql/'
--- a/sql/	2010-12-17 11:11:34 +0000
+++ b/sql/	2011-01-14 15:48:11 +0000
@@ -8014,7 +8014,7 @@ static bool send_server_handshake_packet
   end= strmake(end, plugin_name(mpvio->plugin)->str,
-  int res= my_net_write(mpvio->net, (uchar*) buff, (size_t) (end - buff)) ||
+  int res= my_net_write(mpvio->net, (uchar*) buff, (size_t) (end - buff + 1)) ||
   DBUG_RETURN (res);

Attachment: [text/bzr-bundle] bzr/
bzr commit into mysql-5.5 branch (Georgi.Kodinov:3247) Bug#59453Georgi Kodinov14 Jan
  • Re: bzr commit into mysql-5.5 branch (Georgi.Kodinov:3247)Bug#59453Sergei Golubchik14 Jan
    • Re: bzr commit into mysql-5.5 branch (Georgi.Kodinov:3247) Bug#59453Davi Arnaut14 Jan
      • Re: bzr commit into mysql-5.5 branch (Georgi.Kodinov:3247)Bug#59453Sergei Golubchik14 Jan