diff options
| author | Leah Rowe <leah@libreboot.org> | 2026-03-03 22:55:00 +0000 |
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2026-03-03 22:57:18 +0000 |
| commit | 3268c225d65f207d713a8dd02ae904ed0c0dc173 (patch) | |
| tree | 02ba423877104fa751446e4954a5fc51a4249159 /util | |
| parent | 3077f51c67fcafb749a199807568c62561a0ab39 (diff) | |
util/nvmutil: pass size_t to read, not ssize_t
ssize_t is signed, which could be negative.
in practise, we control the value that's going in anyway.
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'util')
| -rw-r--r-- | util/nvmutil/nvmutil.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c index cfd03e7b..2d1e9fa1 100644 --- a/util/nvmutil/nvmutil.c +++ b/util/nvmutil/nvmutil.c @@ -27,8 +27,8 @@ static void check_mac_separator(int); static void set_mac_nib(int, int, uint8_t *); static uint8_t hextonum(char); static uint8_t rhex(void); -static void read_urandom(uint8_t *, ssize_t); -static int valid_read(const char *, ssize_t, ssize_t, int); +static void read_urandom(uint8_t *, size_t); +static int valid_read(const char *, ssize_t, size_t, int); static int write_mac_part(int); static void cmd_dump(void); static void print_mac_address(int); @@ -367,7 +367,7 @@ rhex(void) } static void -read_urandom(uint8_t *rnum, ssize_t rsize) +read_urandom(uint8_t *rnum, size_t rsize) { int retry = 0; ssize_t rval; @@ -380,7 +380,7 @@ read_urandom(uint8_t *rnum, ssize_t rsize) } static int -valid_read(const char *rpath, ssize_t rval, ssize_t rsize, int retry) +valid_read(const char *rpath, ssize_t rval, size_t rsize, int retry) { if (rval == (ssize_t) rsize) return 1; |