From 7b15b020b573cd98da074a72deb2e57b5a8618e3 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Tue, 3 Mar 2026 23:12:15 +0000 Subject: util/nvmutil: use off_t for partsize (pread/pwrite) size_t can truncate on some platforms. it's best to use the proper variable type (a cast is insufficient). Signed-off-by: Leah Rowe --- util/nvmutil/nvmutil.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c index 204013c8..2dd3028f 100644 --- a/util/nvmutil/nvmutil.c +++ b/util/nvmutil/nvmutil.c @@ -65,7 +65,7 @@ static void set_err(int); static uint8_t buf[SIZE_8KB]; static uint16_t macbuf[3]; -static size_t partsize; +static off_t partsize; static int flags; static int rfd; @@ -250,7 +250,7 @@ read_gbe_part(int p, int invert) for (retry = 0; retry < (int) MAX_RETRY_READ; retry++) { rval = pread(fd, buf + (SIZE_4KB * (p ^ invert)), - SIZE_4KB, p * partsize); + SIZE_4KB, ((off_t ) p) * partsize); if (valid_read(fname, rval, (ssize_t) SIZE_4KB, retry, "pread")) { @@ -582,7 +582,7 @@ write_gbe_part(int p) swap(p); /* swap bytes on big-endian host CPUs */ if (pwrite(fd, buf + (SIZE_4KB * p), - SIZE_4KB, p * partsize) != (ssize_t) SIZE_4KB) { + SIZE_4KB, ((off_t) p) * partsize) != (ssize_t) SIZE_4KB) { err(ECANCELED, "Can't write %d b to '%s' p%d", SIZE_4KB, fname, p); } -- cgit v1.2.1