summaryrefslogtreecommitdiff
path: root/util/nvmutil
diff options
context:
space:
mode:
authorLeah Rowe <leah@libreboot.org>2023-01-28 23:01:41 +0000
committerLeah Rowe <leah@libreboot.org>2023-01-28 23:21:53 +0000
commit6b4a14ce4ac7557bacf607b80cb61d45cb5e8815 (patch)
treeff784b7a7030a1858f86abe0f03d686cd25a033f /util/nvmutil
parent031a0b553ba15e495ce1696ea19a606b239909e4 (diff)
util/nvmutil: tidy up variable declarations
Diffstat (limited to 'util/nvmutil')
-rw-r--r--util/nvmutil/nvmutil.c54
1 files changed, 31 insertions, 23 deletions
diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c
index 687659a5..1a873b2a 100644
--- a/util/nvmutil/nvmutil.c
+++ b/util/nvmutil/nvmutil.c
@@ -58,7 +58,7 @@ void writeGbeFile(int *fd, const char *filename);
#define SIZE_8KB 0x2000
uint8_t buf[SIZE_8KB];
-size_t gbe[2] = {(size_t) buf, ((size_t) buf) + SIZE_4KB};
+size_t gbe[2];
uint8_t skipread[2] = {0, 0};
int part, gbeWriteAttempted = 0, gbeFileModified = 0;
@@ -70,16 +70,18 @@ uint8_t little_endian;
int
main(int argc, char *argv[])
{
- size_t nr = 128;
+ size_t nr;
int fd, flags = O_RDWR;
- char *strMac = NULL, *strRMac = "??:??:??:??:??:??";
void (*cmd)(void) = NULL;
+ const char *strMac = NULL, *strRMac = "??:??:??:??:??:??";
#ifdef HAVE_PLEDGE
if (pledge("stdio wpath", NULL) == -1)
err(errno, "pledge");
#endif
+ gbe[1] = (gbe[0] = (size_t) buf) + SIZE_4KB;
+
test = 1;
little_endian = ((uint8_t *) &test)[0];
@@ -92,7 +94,7 @@ main(int argc, char *argv[])
flags = O_RDONLY;
cmd = &cmd_dump;
} else if (strcmp(COMMAND, "setmac") == 0)
- strMac = strRMac;
+ strMac = (char *) strRMac;
else if (strcmp(COMMAND, "swap") == 0)
cmd = &cmd_swap;
} else if (argc == 4) {
@@ -116,6 +118,9 @@ main(int argc, char *argv[])
if ((cmd == &cmd_copy) || (cmd == &cmd_swap))
nr = SIZE_4KB;
+ else
+ nr = 128;
+
if ((cmd == &cmd_copy) || (cmd == &cmd_setchecksum) ||
(cmd == &cmd_brick))
skipread[part ^ 1] = 1;
@@ -146,7 +151,7 @@ void
readGbeFile(int *fd, const char *path, int flags, size_t nr)
{
struct stat st;
- int p, r, tr = 0;
+ int p, r, tr;
if (opendir(path) != NULL)
err(errno = EISDIR, "%s", path);
@@ -164,7 +169,7 @@ readGbeFile(int *fd, const char *path, int flags, size_t nr)
if (errno != 0)
err(errno, "%s", path);
- for (p = 0; p < 2; p++) {
+ for (tr = 0, p = 0; p < 2; p++) {
if (skipread[p])
continue;
if ((r = pread((*fd), (uint8_t *) gbe[p], nr, p << 12)) == -1)
@@ -177,11 +182,10 @@ readGbeFile(int *fd, const char *path, int flags, size_t nr)
void
cmd_setmac(const char *strMac)
{
- uint8_t o, val8;
- uint16_t val16;
int partnum, byte, nib;
- uint16_t mac[3] = {0, 0, 0};
- uint64_t total = 0;
+ uint8_t o, val8;
+ uint16_t val16, mac[3] = {0, 0, 0};
+ uint64_t total;
if (strnlen(strMac, 20) != 17)
goto invalid_mac_address;
@@ -191,7 +195,7 @@ cmd_setmac(const char *strMac)
if (strMac[o + 2] != ':')
goto invalid_mac_address;
byte = o / 3;
- for (nib = 0; nib < 2; nib++, total += val8) {
+ for (total = 0, nib = 0; nib < 2; nib++, total += val8) {
if ((val8 = hextonum(strMac[o + nib])) > 15)
goto invalid_mac_address;
if ((byte == 0) && (nib == 1)) {
@@ -338,9 +342,9 @@ void
cmd_setchecksum(void)
{
int c;
- uint16_t val16 = 0;
+ uint16_t val16;
- for (c = 0; c < 0x3F; c++)
+ for (val16 = 0, c = 0; c < 0x3F; c++)
val16 += word(c, part);
setWord(0x3F, part, 0xBABA - val16);
@@ -384,9 +388,9 @@ int
validChecksum(int partnum)
{
int w;
- uint16_t total = 0;
+ uint16_t total;
- for(w = 0; w <= 0x3F; w++)
+ for(total = 0, w = 0; w <= 0x3F; w++)
total += word(w, partnum);
if (total == 0xBABA)
@@ -400,12 +404,16 @@ validChecksum(int partnum)
uint16_t
word(int pos16, int partnum)
{
- uint8_t *nbuf = (uint8_t *) gbe[partnum];
- uint16_t pos8 = pos16 << 1;
- uint16_t val16 = nbuf[pos8 + 1];
+ uint8_t *nbuf;
+ uint16_t pos8, val16;
+ nbuf = (uint8_t *) gbe[partnum];
+ pos8 = pos16 << 1;
+ val16 = nbuf[pos8 + 1];
val16 <<= 8;
- return val16 |= nbuf[pos8];
+ val16 |= nbuf[pos8];
+
+ return val16;
}
void
@@ -441,16 +449,16 @@ byteswap(uint8_t *byte)
void
writeGbeFile(int *fd, const char *filename)
{
- int p;
- int tw = 0;
- int nw = SIZE_4KB;
+ int p, nw, tw;
errno = 0;
/* if copy/swap not performed, write only the nvm part */
if ((gbe[0] != gbe[1]) && (gbe[0] < gbe[1]))
nw = 128;
+ else
+ nw = SIZE_4KB;
- for (p = 0; p < 2; p++) {
+ for (tw = 0, p = 0; p < 2; p++) {
if (gbe[0] > gbe[1])
p ^= 1;
if (nvmPartModified[p]) {