mirror of
https://github.com/MariaDB/server.git
synced 2025-07-29 05:21:33 +03:00
Copy arguments given to mysql_server_init()
Made keybuff_size longlong (To make show variables work similar on 32 and 64 bit systems) Fixed some 'not initalized variable errors' in multi-table-update. Fixed memory leak in multi-table-update. Now all tests works under valgrind without any errors. libmysqld/lib_sql.cc: Copy arguments given to mysql_server_init() mysql-test/r/temp_table.result: Update test results (after merge form 3.23) sql/handler.cc: Made keybuff_size longlong sql/mysql_priv.h: Made keybuff_size longlong sql/mysqld.cc: Made keybuff_size longlong sql/set_var.cc: Made keybuff_size longlong sql/set_var.h: Made keybuff_size longlong sql/sql_select.cc: Simple cleanup sql/sql_select.h: Make TMP_TABLE_PARAM to be allocated through Sql_alloc sql/sql_update.cc: Fixed some 'not initalized variable errors' in multi-table-update. Fixed memory leak in multi-table-update
This commit is contained in:
@ -275,11 +275,39 @@ static bool check_user(THD *thd,enum_server_command command, const char *user,
|
||||
}
|
||||
|
||||
|
||||
/*
|
||||
Make a copy of array and the strings array points to
|
||||
*/
|
||||
|
||||
char **copy_arguments(int argc, char **argv)
|
||||
{
|
||||
uint length= 0;
|
||||
char **from, **res, **end= argv+argc;
|
||||
|
||||
for (from=argv ; from != end ; from++)
|
||||
length+= strlen(*from);
|
||||
|
||||
if ((res= (char**) my_malloc(sizeof(argv)*(argc+1)+length+argc,
|
||||
MYF(MY_WME))))
|
||||
{
|
||||
char **to= res, *to_str= (char*) (res+argc+1);
|
||||
for (from=argv ; from != end ;)
|
||||
{
|
||||
*to++= to_str;
|
||||
to_str= strmov(to_str, *from++)+1;
|
||||
}
|
||||
*to= 0; // Last ptr should be null
|
||||
}
|
||||
return res;
|
||||
}
|
||||
|
||||
|
||||
extern "C"
|
||||
{
|
||||
|
||||
static my_bool inited, org_my_init_done;
|
||||
ulong max_allowed_packet, net_buffer_length;
|
||||
char ** copy_arguments_ptr= 0;
|
||||
|
||||
int STDCALL mysql_server_init(int argc, char **argv, char **groups)
|
||||
{
|
||||
@ -303,7 +331,7 @@ int STDCALL mysql_server_init(int argc, char **argv, char **groups)
|
||||
argvp = (char ***) &fake_argv;
|
||||
}
|
||||
if (!groups)
|
||||
groups = (char**) fake_groups;
|
||||
groups = (char**) fake_groups;
|
||||
|
||||
my_umask=0660; // Default umask for new files
|
||||
my_umask_dir=0700; // Default umask for new directories
|
||||
@ -319,6 +347,14 @@ int STDCALL mysql_server_init(int argc, char **argv, char **groups)
|
||||
MY_INIT((char *)"mysql_embedded"); // init my_sys library & pthreads
|
||||
}
|
||||
|
||||
/*
|
||||
Make a copy of the arguments to guard against applications that
|
||||
may change or move the initial arguments.
|
||||
*/
|
||||
if (argvp == &argv)
|
||||
if (!(copy_arguments_ptr= argv= copy_arguments(argc, argv)))
|
||||
return 1;
|
||||
|
||||
tzset(); // Set tzname
|
||||
|
||||
start_time=time((time_t*) 0);
|
||||
@ -566,6 +602,8 @@ int STDCALL mysql_server_init(int argc, char **argv, char **groups)
|
||||
|
||||
void STDCALL mysql_server_end()
|
||||
{
|
||||
my_free((char*) copy_arguments_ptr, MYF(MY_ALLOW_ZERO_PTR));
|
||||
copy_arguments_ptr=0;
|
||||
clean_up(0);
|
||||
#ifdef THREAD
|
||||
/* Don't call my_thread_end() if the application is using MY_INIT() */
|
||||
|
Reference in New Issue
Block a user