summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
11 hoursutil/nvmutil: fix a commentLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
11 hoursutil/nvmutil: unambiguous sign conv_argv_part_numLeah Rowe
yeah, do the verification manually, don't convert to size_t. this avoids a bunch of theoretical bugs that i can't be bothered to explain at 3AM just trust me bro Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: clean up hextonum()Leah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: tidy up some commentsLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: make mac_updated a uint8_tLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: fix prototype indentationLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: reset mac_buf for mac parsingLeah Rowe
preventative fix for later, if the tool is ever expanded to have a better command syntax, for supporting more than one file at a time. Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: comment the unhandled errno exitLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: tidy up set_mac_byteLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
12 hoursutil/nvmutil: further mac address parsing cleanupLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: tidy up mac parsing codeLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: fix a bad commentLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: tidy up set_mac_byteLeah Rowe
send the mac address byte directly to check_mac_separator functionally identicaly, but cleaner, and uses multiplication instead of division (faster). Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: make do_read a uint8_tLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: clearer commenting about commandsLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: comment regarding bufLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
13 hoursutil/nvmutil: minor code cleanupLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: fix an err stringLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: tidy up conv_argv_part_numLeah Rowe
no need for strlen here (1-character string). Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: Shorten a commentLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: fix a bad commentLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: make gbe_st local, not static/globalLeah Rowe
forgot to remove static when making it local Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: rename rfd to urandom_fdLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: remove reset_global_stateLeah Rowe
it was put there for another change i haven't done yet, and probably won't. the program currently just runs once with one operation, on a given file. the current defaults are initialised just fine without this function, so let's remove this bloat for now. Signed-off-by: Leah Rowe <leah@libreboot.org>
14 hoursutil/nvmutil: rename arc4random defineLeah Rowe
specifically refer to the actual function i use. Signed-off-by: Leah Rowe <leah@libreboot.org>
15 hoursutil/nvmutil: reset errno on /dev/random fallbackLeah Rowe
i didn't catch this before. stale error state would have passed through, even on ultimate success. Signed-off-by: Leah Rowe <leah@libreboot.org>
15 hoursutil/nvmutil: remove rval from conv_argv_part_numLeah Rowe
relic from an earlier uncommitted version Signed-off-by: Leah Rowe <leah@libreboot.org>
15 hoursutil/nvmutil: set part_modified to a uint8_tLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
15 hoursutil/nvmutil: make invert a uint8_tLeah Rowe
cleaner intention, and shorter Signed-off-by: Leah Rowe <leah@libreboot.org>
15 hoursutil/nvmutil: safer argv part number parsingLeah Rowe
we now handle signedness properly, which is implementation defined, on char integers where signed/unsigned is not specified. Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: rename check_bound check_nvm_boundLeah Rowe
this makes the intent clearer, because we only want to access the 128-byte nvm area here. Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: be clearer about GbE file sizesLeah Rowe
the partsize variable is also misleading, because it refers to the full half of a file, which might be bigger than 4KB. this variable name is a hangover from when nvmutil only supported 8KB files. Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: rename fd to fd_ptr in xopen()Leah Rowe
clearer intent Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: cleaned up a few prototypesLeah Rowe
make them match how they're declared Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: more readable xopenLeah Rowe
match the argument names of openbsd open(2) Signed-off-by: Leah Rowe <leah@libreboot.org>
16 hoursutil/nvmutil: use unsigned char in hextonumLeah Rowe
char can be signed or unsigned, and this is often implementation-defined. this could result in bad comparisons, so we should specifically cast it. Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: remove redundant breakLeah Rowe
err takes care of it Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: rename write_gbe to write_gbe_fileLeah Rowe
for clarity Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: rename read_gbe to read_gbe_fileLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: unified partnum validationLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: rename global st variableLeah Rowe
rename it to gbe_st, for clarity Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: full prototypesLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: tidy up read_gbeLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: tidy up set_mac_nibLeah Rowe
simplify it again Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: remove unnecessary checckLeah Rowe
the for loop exits when setting cmd checking for NULL is pointless Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursRevert "util/nvmutil: make invert an unsigned char"Leah Rowe
This reverts commit 5b120d71e713ea490bab7e90e21207cb16587779. the others are unsigned char. other variables like this one. better be consistent. Signed-off-by: Leah Rowe <leah@libreboot.org>
17 hoursutil/nvmutil: general code cleanupLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
18 hoursutil/nvmutil: cleaner mac address variable namesLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
18 hoursutil/nvmutil: general code cleanupLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
18 hoursutil/nvmutil: make invert an unsigned charLeah Rowe
Signed-off-by: Leah Rowe <leah@libreboot.org>
generated by cgit v1.2.1 (git 2.18.0) at 2026-03-07 14:20:41 +0000