[PATCH V2 2/2] restripe: fix ignoring return value of ‘read’
From: Guoqing Jiang <hidden>
Date: 2020-05-18 19:31:08
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Guoqing Jiang <hidden>
Date: 2020-05-18 19:31:08
Subsystem:
the rest · Maintainer:
Linus Torvalds
Got below error when run "make everything".
restripe.c: In function ‘test_stripes’:
restripe.c:870:4: error: ignoring return value of ‘read’, declared with attribute warn_unused_result [-Werror=unused-result]
read(source[i], stripes[i], chunk_size);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Fix it by check the return value of ‘read’, and free memory
in the failure case.
Signed-off-by: Guoqing Jiang <redacted>
---
restripe.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/restripe.c b/restripe.c
index 31b07e8..48c6506 100644
--- a/restripe.c
+++ b/restripe.c@@ -867,7 +867,15 @@ int test_stripes(int *source, unsigned long long *offsets, for (i = 0 ; i < raid_disks ; i++) { lseek64(source[i], offsets[i]+start, 0); - read(source[i], stripes[i], chunk_size); + if (read(source[i], stripes[i], chunk_size) != + chunk_size) { + free(q); + free(p); + free(blocks); + free(stripes); + free(stripe_buf); + return -1; + } } for (i = 0 ; i < data_disks ; i++) { int disk = geo_map(i, start/chunk_size, raid_disks,
--
2.17.1