diff options
| author | Purna Pavan Chandra Aekkaladevi <paekkala@redhat.com> | 2019-09-12 19:50:35 +0530 |
|---|---|---|
| committer | Sunny Kumar <sunkumar@redhat.com> | 2019-10-11 08:45:43 +0000 |
| commit | 332bac01dd15aaad53ba144c80edca551fe58e11 (patch) | |
| tree | 8ddcb52b7a49c3b966f6429e17490a1fe797bfd7 /xlators/storage | |
| parent | 9dd57ff963d4d0dde7f3ad0ff230837ff2f4b1df (diff) | |
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 <paekkala@redhat.com>
Diffstat (limited to 'xlators/storage')
| -rw-r--r-- | xlators/storage/posix/src/posix-inode-fd-ops.c | 2 |
1 files changed, 0 insertions, 2 deletions
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" |