mirror of
https://codeberg.org/libreboot/lbmk.git
synced 2026-07-25 07:46:17 +02:00
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>
This commit is contained in:
@@ -27,8 +27,8 @@ static void check_mac_separator(int);
|
|||||||
static void set_mac_nib(int, int, uint8_t *);
|
static void set_mac_nib(int, int, uint8_t *);
|
||||||
static uint8_t hextonum(char);
|
static uint8_t hextonum(char);
|
||||||
static uint8_t rhex(void);
|
static uint8_t rhex(void);
|
||||||
static void read_urandom(uint8_t *, ssize_t);
|
static void read_urandom(uint8_t *, size_t);
|
||||||
static int valid_read(const char *, ssize_t, ssize_t, int);
|
static int valid_read(const char *, ssize_t, size_t, int);
|
||||||
static int write_mac_part(int);
|
static int write_mac_part(int);
|
||||||
static void cmd_dump(void);
|
static void cmd_dump(void);
|
||||||
static void print_mac_address(int);
|
static void print_mac_address(int);
|
||||||
@@ -367,7 +367,7 @@ rhex(void)
|
|||||||
}
|
}
|
||||||
|
|
||||||
static void
|
static void
|
||||||
read_urandom(uint8_t *rnum, ssize_t rsize)
|
read_urandom(uint8_t *rnum, size_t rsize)
|
||||||
{
|
{
|
||||||
int retry = 0;
|
int retry = 0;
|
||||||
ssize_t rval;
|
ssize_t rval;
|
||||||
@@ -380,7 +380,7 @@ read_urandom(uint8_t *rnum, ssize_t rsize)
|
|||||||
}
|
}
|
||||||
|
|
||||||
static int
|
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)
|
if (rval == (ssize_t) rsize)
|
||||||
return 1;
|
return 1;
|
||||||
|
|||||||
Reference in New Issue
Block a user