From 1994ed49ecf09b9de26167a850915ef3de2a4ce6 Mon Sep 17 00:00:00 2001 From: "monty@mysql.com" <> Date: Wed, 29 Mar 2006 14:27:36 +0300 Subject: 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 --- sql/sp_head.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'sql/sp_head.cc') diff --git a/sql/sp_head.cc b/sql/sp_head.cc index cec3a19a542..cb82d6eb1a4 100644 --- a/sql/sp_head.cc +++ b/sql/sp_head.cc @@ -1224,10 +1224,10 @@ sp_head::execute_function(THD *thd, Item **argp, uint argcount, sp_rcontext *octx = thd->spcont; sp_rcontext *nctx = NULL; bool err_status= FALSE; - DBUG_ENTER("sp_head::execute_function"); DBUG_PRINT("info", ("function %s", m_name.str)); + LINT_INIT(binlog_save_options); params = m_pcont->context_pvars(); /* -- cgit v1.2.1