diff --git a/maxscale-system-test/CMakeLists.txt b/maxscale-system-test/CMakeLists.txt index 3440f5f70..b30908f12 100644 --- a/maxscale-system-test/CMakeLists.txt +++ b/maxscale-system-test/CMakeLists.txt @@ -946,6 +946,9 @@ add_test_executable(mxs2115_version_string.cpp mxs2115_version_string replicatio # MXS-2295: COM_CHANGE_USER does not clear out session command history add_test_executable(mxs2295_change_user_loop.cpp mxs2295_change_user_loop mxs2295_change_user_loop LABELS REPL_BACKEND) +# Debug assertion due to double-closed when a slave's response differs from the master +add_test_executable(crash_on_bad_sescmd.cpp crash_on_bad_sescmd crash_on_bad_sescmd LABELS readwritesplit REPL_BACKEND) + # MXS-2300: Prune session command history add_test_executable(mxs2300_history_pruning.cpp mxs2300_history_pruning mxs2300_history_pruning LABELS readwritesplit REPL_BACKEND) diff --git a/maxscale-system-test/cnf/maxscale.cnf.template.crash_on_bad_sescmd b/maxscale-system-test/cnf/maxscale.cnf.template.crash_on_bad_sescmd new file mode 100644 index 000000000..458d371db --- /dev/null +++ b/maxscale-system-test/cnf/maxscale.cnf.template.crash_on_bad_sescmd @@ -0,0 +1,49 @@ +[maxscale] +threads=auto + +[server1] +type=server +address=###node_server_IP_1### +port=###node_server_port_1### +protocol=MySQLBackend + +[server2] +type=server +address=###node_server_IP_2### +port=###node_server_port_2### +protocol=MySQLBackend + +[server3] +type=server +address=###node_server_IP_3### +port=###node_server_port_3### +protocol=MySQLBackend + +[server4] +type=server +address=###node_server_IP_4### +port=###node_server_port_4### +protocol=MySQLBackend + +[MySQL Monitor] +type=monitor +module=mysqlmon +servers=server1,server2,server3,server4 +user=maxskysql +password=skysql +monitor_interval=1000 + +[RW Split Router] +type=service +router=readwritesplit +servers=server1,server2,server3,server4 +user=maxskysql +password=skysql +max_sescmd_history=20 +disable_sescmd_history=false + +[RW Split Listener] +type=listener +service=RW Split Router +protocol=MySQLClient +port=4006 diff --git a/maxscale-system-test/crash_on_bad_sescmd.cpp b/maxscale-system-test/crash_on_bad_sescmd.cpp new file mode 100644 index 000000000..4ee88315d --- /dev/null +++ b/maxscale-system-test/crash_on_bad_sescmd.cpp @@ -0,0 +1,69 @@ +/** + * Double-close on bad session command result + */ + +#include "testconnections.h" + +void run_test(TestConnections& test) +{ + Connection conn = test.maxscales->rwsplit(); + conn.connect(); + + for (int i = 0; i <= 300 && test.global_result == 0; i++) + { + if (conn.query("SET @a = 1") + && conn.query("USE test") + && conn.query("SET SQL_MODE=''") + && conn.query("USE test") + && conn.query("SELECT @@last_insert_id") + && conn.query("SELECT 1") + && conn.query("USE test") + && conn.query("SELECT 1") + && conn.query("SET @a = 123") + && conn.query("BEGIN") + && conn.query("SELECT @a") + && conn.query("COMMIT") + && conn.query("SET @a = 321") + && conn.query("SELECT @a") + && conn.query("SET @a = 456") + && conn.query("START TRANSACTION READ ONLY") + && conn.query("SELECT @a") + && conn.query("COMMIT") + && conn.query("PREPARE ps FROM 'SELECT 1'") + && conn.query("EXECUTE ps") + && conn.query("DEALLOCATE PREPARE ps")) + { + conn.reset_connection(); + } + else + { + break; + } + } +} + +int main(int argc, char* argv[]) +{ + TestConnections test(argc, argv); + + std::vector threads; + + for (int i = 0; i < 5; i++) + { + threads.emplace_back(run_test, std::ref(test)); + } + + for (int i = 0; i < 5; i++) + { + test.repl->stop_node(1 + i % 3); + test.repl->start_node(1 + i % 3); + sleep(1); + } + + for (auto& a : threads) + { + a.join(); + } + + return test.global_result; +} diff --git a/server/modules/routing/readwritesplit/rwsplit_session_cmd.cc b/server/modules/routing/readwritesplit/rwsplit_session_cmd.cc index fce3ad819..7726b9985 100644 --- a/server/modules/routing/readwritesplit/rwsplit_session_cmd.cc +++ b/server/modules/routing/readwritesplit/rwsplit_session_cmd.cc @@ -65,7 +65,7 @@ static void discard_if_response_differs(SRWBackend backend, uint8_t slave_response, SSessionCommand sescmd) { - if (master_response != slave_response) + if (master_response != slave_response && backend->in_use()) { uint8_t cmd = sescmd->get_command(); std::string query = sescmd->to_string();