Bug #6660 mysqldump creates bad pathnames on Windows

This is a modifiction of my previous patch after receiving feedback. This is a better way to fix the problem.  With this patch, data directory and index directory will use only forward slashes (/) when on Windows.

mysqldump.c:
  Removed fixPaths routine.  Was improper fix for bug #6660
sql_show.cc:
  Changed append_directory to convert backslashes to foward slashes when on Windows.


sql/sql_show.cc:
  Changed append_directory to convert backslashes to foward slashes when on Windows.
client/mysqldump.c:
  Removed fixPaths routine.  Was improper fix for bug #6660

1 files changed, 0 insertions, 21 deletions

diff --git a/client/mysqldump.c b/client/mysqldump.c
index 5ac5efb5128..fa36ce0242d 100644
--- a/client/mysqldump.c
+++ b/client/mysqldump.c
@@ -1081,26 +1081,6 @@ static void print_xml_row(FILE *xml_file, const char *row_name,
 }
 
 
-/*  fixPaths -- on Windows only, this function will iterate through the output
-  of show create table and change any \ characters that appear in the data directory
-  or index directory elements to be /  
-
-  RETURN    
-     void
-*/
-static void fixPaths(char *buf, int buflen) 
-{
-#ifdef __WIN__
-   int i = 0;
-   for (i=0; i < buflen; i++)
-   {
-      if (buf[i] != '\\') continue;
-      if (i != 0 && buf[i-1] == '\\') continue;
-      if (i != (buflen-1) && buf[i+1] == '\\') continue;
-      buf[i] = '/';}
-#endif
-}
-
 /*
   getStructure -- retrievs database structure, prints out corresponding
   CREATE statement and fills out insert_pat.
@@ -1180,7 +1160,6 @@ static uint getTableStructure(char *table, char* db)
 
       tableRes=mysql_store_result(sock);
       row=mysql_fetch_row(tableRes);
-      fixPaths(row[1], strlen(row[1]));    // this really only does something on Windows
       fprintf(sql_file, "%s;\n", row[1]);
       check_io(sql_file);
       mysql_free_result(tableRes);