diff options
| author | Amar Tumballi <amar@gluster.com> | 2012-02-14 15:27:49 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-02-14 04:34:06 -0800 |
| commit | 5f002b5418b557fd03ce59fbfd5d41272bdf491a (patch) | |
| tree | 16d04d59e28a4c02dc9b37b5ee38ae80da3c190d /xlators/protocol/client/src/client-handshake.c | |
| parent | 69a7f2fea23880fbdc776b657dd6aa32c643c6c4 (diff) | |
protocol xdr: remove 'path<>'
client was sending 'path' on wire, which gets ignored on server
side, and also doesn't get freed up, which causes memory leak.
also with not having path on wire, the xdr size on wire most of the
time can remain constant, which helps in allocating RDMA buffers.
Change-Id: Ie0d36a670be60b02fd1e925c6f977b1a71def5cd
BUG: 790298
Signed-off-by: Amar Tumballi <amar@gluster.com>
Reviewed-on: http://review.gluster.com/2744
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'xlators/protocol/client/src/client-handshake.c')
| -rw-r--r-- | xlators/protocol/client/src/client-handshake.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/xlators/protocol/client/src/client-handshake.c b/xlators/protocol/client/src/client-handshake.c index be114f74050..91cda6d0c45 100644 --- a/xlators/protocol/client/src/client-handshake.c +++ b/xlators/protocol/client/src/client-handshake.c @@ -623,7 +623,6 @@ protocol_client_reopendir (xlator_t *this, clnt_fd_ctx_t *fdctx) } memcpy (req.gfid, inode->gfid, 16); - req.path = (char *)local->loc.path; gf_log (frame->this->name, GF_LOG_DEBUG, "attempting reopen on %s", local->loc.path); @@ -707,7 +706,6 @@ protocol_client_reopen (xlator_t *this, clnt_fd_ctx_t *fdctx) memcpy (req.gfid, inode->gfid, 16); req.flags = gf_flags_from_flags (fdctx->flags); req.wbflags = fdctx->wbflags; - req.path = (char *)local->loc.path; gf_log (frame->this->name, GF_LOG_DEBUG, "attempting reopen on %s", local->loc.path); |