aba27b5983
Fix coverity issue 34533 and /dev/md symlink not created for second RAID container issue 50776 Resolves: RHEL-34533, RHEL50776 Signed-off-by: Xiao Ni <xni@redhat.com>
68 lines
1.6 KiB
Diff
68 lines
1.6 KiB
Diff
From eb9834599c8c9764bb3e711b6f291b10797eff27 Mon Sep 17 00:00:00 2001
|
|
From: Xiao Ni <xni@redhat.com>
|
|
Date: Fri, 26 Jul 2024 15:14:13 +0800
|
|
Subject: [PATCH 147/157] mdadm/super1: fix coverity issue CHECKED_RETURN
|
|
|
|
It needs to check return value when functions return value.
|
|
|
|
Signed-off-by: Xiao Ni <xni@redhat.com>
|
|
Signed-off-by: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
|
|
---
|
|
super1.c | 20 ++++++++++++++++----
|
|
1 file changed, 16 insertions(+), 4 deletions(-)
|
|
|
|
diff --git a/super1.c b/super1.c
|
|
index 81d29a65..4e4c7bfd 100644
|
|
--- a/super1.c
|
|
+++ b/super1.c
|
|
@@ -260,7 +260,10 @@ static int aread(struct align_fd *afd, void *buf, int len)
|
|
n = read(afd->fd, b, iosize);
|
|
if (n <= 0)
|
|
return n;
|
|
- lseek(afd->fd, len - n, 1);
|
|
+ if (lseek(afd->fd, len - n, 1) < 0) {
|
|
+ pr_err("lseek fails\n");
|
|
+ return -1;
|
|
+ }
|
|
if (n > len)
|
|
n = len;
|
|
memcpy(buf, b, n);
|
|
@@ -294,14 +297,20 @@ static int awrite(struct align_fd *afd, void *buf, int len)
|
|
n = read(afd->fd, b, iosize);
|
|
if (n <= 0)
|
|
return n;
|
|
- lseek(afd->fd, -n, 1);
|
|
+ if (lseek(afd->fd, -n, 1) < 0) {
|
|
+ pr_err("lseek fails\n");
|
|
+ return -1;
|
|
+ }
|
|
}
|
|
|
|
memcpy(b, buf, len);
|
|
n = write(afd->fd, b, iosize);
|
|
if (n <= 0)
|
|
return n;
|
|
- lseek(afd->fd, len - n, 1);
|
|
+ if (lseek(afd->fd, len - n, 1) < 0) {
|
|
+ pr_err("lseek fails\n");
|
|
+ return -1;
|
|
+ }
|
|
return len;
|
|
}
|
|
|
|
@@ -2667,7 +2676,10 @@ static int locate_bitmap1(struct supertype *st, int fd, int node_num)
|
|
}
|
|
if (mustfree)
|
|
free(sb);
|
|
- lseek64(fd, offset<<9, 0);
|
|
+ if (lseek64(fd, offset<<9, 0) < 0) {
|
|
+ pr_err("lseek fails\n");
|
|
+ ret = -1;
|
|
+ }
|
|
return ret;
|
|
}
|
|
|
|
--
|
|
2.41.0
|
|
|