diff options
| author | Sergey Vojtovich <svoj@sun.com> | 2009-10-28 19:39:08 +0400 |
|---|---|---|
| committer | Sergey Vojtovich <svoj@sun.com> | 2009-10-28 19:39:08 +0400 |
| commit | 17ed7089756f9001d7bc6eac063d91b374de0181 (patch) | |
| tree | e463afef840ef0b005614eae4b2fd868d2e051ff /sql/handler.cc | |
| parent | 9a5a77eb68a35b3e806bb65f94ec9cd9ab9b1b5e (diff) | |
| download | mariadb-git-17ed7089756f9001d7bc6eac063d91b374de0181.tar.gz | |
BUG#43171 - Assertion failed: thd->transaction.xid_state.xid.is_null()
XA START may cause assertion failure/server crash when it is called
after unilateral roll back issued by the Resource Manager (both
in regular transaction and after XA transaction).
The problem was that rm_error variable wasn't set/reset properly.
mysql-test/r/xa.result:
A test case for BUG#43171.
mysql-test/t/xa.test:
A test case for BUG#43171.
sql/handler.cc:
Setting rm_error when we're out of XA transaction has no
special meaning. But it blocks reset of thd->transaction.xid
structure later.
sql/sql_parse.cc:
Reset rm_error before we enter ha_rollback(), so
thd->transaction.xid strucure is reinitialized.
Diffstat (limited to 'sql/handler.cc')
| -rw-r--r-- | sql/handler.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sql/handler.cc b/sql/handler.cc index d07ebed8ab9..216228ed509 100644 --- a/sql/handler.cc +++ b/sql/handler.cc @@ -1311,7 +1311,8 @@ int ha_rollback_trans(THD *thd, bool all) } trans->ha_list= 0; trans->no_2pc=0; - if (is_real_trans && thd->transaction_rollback_request) + if (is_real_trans && thd->transaction_rollback_request && + thd->transaction.xid_state.xa_state != XA_NOTR) thd->transaction.xid_state.rm_error= thd->main_da.sql_errno(); if (all) thd->variables.tx_isolation=thd->session_tx_isolation; |