summaryrefslogtreecommitdiff
path: root/server-tools/instance-manager
diff options
context:
space:
mode:
Diffstat (limited to 'server-tools/instance-manager')
-rw-r--r--server-tools/instance-manager/commands.h1
-rw-r--r--server-tools/instance-manager/guardian.cc6
-rw-r--r--server-tools/instance-manager/instance_options.cc2
-rw-r--r--server-tools/instance-manager/mysql_connection.cc6
-rw-r--r--server-tools/instance-manager/options.cc16
-rw-r--r--server-tools/instance-manager/options.h1
-rw-r--r--server-tools/instance-manager/parse.cc1
-rw-r--r--server-tools/instance-manager/user_map.cc1
-rw-r--r--server-tools/instance-manager/user_map.h1
9 files changed, 18 insertions, 17 deletions
diff --git a/server-tools/instance-manager/commands.h b/server-tools/instance-manager/commands.h
index 2a3022c9d7e..2ab31c18b32 100644
--- a/server-tools/instance-manager/commands.h
+++ b/server-tools/instance-manager/commands.h
@@ -162,6 +162,7 @@ public:
class Syntax_error : public Command
{
public:
+ Syntax_error() {}
int execute(struct st_net *net, ulong connection_id);
};
diff --git a/server-tools/instance-manager/guardian.cc b/server-tools/instance-manager/guardian.cc
index cc3ff06256f..ad5d766ea14 100644
--- a/server-tools/instance-manager/guardian.cc
+++ b/server-tools/instance-manager/guardian.cc
@@ -77,7 +77,7 @@ void Guardian_thread::request_shutdown()
void Guardian_thread::process_instance(Instance *instance,
GUARD_NODE *current_node,
- LIST **guarded_instances,
+ LIST **guarded_instances_arg,
LIST *node)
{
uint waitchild= (uint) Instance::DEFAULT_SHUTDOWN_DELAY;
@@ -93,7 +93,7 @@ void Guardian_thread::process_instance(Instance *instance,
/* this returns true if and only if an instance was stopped for sure */
if (instance->is_crashed())
- *guarded_instances= list_delete(*guarded_instances, node);
+ *guarded_instances_arg= list_delete(*guarded_instances_arg, node);
else if ( (uint) (current_time - current_node->last_checked) > waitchild)
{
instance->kill_instance(SIGKILL);
@@ -101,7 +101,7 @@ void Guardian_thread::process_instance(Instance *instance,
Later we do node= node->next. This is ok, as we are only removing
the node from the list. The pointer to the next one is still valid.
*/
- *guarded_instances= list_delete(*guarded_instances, node);
+ *guarded_instances_arg= list_delete(*guarded_instances_arg, node);
}
return;
diff --git a/server-tools/instance-manager/instance_options.cc b/server-tools/instance-manager/instance_options.cc
index c816069c51e..3affb43bd25 100644
--- a/server-tools/instance-manager/instance_options.cc
+++ b/server-tools/instance-manager/instance_options.cc
@@ -221,8 +221,6 @@ err:
int Instance_options::fill_log_options()
{
Buffer buff;
- uint position= 0;
- char **tmp_argv= argv;
enum { MAX_LOG_OPTION_LENGTH= 256 };
char datadir[MAX_LOG_OPTION_LENGTH];
char hostname[MAX_LOG_OPTION_LENGTH];
diff --git a/server-tools/instance-manager/mysql_connection.cc b/server-tools/instance-manager/mysql_connection.cc
index 7dabc00ad2d..32618a6cdd6 100644
--- a/server-tools/instance-manager/mysql_connection.cc
+++ b/server-tools/instance-manager/mysql_connection.cc
@@ -332,12 +332,12 @@ int Mysql_connection_thread::dispatch_command(enum enum_server_command command,
{
log_info("query for connection %lu : ----\n%s\n-------------------------",
connection_id,packet);
- if (Command *command= parse_command(&instance_map, packet))
+ if (Command *com= parse_command(&instance_map, packet))
{
int res= 0;
log_info("query for connection %lu successefully parsed",connection_id);
- res= command->execute(&net, connection_id);
- delete command;
+ res= com->execute(&net, connection_id);
+ delete com;
if (!res)
log_info("query for connection %lu executed ok",connection_id);
else
diff --git a/server-tools/instance-manager/options.cc b/server-tools/instance-manager/options.cc
index 63de47a7b69..f7eab075952 100644
--- a/server-tools/instance-manager/options.cc
+++ b/server-tools/instance-manager/options.cc
@@ -312,27 +312,27 @@ int Options::load(int argc, char **argv)
'.angel.pid'.
*/
- char *angel_pid_file_name;
+ char *local_angel_pid_file_name;
char *base_name_ptr;
char *ext_ptr;
- angel_pid_file_name= (char *) malloc(strlen(Options::pid_file_name) +
- ANGEL_PID_FILE_SUFFIX_LEN);
+ local_angel_pid_file_name= (char *) malloc(strlen(Options::pid_file_name) +
+ ANGEL_PID_FILE_SUFFIX_LEN);
- strcpy(angel_pid_file_name, Options::pid_file_name);
+ strcpy(local_angel_pid_file_name, Options::pid_file_name);
- base_name_ptr= strrchr(angel_pid_file_name, '/');
+ base_name_ptr= strrchr(local_angel_pid_file_name, '/');
if (!base_name_ptr)
- base_name_ptr= angel_pid_file_name + 1;
+ base_name_ptr= local_angel_pid_file_name + 1;
ext_ptr= strrchr(base_name_ptr, '.');
if (ext_ptr)
*ext_ptr= 0;
- strcat(angel_pid_file_name, ANGEL_PID_FILE_SUFFIX);
+ strcat(local_angel_pid_file_name, ANGEL_PID_FILE_SUFFIX);
- Options::angel_pid_file_name= angel_pid_file_name;
+ Options::angel_pid_file_name= local_angel_pid_file_name;
}
else
{
diff --git a/server-tools/instance-manager/options.h b/server-tools/instance-manager/options.h
index 152f0f1643f..f7186bb4ff4 100644
--- a/server-tools/instance-manager/options.h
+++ b/server-tools/instance-manager/options.h
@@ -53,6 +53,7 @@ struct Options
static char **saved_argv;
int load(int argc, char **argv);
+ Options() {}
void cleanup();
#ifdef __WIN__
int setup_windows_defaults();
diff --git a/server-tools/instance-manager/parse.cc b/server-tools/instance-manager/parse.cc
index bdbc8045d9d..71f69b596c1 100644
--- a/server-tools/instance-manager/parse.cc
+++ b/server-tools/instance-manager/parse.cc
@@ -124,7 +124,6 @@ Command *parse_command(Instance_map *map, const char *text)
uint option_value_len;
const char *log_size;
Command *command;
- const char *saved_text= text;
bool skip= false;
const char *tmp;
diff --git a/server-tools/instance-manager/user_map.cc b/server-tools/instance-manager/user_map.cc
index aac536cee97..2f77d2f59c1 100644
--- a/server-tools/instance-manager/user_map.cc
+++ b/server-tools/instance-manager/user_map.cc
@@ -29,6 +29,7 @@ struct User
char user[USERNAME_LENGTH + 1];
uint8 user_length;
uint8 salt[SCRAMBLE_LENGTH];
+ User() {}
int init(const char *line);
};
diff --git a/server-tools/instance-manager/user_map.h b/server-tools/instance-manager/user_map.h
index 126533119eb..ca9bc861bda 100644
--- a/server-tools/instance-manager/user_map.h
+++ b/server-tools/instance-manager/user_map.h
@@ -32,6 +32,7 @@
class User_map
{
public:
+ User_map() {}
~User_map();
int init();