diff options
author | Barak Sason Rofman <bsasonro@redhat.com> | 2021-01-22 14:19:31 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-22 17:49:31 +0530 |
commit | de19b9a00ea35cf01076c9eb48067ad8a2e1c4aa (patch) | |
tree | 83c48198c17e360836e23d7ee1b5d56585aadaf3 /xlators | |
parent | 8d1bb816ad50da53882daa443ba8b724ac5babdf (diff) | |
download | glusterfs-de19b9a00ea35cf01076c9eb48067ad8a2e1c4aa.tar.gz glusterfs-de19b9a00ea35cf01076c9eb48067ad8a2e1c4aa.tar.xz glusterfs-de19b9a00ea35cf01076c9eb48067ad8a2e1c4aa.zip |
AFR - fixing coverity issue (Argument cannot be negative) (#2026)
CID 1430124
A negative value is being passed to a parameter hat cannot be negative.
Modified the value which is being passed.
Change-Id: I06dca105f7a78ae16145b0876910851fb631e366
Signed-off-by: Barak Sason Rofman <bsasonro@redhat.com>
Diffstat (limited to 'xlators')
-rw-r--r-- | xlators/cluster/afr/src/afr-inode-read.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/cluster/afr/src/afr-inode-read.c b/xlators/cluster/afr/src/afr-inode-read.c index c5521704de..5baf9c42ee 100644 --- a/xlators/cluster/afr/src/afr-inode-read.c +++ b/xlators/cluster/afr/src/afr-inode-read.c @@ -1581,7 +1581,6 @@ afr_getxattr(call_frame_t *frame, xlator_t *this, loc_t *loc, const char *name, * Heal daemons don't have IO threads ... and as a result they * send this getxattr down and eventually crash :( */ - op_errno = -1; GF_CHECK_XATTR_KEY_AND_GOTO(name, IO_THREADS_QUEUE_SIZE_KEY, op_errno, out); /* @@ -1609,6 +1608,7 @@ no_name: out: if (ret < 0) AFR_STACK_UNWIND(getxattr, frame, -1, op_errno, NULL, NULL); + return 0; } |