diff options
author | Atin Mukherjee <amukherj@redhat.com> | 2019-06-25 11:11:10 +0530 |
---|---|---|
committer | Rinku Kothiya <rkothiya@redhat.com> | 2019-07-20 07:40:12 +0000 |
commit | 4ad6dd3ca8cd558b853c7df0d6cd34dd22dff7d3 (patch) | |
tree | 9f966f9a0238d1f9a51444c2ba88678fed878f05 | |
parent | 908d3bc0b34489c506fafa0f796f6d710768d934 (diff) |
glusterd: conditionally clear txn_opinfo in stage op
...otherwise this leads to a crash when volume status is run on a
heterogeneous mode.
> Fixes: bz#1723658
> Change-Id: I0d39f412b2e5e9d3ef0a3462b90b38bb5364b09d
> Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
> (cherry picked from commit a72452fcf90679b28baec12d2769cbaa982bb4e4)
Fixes: bz#1728127
Change-Id: I0d39f412b2e5e9d3ef0a3462b90b38bb5364b09d
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
-rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-op-sm.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c index e9eefdfd859..9ab8930be26 100644 --- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c +++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c @@ -5469,9 +5469,14 @@ glusterd_op_ac_stage_op(glusterd_op_sm_event_t *event, void *ctx) glusterd_op_info_t txn_op_info = { {0}, }; + glusterd_conf_t *priv = NULL; this = THIS; GF_ASSERT(this); + + priv = this->private; + GF_ASSERT(priv); + GF_ASSERT(ctx); req_ctx = ctx; @@ -5523,9 +5528,12 @@ out: gf_msg_debug(this->name, 0, "Returning with %d", ret); /* for no volname transactions, the txn_opinfo needs to be cleaned up - * as there's no unlock event triggered + * as there's no unlock event triggered. However if the originator node of + * this transaction is still running with a version lower than 60000, + * txn_opinfo can't be cleared as that'll lead to a race of referring op_ctx + * after it's being freed. */ - if (txn_op_info.skip_locking) + if (txn_op_info.skip_locking && priv->op_version >= GD_OP_VERSION_6_0) ret = glusterd_clear_txn_opinfo(txn_id); if (rsp_dict) |