diff options
| author | Raghavendra Bhat <raghavendrabhat@gluster.com> | 2010-09-14 09:27:14 +0000 |
|---|---|---|
| committer | Vijay Bellur <vijay@dev.gluster.com> | 2010-09-14 08:15:56 -0700 |
| commit | c55cc2d5362a607692beac96a34add2e79a9ad44 (patch) | |
| tree | c1f5afd5aac488c28b729063e27c59e5c9bca8aa /xlators/mgmt/glusterd/src/glusterd-op-sm.c | |
| parent | a9cbdd2916ab6851d108833e30536c02bf502f17 (diff) | |
compare op_errstr before freeing
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com>
Signed-off-by: Vijay Bellur <vijay@dev.gluster.com>
BUG: 1598 ()
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=1598
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-op-sm.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-op-sm.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c index 6ef471c4918..4d5fb05e162 100644 --- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c +++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c @@ -3341,7 +3341,7 @@ out: op_errno, req, ctx, op_errstr); if (ctx_free && ctx && (op != -1)) glusterd_op_free_ctx (op, ctx, ctx_free); - if (op_errstr) + if (op_errstr && (strcmp (op_errstr, ""))) GF_FREE (op_errstr); gf_log ("glusterd", GF_LOG_NORMAL, "Returning %d", ret); @@ -3399,7 +3399,7 @@ glusterd_op_ac_stage_op (glusterd_op_sm_event_t *event, void *ctx) ret = glusterd_op_stage_send_resp (stage_ctx->req, req->op, status, op_errstr); - if (op_errstr) + if (op_errstr && (strcmp (op_errstr, ""))) GF_FREE (op_errstr); gf_log ("", GF_LOG_DEBUG, "Returning with %d", ret); @@ -3430,7 +3430,7 @@ glusterd_op_ac_commit_op (glusterd_op_sm_event_t *event, void *ctx) ret = glusterd_op_commit_send_resp (commit_ctx->req, req->op, status, op_errstr); - if (op_errstr) + if (op_errstr && (strcmp (op_errstr, ""))) GF_FREE (op_errstr); gf_log ("", GF_LOG_DEBUG, "Returning with %d", ret); |