From 332bac01dd15aaad53ba144c80edca551fe58e11 Mon Sep 17 00:00:00 2001 From: Purna Pavan Chandra Aekkaladevi Date: Thu, 12 Sep 2019 19:50:35 +0530 Subject: Posix: UNUSED VALUE coverity fix This patch fixes Coverity issue with CID 1274206 Problem : -1 is assigned to op_ret, but that stored value is overwritten before it can be used. Fix : Removal of the line that assigns -1 to op_ret which has no significance. Change-Id: Icb881549ac946003710551c9b9e88b33b6a06239 Updates: bz#789278 Signed-off-by: Purna Pavan Chandra Aekkaladevi --- xlators/storage/posix/src/posix-inode-fd-ops.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'xlators') diff --git a/xlators/storage/posix/src/posix-inode-fd-ops.c b/xlators/storage/posix/src/posix-inode-fd-ops.c index 396bd883383..56f8d344640 100644 --- a/xlators/storage/posix/src/posix-inode-fd-ops.c +++ b/xlators/storage/posix/src/posix-inode-fd-ops.c @@ -4083,7 +4083,6 @@ posix_fgetxattr(call_frame_t *frame, xlator_t *this, fd_t *fd, const char *name, size = sys_fgetxattr(_fd, key, NULL, 0); } if (size == -1) { - op_ret = -1; op_errno = errno; gf_msg(this->name, GF_LOG_ERROR, errno, P_MSG_XATTR_FAILED, "fgetxattr failed " @@ -4104,7 +4103,6 @@ posix_fgetxattr(call_frame_t *frame, xlator_t *this, fd_t *fd, const char *name, bzero(value, size + 1); size = sys_fgetxattr(_fd, key, value, size); if (size == -1) { - op_ret = -1; op_errno = errno; gf_msg(this->name, GF_LOG_ERROR, errno, P_MSG_XATTR_FAILED, "fgetxattr failed o" -- cgit