Fixed compiler and valgrind warnings

Added missing DBUG_xxx_RETURN statements
Fixed some usage of not initialized variables (as found by valgrind)
Ensure that we don't remove locked tables used as name locks from open table cache until unlock_table_names() are called.
This was fixed by having drop_locked_name() returning any table used as a name lock so that we can free it in unlock_table_names()
This will allow Tomas to continue with his work to use namelocks to syncronize things.

Note: valgrind still produces a lot of warnings about using not initialized code and shows memory loss errors when running the ndb tests

1 files changed, 3 insertions, 1 deletions

diff --git a/sql/log.h b/sql/log.h
index 391ef54dff7..03d5466e549 100644
--- a/sql/log.h
+++ b/sql/log.h
@@ -40,7 +40,8 @@ class TC_LOG
 
 class TC_LOG_DUMMY: public TC_LOG // use it to disable the logging
 {
-  public:
+public:
+  TC_LOG_DUMMY() {}
   int open(const char *opt_name)        { return 0; }
   void close()                          { }
   int log(THD *thd, my_xid xid)         { return 1; }
@@ -354,6 +355,7 @@ public:
 class Log_event_handler
 {
 public:
+  Log_event_handler() {}
   virtual bool init()= 0;
   virtual void cleanup()= 0;