List:Commits« Previous MessageNext Message »
From:Bjorn Munch Date:January 11 2011 2:24pm
Subject:bzr commit into mysql-5.5-mtr branch (bjorn.munch:3140) Bug#59153
View as plain text  
#At file:///home/bm136801/my/genplug-55/ based on revid:bjorn.munch@stripped

 3140 Bjorn Munch	2011-01-11
      Bug #59153 mysqltest produces a valgrind warning when the running test fails
      Local variable ds_warnings in run_query not cleared.
      But when we call die() we don't have access to it.
      Set global var. to point to it when allocated.

    modified:
      client/mysqltest.cc
=== modified file 'client/mysqltest.cc'
--- a/client/mysqltest.cc	2011-01-11 14:15:25 +0000
+++ b/client/mysqltest.cc	2011-01-11 14:23:39 +0000
@@ -468,6 +468,8 @@ TYPELIB command_typelib= {array_elements
 			  command_names, 0};
 
 DYNAMIC_STRING ds_res;
+/* Points to ds_warning in run_query, so it can be freed */
+DYNAMIC_STRING *ds_warn= 0;
 
 char builtin_echo[FN_REFLEN];
 
@@ -1275,6 +1277,8 @@ void free_used_memory()
     my_free(embedded_server_args[--embedded_server_arg_count]);
   delete_dynamic(&q_lines);
   dynstr_free(&ds_res);
+  if (ds_warn)
+    dynstr_free(ds_warn);
   free_all_replace();
   my_free(opt_pass);
   free_defaults(default_argv);
@@ -7693,6 +7697,8 @@ void run_query(struct st_connection *cn,
     die ("Cannot reap on a connection without pending send");
   
   init_dynamic_string(&ds_warnings, NULL, 0, 256);
+  ds_warn= &ds_warnings;
+  
   /*
     Evaluate query if this is an eval command
   */
@@ -7850,6 +7856,7 @@ void run_query(struct st_connection *cn,
 		     ds, &ds_warnings);
 
   dynstr_free(&ds_warnings);
+  ds_warn= 0;
   if (command->type == Q_EVAL || command->type == Q_SEND_EVAL)
     dynstr_free(&eval_query);
 


Attachment: [text/bzr-bundle] bzr/bjorn.munch@oracle.com-20110111142339-2f3tgt6m1pdj24br.bundle
Thread
bzr commit into mysql-5.5-mtr branch (bjorn.munch:3140) Bug#59153Bjorn Munch11 Jan