summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVijay Bellur <vijay@gluster.com>2012-03-19 18:02:57 +0530
committerVijay Bellur <vijay@gluster.com>2012-03-19 05:58:30 -0700
commitf076c23f381d9013d4fa2145c329033a51194597 (patch)
tree5d411980bf8e22a7ae2194ca669e6e59591691c4
parentb315c42344f808bd9e2cba06bade0e5da58b0381 (diff)
protocol/client: Avoid STACK_DESTROYing more than once in RELEASE fops.
Change-Id: I435327c6133aa6739731dabddde860b0b43c1497 BUG: 804607 Signed-off-by: Vijay Bellur <vijay@gluster.com> Reviewed-on: http://review.gluster.com/2978 Reviewed-by: Amar Tumballi <amarts@redhat.com> Tested-by: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r--xlators/protocol/client/src/client3_1-fops.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/xlators/protocol/client/src/client3_1-fops.c b/xlators/protocol/client/src/client3_1-fops.c
index c86cc82c1b5..3322f561ce6 100644
--- a/xlators/protocol/client/src/client3_1-fops.c
+++ b/xlators/protocol/client/src/client3_1-fops.c
@@ -2281,10 +2281,12 @@ client_fdctx_destroy (xlator_t *this, clnt_fd_ctx_t *fdctx)
goto out;
}
+ ret = 0;
+
if (fdctx->is_dir) {
gfs3_releasedir_req req = {{0,},};
req.fd = fdctx->remote_fd;
- gf_log (this->name, GF_LOG_INFO, "sending releasedir on fd");
+ gf_log (this->name, GF_LOG_DEBUG, "sending releasedir on fd");
client_submit_request (this, &req, fr, &clnt3_1_fop_prog,
GFS3_OP_RELEASEDIR,
client3_1_releasedir_cbk,
@@ -2293,7 +2295,7 @@ client_fdctx_destroy (xlator_t *this, clnt_fd_ctx_t *fdctx)
} else {
gfs3_release_req req = {{0,},};
req.fd = fdctx->remote_fd;
- gf_log (this->name, GF_LOG_INFO, "sending release on fd");
+ gf_log (this->name, GF_LOG_DEBUG, "sending release on fd");
client_submit_request (this, &req, fr, &clnt3_1_fop_prog,
GFS3_OP_RELEASE,
client3_1_release_cbk, NULL,