Commit 76c982ea authored by unknown's avatar unknown

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


BitKeeper/etc/ignore:
  Added innobase/stamp-h1 stamp-h1 to the ignore list
parent 91e1b1b1
...@@ -331,3 +331,5 @@ bdb/dist/autom4te.cache/traces.0 ...@@ -331,3 +331,5 @@ bdb/dist/autom4te.cache/traces.0
innobase/autom4te.cache/output.0 innobase/autom4te.cache/output.0
innobase/autom4te.cache/requests innobase/autom4te.cache/requests
innobase/autom4te.cache/traces.0 innobase/autom4te.cache/traces.0
innobase/stamp-h1
stamp-h1
...@@ -1528,8 +1528,14 @@ int mysql_alter_table(THD *thd,char *new_db, char *new_name, ...@@ -1528,8 +1528,14 @@ int mysql_alter_table(THD *thd,char *new_db, char *new_name,
/* We changed a temporary table */ /* We changed a temporary table */
if (error) 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); close_temporary_table(thd,new_db,tmp_name);
my_free((gptr) new_table,MYF(0));
goto err; goto err;
} }
/* Close lock if this is a transactional table */ /* Close lock if this is a transactional table */
......
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