Commit a0d5521f authored by Tatiana A. Nurnberg's avatar Tatiana A. Nurnberg

Bug#47655: Memory free error when connecting to 4.1 server from 5.1+ client

When starting the (5.1+) mysql command-line client, we try to get
"select @@version_comment" from the server to present it to the
user. Recent clients are aware that older servers do not have that
variable and fall back on other info to be able to present *something*
at least. This fallback string was allocated through the POSIX interface,
but released through the my*() suite, which rightfully complained about
the imbalance in calls when compiled with --debug. While this wasn't
as bad as it looked (no double-free, use of uninitialized or freed
buffer, etc.), it did look funky.

Using my_strdup() now for what will be my_free()d later.
parent 842039a1
...@@ -4474,9 +4474,7 @@ server_version_string(MYSQL *con) ...@@ -4474,9 +4474,7 @@ server_version_string(MYSQL *con)
*/ */
if (server_version == NULL) if (server_version == NULL)
{ server_version= my_strdup(mysql_get_server_info(con), MYF(MY_WME));
server_version= strdup(mysql_get_server_info(con));
}
} }
return server_version ? server_version : ""; return server_version ? server_version : "";
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment