A fix for double free'd pointer bug ...

d527bfbd · Sinisa@sinisa.nasamreza.org · cb6d9e0c · d527bfbd · d527bfbd
Commit d527bfbd authored Dec 04, 2002 by Sinisa@sinisa.nasamreza.org
Show whitespace changes
Inline Side-by-side

Showing with 9 additions and 1 deletion

.bzrignore .bzrignore +2 -0

sql/sql_table.cc sql/sql_table.cc +7 -1

No files found.
--- a/.bzrignore
+++ b/.bzrignore
@@ -331,3 +331,5 @@ bdb/dist/autom4te.cache/traces.0
 innobase/autom4te.cache/output.0
 innobase/autom4te.cache/requests
 innobase/autom4te.cache/traces.0
+innobase/stamp-h1
+stamp-h1
--- a/sql/sql_table.cc
+++ b/sql/sql_table.cc
@@ -1528,8 +1528,14 @@ int mysql_alter_table(THD *thd,char *new_db, char *new_name,
    /* We changed a temporary table */
    if (error)
    {
+      /* 
+       * The following function call will also free a 
+       * new_table pointer.
+       * Therefore, here new_table pointer is not free'd as it is 
+       * free'd in close_temporary() which is called by by the 
+       * close_temporary_table() function.
+      */
      close_temporary_table(thd,new_db,tmp_name);
-      my_free((gptr) new_table,MYF(0));
      goto err;
    }
    /* Close lock if this is a transactional table */