MySQL Lists are EOL. Please join:

List:Commits« Previous MessageNext Message »
From:Chad MILLER Date:August 18 2008 4:16pm
Subject:bzr commit into mysql-5.1 branch (chad:2711) Bug#36399
View as plain text  
#At file:///home/cmiller/work/mysqlbzr/mysql-5.1-bugteam/

 2711 Chad MILLER	2008-08-18
      Fix for bug#36399: Double free bug when INFORMATION_SCHEMA
      plugin init function fails
      
      Problem: if an INFORMATION_SCHEMA plugin initialization fails
      we free some inner plugin's data (schema_table) twice during the 
      INSTALL PLUGIN command.
      
      Fix: free it once.
modified:
  sql/sql_show.cc

=== modified file 'sql/sql_show.cc'
--- a/sql/sql_show.cc	2008-08-15 20:42:29 +0000
+++ b/sql/sql_show.cc	2008-08-18 16:15:56 +0000
@@ -6644,17 +6644,15 @@ int initialize_schema_table(st_plugin_in
     {
       sql_print_error("Plugin '%s' init function returned error.",
                       plugin->name.str);
-      goto err;
+      plugin->data= NULL;
+      my_free(schema_table, MYF(0));
+      DBUG_RETURN(1);
     }
     
     /* Make sure the plugin name is not set inside the init() function. */
     schema_table->table_name= plugin->name.str;
   }
-
   DBUG_RETURN(0);
-err:
-  my_free(schema_table, MYF(0));
-  DBUG_RETURN(1);
 }
 
 int finalize_schema_table(st_plugin_int *plugin)

Thread
bzr commit into mysql-5.1 branch (chad:2711) Bug#36399Chad MILLER18 Aug