diff options
| author | Vijay Bellur <vijay@gluster.com> | 2012-03-19 18:02:57 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-03-19 05:58:30 -0700 |
| commit | f076c23f381d9013d4fa2145c329033a51194597 (patch) | |
| tree | 5d411980bf8e22a7ae2194ca669e6e59591691c4 | |
| parent | b315c42344f808bd9e2cba06bade0e5da58b0381 (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.c | 6 |
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, |