coja/lbmk
1
0
Fork 0
mirror of https://codeberg.org/libreboot/lbmk.git synced 2026-07-11 22:12:40 +02:00
Code Issues Packages Projects Releases Wiki Activity
3,342 Commits 22 Branches 45 Tags
a0da88d361b7b20004d2a23edbaf38fa3b8deb6d
Commit Graph

3342 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Leah Rowe a0da88d361 util/nvmutil: fix a comment 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 03:08:31 +00:00
Leah Rowe ea1046dc46 util/nvmutil: unambiguous sign conv_argv_part_num 
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>
 2026-03-07 03:00:35 +00:00
Leah Rowe b98f89c272 util/nvmutil: clean up hextonum() 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:50:30 +00:00
Leah Rowe b8bd1ca65a util/nvmutil: tidy up some comments 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:44:11 +00:00
Leah Rowe 49eca198da util/nvmutil: make mac_updated a uint8_t 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:38:36 +00:00
Leah Rowe ef2937edbd util/nvmutil: fix prototype indentation 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:37:46 +00:00
Leah Rowe 7bfe134a19 util/nvmutil: reset mac_buf for mac parsing 
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>
 2026-03-07 02:35:03 +00:00
Leah Rowe 14ad5a9818 util/nvmutil: comment the unhandled errno exit 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:09:56 +00:00
Leah Rowe 90bd395cb1 util/nvmutil: tidy up set_mac_byte 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:05:57 +00:00
Leah Rowe 4619dad0b7 util/nvmutil: further mac address parsing cleanup 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 02:00:14 +00:00
Leah Rowe 921144856c util/nvmutil: tidy up mac parsing code 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 01:39:57 +00:00
Leah Rowe 86357dbe52 util/nvmutil: fix a bad comment 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 01:28:50 +00:00
Leah Rowe 7e88f53e99 util/nvmutil: tidy up set_mac_byte 
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>
 2026-03-07 01:21:29 +00:00
Leah Rowe c02f3166bb util/nvmutil: make do_read a uint8_t 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 01:11:07 +00:00
Leah Rowe a9f5cbb542 util/nvmutil: clearer commenting about commands 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 01:09:52 +00:00
Leah Rowe 0084452c4a util/nvmutil: comment regarding buf 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 01:01:44 +00:00
Leah Rowe 364abddeab util/nvmutil: minor code cleanup 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:52:57 +00:00
Leah Rowe 68d689336b util/nvmutil: fix an err string 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:10:36 +00:00
Leah Rowe 658ed55bc1 util/nvmutil: tidy up conv_argv_part_num 
no need for strlen here (1-character string).

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:08:17 +00:00
Leah Rowe 330b2da20c util/nvmutil: Shorten a comment 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:04:57 +00:00
Leah Rowe 58182b48c3 util/nvmutil: fix a bad comment 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:03:10 +00:00
Leah Rowe 884a6779f9 util/nvmutil: make gbe_st local, not static/global 
forgot to remove static when making it local

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-07 00:01:27 +00:00
Leah Rowe 44e6b00626 util/nvmutil: rename rfd to urandom_fd 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 23:58:04 +00:00
Leah Rowe 9e5736baa9 util/nvmutil: remove reset_global_state 
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>
 2026-03-06 23:56:19 +00:00
Leah Rowe 13b35ed1fc util/nvmutil: rename arc4random define 
specifically refer to the actual function i use.

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 23:51:57 +00:00
Leah Rowe 79c972367e util/nvmutil: reset errno on /dev/random fallback 
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>
 2026-03-06 23:50:06 +00:00
Leah Rowe 6e47315192 util/nvmutil: remove rval from conv_argv_part_num 
relic from an earlier uncommitted version

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 23:42:33 +00:00
Leah Rowe 85b09c4275 util/nvmutil: set part_modified to a uint8_t 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 23:15:50 +00:00
Leah Rowe e5b108a764 util/nvmutil: make invert a uint8_t 
cleaner intention, and shorter

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 23:14:26 +00:00
Leah Rowe 69c4d70650 util/nvmutil: safer argv part number parsing 
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>
 2026-03-06 23:07:29 +00:00
Leah Rowe 9b6f2a2f7e util/nvmutil: rename check_bound check_nvm_bound 
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>
 2026-03-06 22:39:43 +00:00
Leah Rowe a7d7f68a5c util/nvmutil: be clearer about GbE file sizes 
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>
 2026-03-06 22:36:08 +00:00
Leah Rowe e273391792 util/nvmutil: rename fd to fd_ptr in xopen() 
clearer intent

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 22:26:22 +00:00
Leah Rowe 4635a0eae9 util/nvmutil: cleaned up a few prototypes 
make them match how they're declared

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 22:20:15 +00:00
Leah Rowe 0f5fcbd883 util/nvmutil: more readable xopen 
match the argument names of openbsd open(2)

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 22:18:26 +00:00
Leah Rowe 2c2bda31ea util/nvmutil: use unsigned char in hextonum 
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>
 2026-03-06 22:09:56 +00:00
Leah Rowe f57358ac1e util/nvmutil: remove redundant break 
err takes care of it

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:43:11 +00:00
Leah Rowe f59f45893b util/nvmutil: rename write_gbe to write_gbe_file 
for clarity

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:39:58 +00:00
Leah Rowe 85877a93bb util/nvmutil: rename read_gbe to read_gbe_file 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:39:21 +00:00
Leah Rowe dad1f613fc util/nvmutil: unified partnum validation 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:35:49 +00:00
Leah Rowe ed32108613 util/nvmutil: rename global st variable 
rename it to gbe_st, for clarity

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:22:50 +00:00
Leah Rowe b65c988007 util/nvmutil: full prototypes 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:21:40 +00:00
Leah Rowe 3aa30c5039 util/nvmutil: tidy up read_gbe 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:08:47 +00:00
Leah Rowe 040eee4607 util/nvmutil: tidy up set_mac_nib 
simplify it again

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 21:06:58 +00:00
Leah Rowe 6b924787be util/nvmutil: remove unnecessary checck 
the for loop exits when setting cmd

checking for NULL is pointless

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 20:59:50 +00:00
Leah Rowe af55dd8959 Revert "util/nvmutil: make invert an unsigned char" 
This reverts commit 5b120d71e7.

the others are unsigned char. other variables like this one.

better be consistent.

Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 20:58:04 +00:00
Leah Rowe 10c8be92aa util/nvmutil: general code cleanup 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 20:52:13 +00:00
Leah Rowe a34d430754 util/nvmutil: cleaner mac address variable names 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 20:34:20 +00:00
Leah Rowe 00e0197cfe util/nvmutil: general code cleanup 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 20:11:30 +00:00
Leah Rowe 5b120d71e7 util/nvmutil: make invert an unsigned char 
Signed-off-by: Leah Rowe <leah@libreboot.org>
 2026-03-06 19:59:51 +00:00