|
 |
ac737d |
From 63d66e7a3356da4bca77f521fd93df7cdf09b41a Mon Sep 17 00:00:00 2001
|
|
|
ac737d |
From: Ondrej Kozina <okozina@redhat.com>
|
|
|
ac737d |
Date: Tue, 19 Jun 2018 15:10:33 +0200
|
|
|
ac737d |
Subject: [PATCH 3/4] Fix write_blockwise on short files.
|
|
|
ac737d |
|
|
|
ac737d |
see unit test write_blockwise(length=2097153, bsize=4096), on x86
|
|
|
ac737d |
with original test file size=2097152.
|
|
|
ac737d |
|
|
|
ac737d |
The test is trying to write_blockwise 1 more byte than actual file
|
|
|
ac737d |
size.
|
|
|
ac737d |
---
|
|
|
ac737d |
lib/utils_io.c | 8 +++-----
|
|
|
ac737d |
1 file changed, 3 insertions(+), 5 deletions(-)
|
|
|
ac737d |
|
|
|
ac737d |
diff --git a/lib/utils_io.c b/lib/utils_io.c
|
|
|
ac737d |
index 8336b18..e0c2381 100644
|
|
|
ac737d |
--- a/lib/utils_io.c
|
|
|
ac737d |
+++ b/lib/utils_io.c
|
|
|
ac737d |
@@ -105,15 +105,13 @@ ssize_t write_blockwise(int fd, size_t bsize, size_t alignment,
|
|
|
ac737d |
if (hangover) {
|
|
|
ac737d |
if (posix_memalign(&hangover_buf, alignment, bsize))
|
|
|
ac737d |
goto out;
|
|
|
ac737d |
+ memset(hangover_buf, 0, bsize);
|
|
|
ac737d |
|
|
|
ac737d |
r = read_buffer(fd, hangover_buf, bsize);
|
|
|
ac737d |
- if (r < 0 || r < (ssize_t)hangover)
|
|
|
ac737d |
+ if (r < 0)
|
|
|
ac737d |
goto out;
|
|
|
ac737d |
|
|
|
ac737d |
- if (r < (ssize_t)bsize)
|
|
|
ac737d |
- bsize = r;
|
|
|
ac737d |
-
|
|
|
ac737d |
- if (lseek(fd, -(off_t)bsize, SEEK_CUR) < 0)
|
|
|
ac737d |
+ if (lseek(fd, -(off_t)r, SEEK_CUR) < 0)
|
|
|
ac737d |
goto out;
|
|
|
ac737d |
|
|
|
ac737d |
memcpy(hangover_buf, (char*)buf + solid, hangover);
|
|
|
ac737d |
--
|
|
|
ac737d |
1.8.3.1
|
|
|
ac737d |
|