Commit 61d8b4a2 authored by Nirbhay Choubey's avatar Nirbhay Choubey

MDEV-6833: SIGSEGV on shutdown with non-default wsrep_slave_threads

thd->variables' table_plugin & tmp_table_plugin should be set to
NULL for wsrep system threads.
Also made a minor change to skip checking of wsrep options if
wsrep_on is not set.
parent c768af75
...@@ -4981,8 +4981,10 @@ a file name for --log-bin-index option", opt_binlog_index_name); ...@@ -4981,8 +4981,10 @@ a file name for --log-bin-index option", opt_binlog_index_name);
} }
plugins_are_initialized= TRUE; /* Don't separate from init function */ plugins_are_initialized= TRUE; /* Don't separate from init function */
if (wsrep_check_opts()) #ifdef WITH_WSREP
unireg_abort(1); if (WSREP_ON && wsrep_check_opts())
global_system_variables.wsrep_on= 0;
#endif
/* we do want to exit if there are any other unknown options */ /* we do want to exit if there are any other unknown options */
if (remaining_argc > 1) if (remaining_argc > 1)
......
...@@ -3074,12 +3074,12 @@ void plugin_thdvar_init(THD *thd) ...@@ -3074,12 +3074,12 @@ void plugin_thdvar_init(THD *thd)
plugin_ref old_table_plugin= thd->variables.table_plugin; plugin_ref old_table_plugin= thd->variables.table_plugin;
plugin_ref old_tmp_table_plugin= thd->variables.tmp_table_plugin; plugin_ref old_tmp_table_plugin= thd->variables.tmp_table_plugin;
DBUG_ENTER("plugin_thdvar_init"); DBUG_ENTER("plugin_thdvar_init");
// This function may be called many times per THD (e.g. on COM_CHANGE_USER) // This function may be called many times per THD (e.g. on COM_CHANGE_USER)
thd->variables.table_plugin= NULL; thd->variables.table_plugin= NULL;
thd->variables.tmp_table_plugin= NULL; thd->variables.tmp_table_plugin= NULL;
cleanup_variables(thd, &thd->variables); cleanup_variables(thd, &thd->variables);
thd->variables= global_system_variables; thd->variables= global_system_variables;
/* we are going to allocate these lazily */ /* we are going to allocate these lazily */
...@@ -3098,6 +3098,9 @@ void plugin_thdvar_init(THD *thd) ...@@ -3098,6 +3098,9 @@ void plugin_thdvar_init(THD *thd)
intern_plugin_unlock(NULL, old_table_plugin); intern_plugin_unlock(NULL, old_table_plugin);
intern_plugin_unlock(NULL, old_tmp_table_plugin); intern_plugin_unlock(NULL, old_tmp_table_plugin);
mysql_mutex_unlock(&LOCK_plugin); mysql_mutex_unlock(&LOCK_plugin);
} else {
thd->variables.table_plugin= NULL;
thd->variables.tmp_table_plugin= NULL;
} }
DBUG_VOID_RETURN; DBUG_VOID_RETURN;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment