From e9e4d936dcf324a030c2332c4270dc0e67d2b88c Mon Sep 17 00:00:00 2001 From: "reggie@bob.(none)" <> Date: Thu, 27 Jan 2005 14:48:26 -0600 Subject: [PATCH 1/2] Bug #5185 mysqldump for windows database gets table names with different case Added the get_actual_table_name function that issues a SHOW TABLES LIKE '%s'. This will get the table name in the proper case. We use this table name rather than the one given on the command line. This will prevent problems when importing SQL on Linux that was generated on a Windows platform where case can be an issue. mysqldump.c: call get_actual_table_name to get the table name in the proper case --- client/mysqldump.c | 37 +++++++++++++++++++++++++++++++++++-- 1 file changed, 35 insertions(+), 2 deletions(-) diff --git a/client/mysqldump.c b/client/mysqldump.c index 00f5272e3d7..db86a3714c1 100644 --- a/client/mysqldump.c +++ b/client/mysqldump.c @@ -2082,6 +2082,34 @@ static int dump_all_tables_in_db(char *database) } /* dump_all_tables_in_db */ +/* + get_actual_table_name -- executes a SHOW TABLES LIKE '%s' to get the actual table name + from the server for the table name given on the command line. we do this because + the table name given on the command line may be a different case (e.g. T1 vs t1) + + RETURN + void +*/ +static void get_actual_table_name( const char *old_table_name, char *new_table_name, int buf_size ) +{ + MYSQL_RES *tableRes; + MYSQL_ROW row; + char query[ NAME_LEN*2+3 + 50 ]; + + DBUG_ENTER("get_actual_table_name"); + + sprintf( query, "SHOW TABLES LIKE '%s'", old_table_name ); + if (mysql_query_with_error_report(sock, 0, query)) + { + safe_exit(EX_MYSQLERR); + } + + tableRes = mysql_store_result( sock ); + row = mysql_fetch_row( tableRes ); + strncpy( new_table_name, row[0], buf_size ); + mysql_free_result(tableRes); +} /* get_actual_table_name */ + static int dump_selected_tables(char *db, char **table_names, int tables) { @@ -2116,9 +2144,14 @@ static int dump_selected_tables(char *db, char **table_names, int tables) print_xml_tag1(md_result_file, "", "database name=", db, "\n"); for (; tables > 0 ; tables-- , table_names++) { - numrows = getTableStructure(*table_names, db); + char new_table_name[NAME_LEN*+3]; + + /* the table name passed on commandline may be wrong case */ + get_actual_table_name( *table_names, new_table_name, sizeof(new_table_name) ); + + numrows = getTableStructure(new_table_name, db); if (!dFlag && numrows > 0) - dumpTable(numrows, *table_names); + dumpTable(numrows, new_table_name); my_free(order_by, MYF(MY_ALLOW_ZERO_PTR)); order_by= 0; } From 9af0b70d9e48cf9e16ebd4ea1b203dece9b7b575 Mon Sep 17 00:00:00 2001 From: "reggie@bob.(none)" <> Date: Sat, 29 Jan 2005 09:25:56 -0600 Subject: [PATCH 2/2] Bug #5185 mysqldump for windows database gets table names with different case mysqldump.c: Trimmed some lines to be less than 80 chars. Using just NAME_LEN now for table name buffers --- client/mysqldump.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/client/mysqldump.c b/client/mysqldump.c index db86a3714c1..3a93adf5b36 100644 --- a/client/mysqldump.c +++ b/client/mysqldump.c @@ -2083,18 +2083,21 @@ static int dump_all_tables_in_db(char *database) /* - get_actual_table_name -- executes a SHOW TABLES LIKE '%s' to get the actual table name - from the server for the table name given on the command line. we do this because - the table name given on the command line may be a different case (e.g. T1 vs t1) + get_actual_table_name -- executes a SHOW TABLES LIKE '%s' to get the actual + table name from the server for the table name given on the command line. + we do this because the table name given on the command line may be a + different case (e.g. T1 vs t1) RETURN void */ -static void get_actual_table_name( const char *old_table_name, char *new_table_name, int buf_size ) +static void get_actual_table_name( const char *old_table_name, + char *new_table_name, + int buf_size ) { MYSQL_RES *tableRes; MYSQL_ROW row; - char query[ NAME_LEN*2+3 + 50 ]; + char query[ NAME_LEN + 50 ]; DBUG_ENTER("get_actual_table_name"); @@ -2144,7 +2147,7 @@ static int dump_selected_tables(char *db, char **table_names, int tables) print_xml_tag1(md_result_file, "", "database name=", db, "\n"); for (; tables > 0 ; tables-- , table_names++) { - char new_table_name[NAME_LEN*+3]; + char new_table_name[NAME_LEN]; /* the table name passed on commandline may be wrong case */ get_actual_table_name( *table_names, new_table_name, sizeof(new_table_name) );