From 0272f7de41fa529f51bc69fdaf694c77b47250f2 Mon Sep 17 00:00:00 2001 From: Amar Tumballi Date: Fri, 3 Feb 2012 13:54:02 +0530 Subject: protocol/client: assign the right value to 'conf' before de-refing it variable assignment was done after it was actually getting de-referenced. moved the assignment few lines up. Change-Id: Id65e3e2d3dfe071e1c5b14c32488647070398ae4 BUG: 787117 Signed-off-by: Amar Tumballi Reviewed-on: http://review.gluster.com/2712 Tested-by: Gluster Build System Reviewed-by: Jeff Darcy --- xlators/protocol/client/src/client3_1-fops.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'xlators/protocol/client') diff --git a/xlators/protocol/client/src/client3_1-fops.c b/xlators/protocol/client/src/client3_1-fops.c index 3d39fc579..e28079ff7 100644 --- a/xlators/protocol/client/src/client3_1-fops.c +++ b/xlators/protocol/client/src/client3_1-fops.c @@ -4639,14 +4639,14 @@ client3_1_fremovexattr (call_frame_t *frame, xlator_t *this, if (!(args->fd && args->fd->inode)) goto unwind; + conf = this->private; + CLIENT_GET_REMOTE_FD(conf, args->fd, remote_fd, op_errno, unwind); memcpy (req.gfid, args->fd->inode->gfid, 16); req.name = (char *)args->name; req.fd = remote_fd; - conf = this->private; - ret = client_submit_request (this, &req, frame, conf->fops, GFS3_OP_FREMOVEXATTR, client3_1_fremovexattr_cbk, NULL, -- cgit