Skip to content

Commit 3bb3785

Browse files
committed
Bug#36248967: Issue in mysqldump (mysql dump utility)
Post push fix. Remove compiler warnings when building for MySQL 5.7, MySQL Cluster 7.5 and 7.6. Change-Id: I5e8decbf776f8a1f4171dd3beae257235418756f
1 parent c532ed1 commit 3bb3785

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

client/mysqldump.c

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -705,29 +705,33 @@ static void short_usage(void)
705705
static void get_safe_server_info(char *safe_server_info,
706706
size_t safe_server_info_len) {
707707
const char *server_info = mysql_get_server_info(&mysql_connection);
708+
size_t i;
708709
if (server_info == NULL) {
709710
safe_server_info[0] = 0;
710711
return;
711712
}
712-
DBUG_EXECUTE_IF("server_version_injection_test", {
713-
const char *payload = "5.7.0-injection_test\n\\! touch /tmp/xxx";
714-
server_info = payload;
713+
DBUG_EXECUTE_IF("server_version_injection_test",
714+
{
715+
const char *payload= "5.7.0-injection_test\n\\! touch /tmp/xxx";
716+
server_info= payload;
715717
});
716-
for (size_t i = 0; i < safe_server_info_len; ++i) {
718+
for (i = 0; i < safe_server_info_len; ++i) {
717719
// End of string.
718-
if (server_info[i] == 0) {
719-
safe_server_info[i] = 0;
720+
if (server_info[i] == 0)
721+
{
722+
safe_server_info[i]= 0;
720723
return;
721724
}
722725
// Version may include only alphanumeric and punctuation characters.
723726
// Cut off the rest of the string if incorrect character found.
724-
if (!(isalnum(server_info[i]) || ispunct(server_info[i]))) {
727+
if (!(my_isalnum(charset_info, server_info[i]) || my_ispunct(charset_info, server_info[i])))
728+
{
725729
safe_server_info[i] = 0;
726730
fprintf(stderr,
727731
"-- Warning: version string returned by server is incorrect.\n");
728732
return;
729733
}
730-
safe_server_info[i] = server_info[i];
734+
safe_server_info[i]= server_info[i];
731735
}
732736
safe_server_info[safe_server_info_len - 1] = 0;
733737
}

0 commit comments

Comments
 (0)