diff options
author | monty@mysql.com <> | 2006-03-29 14:27:36 +0300 |
---|---|---|
committer | monty@mysql.com <> | 2006-03-29 14:27:36 +0300 |
commit | 1994ed49ecf09b9de26167a850915ef3de2a4ce6 (patch) | |
tree | 4d8a2c78f1c611f41d2ea762390198c528239c80 /sql/sp_head.cc | |
parent | 8cb3cf2382e9c8a3afb5d03ec4199e1ac4528bde (diff) | |
download | mariadb-git-1994ed49ecf09b9de26167a850915ef3de2a4ce6.tar.gz |
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
Diffstat (limited to 'sql/sp_head.cc')
-rw-r--r-- | sql/sp_head.cc | 2 |
1 files changed, 1 insertions, 1 deletions
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(); /* |