summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rwxr-xr-xBuild-tools/Do-compile4
-rw-r--r--include/mysql.h2
-rw-r--r--mysql-test/t/innodb_handler.test13
-rw-r--r--sql/mini_client.cc4
-rw-r--r--sql/mysqld.cc5
-rw-r--r--sql/slave.cc19
-rw-r--r--sql/slave.h2
-rw-r--r--sql/sql_acl.cc20
-rw-r--r--sql/sql_base.cc1
-rw-r--r--sql/sql_cache.cc1
-rw-r--r--sql/sql_select.cc3
-rw-r--r--vio/vio.c5
12 files changed, 34 insertions, 45 deletions
diff --git a/Build-tools/Do-compile b/Build-tools/Do-compile
index a4156f6ac98..b41733394ab 100755
--- a/Build-tools/Do-compile
+++ b/Build-tools/Do-compile
@@ -123,7 +123,9 @@ if ($opt_stage == 0 && ! $opt_use_old_distribution)
system("touch timestamp");
sleep(2);
system("touch timestamp2");
- system("find . -newer timestamp -print | xargs touch; rm -f timestamp timestamp2");
+ system("find . -newer timestamp -print | xargs touch");
+ unlink("timestamp");
+ unlink("timestamp2");
sleep(2);
# Ensure that files we don't want to rebuild are newer than other files
safe_cd($ver);
diff --git a/include/mysql.h b/include/mysql.h
index af3e8c773ef..d581df91f64 100644
--- a/include/mysql.h
+++ b/include/mysql.h
@@ -100,7 +100,7 @@ typedef struct st_mysql_rows {
typedef MYSQL_ROWS *MYSQL_ROW_OFFSET; /* offset to current row */
-#include <my_alloc.h>
+#include "my_alloc.h"
typedef struct st_mysql_data {
my_ulonglong rows;
diff --git a/mysql-test/t/innodb_handler.test b/mysql-test/t/innodb_handler.test
index 2f53de3beb7..6b85be9c7ab 100644
--- a/mysql-test/t/innodb_handler.test
+++ b/mysql-test/t/innodb_handler.test
@@ -30,13 +30,15 @@ handler t2 read a next;
handler t2 read a=(15);
handler t2 read a=(16);
-!$1070 handler t2 read a=(19,"fff");
+--error 1070
+handler t2 read a=(19,"fff");
handler t2 read b=(19,"fff");
handler t2 read b=(19,"yyy");
handler t2 read b=(19);
-!$1109 handler t1 read a last;
+--error 1109
+handler t1 read a last;
handler t2 read a=(11);
handler t2 read a>=(11);
@@ -60,9 +62,14 @@ handler t2 read a=(19) where b="yyy";
handler t2 read first;
handler t2 read next;
+#
+# We alter the table even if it's still in use by to test the Innodb
+# delayed-drop code. This will generate a warning in the master.err log.
+#
alter table t1 type=innodb;
handler t2 read next;
-!$1064 handler t2 read last;
+--error 1064
+handler t2 read last;
handler t2 close;
drop table if exists t1;
diff --git a/sql/mini_client.cc b/sql/mini_client.cc
index 8a0c6963d4f..743d522e4bd 100644
--- a/sql/mini_client.cc
+++ b/sql/mini_client.cc
@@ -73,10 +73,6 @@ extern "C" { // Because of SCO 3.2V4.2
}
static void mc_free_rows(MYSQL_DATA *cur);
-static MYSQL_FIELD *unpack_fields(MYSQL_DATA *data,MEM_ROOT *alloc,uint fields,
- my_bool default_value,
- my_bool long_flag_protocol);
-
void mc_end_server(MYSQL *mysql);
static int mc_sock_connect(File s, const struct sockaddr *name, uint namelen, uint to);
static void mc_free_old_query(MYSQL *mysql);
diff --git a/sql/mysqld.cc b/sql/mysqld.cc
index aa6eeb33499..4ecded767d1 100644
--- a/sql/mysqld.cc
+++ b/sql/mysqld.cc
@@ -3619,8 +3619,9 @@ struct my_option my_long_options[] =
(gptr*) &max_system_variables.read_buff_size,0, GET_ULONG, REQUIRED_ARG,
128*1024L, IO_SIZE*2+MALLOC_OVERHEAD, ~0L, MALLOC_OVERHEAD, IO_SIZE, 0},
{"relay_log_space_limit", OPT_RELAY_LOG_SPACE_LIMIT,
- "Undocumented", (gptr*) &relay_log_space_limit,
- (gptr*) &relay_log_space_limit, 0, GET_ULONG, REQUIRED_ARG, 0L, 0L,
+ "Max space to use for all relay logs",
+ (gptr*) &relay_log_space_limit,
+ (gptr*) &relay_log_space_limit, 0, GET_ULL, REQUIRED_ARG, 0L, 0L,
(longlong) ULONG_MAX, 0, 1, 0},
{"slave_compressed_protocol", OPT_SLAVE_COMPRESSED_PROTOCOL,
"Use compression on master/slave protocol",
diff --git a/sql/slave.cc b/sql/slave.cc
index 7408907e49f..93a5c6171df 100644
--- a/sql/slave.cc
+++ b/sql/slave.cc
@@ -41,9 +41,7 @@ DYNAMIC_ARRAY replicate_wild_do_table, replicate_wild_ignore_table;
bool do_table_inited = 0, ignore_table_inited = 0;
bool wild_do_table_inited = 0, wild_ignore_table_inited = 0;
bool table_rules_on = 0;
-static TABLE* save_temporary_tables = 0;
-/* TODO: fix variables to access ulonglong values and make it ulonglong */
-ulong relay_log_space_limit = 0;
+ulonglong relay_log_space_limit = 0;
/*
When slave thread exits, we need to remember the temporary tables so we
@@ -53,17 +51,14 @@ ulong relay_log_space_limit = 0;
*/
int disconnect_slave_event_count = 0, abort_slave_event_count = 0;
-static int events_till_disconnect = -1;
int events_till_abort = -1;
-static int stuck_count = 0;
+static int events_till_disconnect = -1;
typedef enum { SLAVE_THD_IO, SLAVE_THD_SQL} SLAVE_THD_TYPE;
void skip_load_data_infile(NET* net);
static int process_io_rotate(MASTER_INFO* mi, Rotate_log_event* rev);
static int process_io_create_file(MASTER_INFO* mi, Create_file_log_event* cev);
-static int queue_old_event(MASTER_INFO* mi, const char* buf,
- uint event_len);
static bool wait_for_relay_log_space(RELAY_LOG_INFO* rli);
static inline bool io_slave_killed(THD* thd,MASTER_INFO* mi);
static inline bool sql_slave_killed(THD* thd,RELAY_LOG_INFO* rli);
@@ -288,7 +283,6 @@ err:
void init_slave_skip_errors(const char* arg)
{
const char *p;
- my_bool last_was_digit = 0;
if (bitmap_init(&slave_error_mask,MAX_SLAVE_ERROR,0))
{
fprintf(stderr, "Badly out of memory, please check your system status\n");
@@ -669,11 +663,14 @@ static void free_string_array(DYNAMIC_ARRAY *a)
delete_dynamic(a);
}
+#ifdef NOT_USED_YET
+
static int end_slave_on_walk(MASTER_INFO* mi, gptr /*unused*/)
{
end_master_info(mi);
return 0;
}
+#endif
void end_slave()
{
@@ -1818,7 +1815,6 @@ slave_begin:
MASTER_INFO* mi = (MASTER_INFO*)arg;
char llbuff[22];
uint retry_count= 0;
- ulonglong last_failed_pos = 0; // TODO: see if last_failed_pos is needed
DBUG_ASSERT(mi->inited);
pthread_mutex_lock(&mi->run_lock);
@@ -2079,9 +2075,6 @@ pthread_handler_decl(handle_slave_sql,arg)
slave_begin:
#endif
THD *thd; /* needs to be first for thread_stack */
- MYSQL *mysql = NULL ;
- bool retried_once = 0;
- ulonglong last_failed_pos = 0; // TODO: see if this can be removed
char llbuff[22],llbuff1[22];
RELAY_LOG_INFO* rli = &((MASTER_INFO*)arg)->rli;
const char* errmsg=0;
@@ -2700,7 +2693,7 @@ Log_event* next_event(RELAY_LOG_INFO* rli)
*/
pthread_mutex_lock(&rli->data_lock);
- while (!(was_killed=sql_slave_killed(thd,rli)))
+ while (!sql_slave_killed(thd,rli))
{
/*
We can have two kinds of log reading:
diff --git a/sql/slave.h b/sql/slave.h
index bc95f075dce..4be01785177 100644
--- a/sql/slave.h
+++ b/sql/slave.h
@@ -32,7 +32,7 @@ extern my_string master_info_file,relay_log_info_file;
extern my_string opt_relay_logname, opt_relaylog_index_name;
extern my_bool opt_skip_slave_start, opt_reckless_slave;
extern my_bool opt_log_slave_updates;
-extern ulong relay_log_space_limit;
+extern ulonglong relay_log_space_limit;
struct st_master_info;
/*
diff --git a/sql/sql_acl.cc b/sql/sql_acl.cc
index aad37477a70..5e32da07aad 100644
--- a/sql/sql_acl.cc
+++ b/sql/sql_acl.cc
@@ -501,7 +501,6 @@ ulong acl_getroot(THD *thd, const char *host, const char *ip, const char *user,
{
ulong user_access=NO_ACCESS;
*priv_user=(char*) user;
- char *ptr=0;
DBUG_ENTER("acl_getroot");
bzero(mqh,sizeof(USER_RESOURCES));
@@ -578,34 +577,29 @@ ulong acl_getroot(THD *thd, const char *host, const char *ip, const char *user,
if (acl_user->x509_issuer)
{
DBUG_PRINT("info",("checkpoint 3"));
- ptr = X509_NAME_oneline(X509_get_issuer_name(cert), 0, 0);
+ char *ptr = X509_NAME_oneline(X509_get_issuer_name(cert), 0, 0);
DBUG_PRINT("info",("comparing issuers: '%s' and '%s'",
acl_user->x509_issuer, ptr));
- if (!strcmp(acl_user->x509_issuer,ptr))
- user_access=acl_user->access;
- else
+ if (strcmp(acl_user->x509_issuer, ptr))
{
user_access=NO_ACCESS;
free(ptr);
break;
}
+ user_access=acl_user->access;
free(ptr);
}
DBUG_PRINT("info",("checkpoint 4"));
/* X509 subject is specified, we check it .. */
if (acl_user->x509_subject)
{
- ptr = X509_NAME_oneline(X509_get_subject_name(cert), 0, 0);
+ char *ptr= X509_NAME_oneline(X509_get_subject_name(cert), 0, 0);
DBUG_PRINT("info",("comparing subjects: '%s' and '%s'",
acl_user->x509_subject, ptr));
- if (!strcmp(acl_user->x509_subject,ptr))
- user_access=acl_user->access;
- else
- {
+ if (strcmp(acl_user->x509_subject,ptr))
user_access=NO_ACCESS;
- free(ptr);
- break;
- }
+ else
+ user_access=acl_user->access;
free(ptr);
}
break;
diff --git a/sql/sql_base.cc b/sql/sql_base.cc
index fddc2080764..307de1a319e 100644
--- a/sql/sql_base.cc
+++ b/sql/sql_base.cc
@@ -129,7 +129,6 @@ static void check_unused(void)
OPEN_TABLE_LIST *list_open_tables(THD *thd, const char *wild)
{
int result = 0;
- uint col_access=thd->col_access;
OPEN_TABLE_LIST **start_list, *open_list;
TABLE_LIST table_list;
char name[NAME_LEN*2];
diff --git a/sql/sql_cache.cc b/sql/sql_cache.cc
index c25a7e5afe4..1acdc3149ab 100644
--- a/sql/sql_cache.cc
+++ b/sql/sql_cache.cc
@@ -2042,7 +2042,6 @@ Query_cache::get_free_block(ulong len, my_bool not_less, ulong min)
if (bins[start].number != 0)
{
Query_cache_block *list = bins[start].free_blocks;
- ulong max_len = list->prev->length;
if (list->prev->length >= len) // check block with max size
{
first = list;
diff --git a/sql/sql_select.cc b/sql/sql_select.cc
index 6adf680b6f1..11b89722017 100644
--- a/sql/sql_select.cc
+++ b/sql/sql_select.cc
@@ -214,7 +214,6 @@ mysql_select(THD *thd,TABLE_LIST *tables,List<Item> &fields,COND *conds,
Procedure *procedure;
List<Item> all_fields(fields);
bool select_distinct;
- SELECT_LEX *select_lex = &(thd->lex.select_lex);
SELECT_LEX *cur_sel = thd->lex.select;
DBUG_ENTER("mysql_select");
@@ -4277,7 +4276,7 @@ do_select(JOIN *join,List<Item> *fields,TABLE *table,Procedure *procedure)
}
if (table)
{
- int old_error=error,tmp;
+ int tmp;
if ((tmp=table->file->extra(HA_EXTRA_NO_CACHE)))
{
my_errno=tmp;
diff --git a/vio/vio.c b/vio/vio.c
index 445aec3e3a7..bed380c6cd9 100644
--- a/vio/vio.c
+++ b/vio/vio.c
@@ -24,11 +24,10 @@
#define DONT_MAP_VIO
#include <my_global.h>
#include <mysql_com.h>
-#include <violite.h>
-#include <errno.h>
#include <my_sys.h>
-#include <my_net.h>
#include <m_string.h>
+#include <violite.h>
+#include <errno.h>
/*
* Helper to fill most of the Vio* with defaults.