lbmk.git/util/libreboot-utils/lib/file.c, branch x220normal

delete util/libreboot-utils (unused code)

2026-09-07T07:12:24+00:00

this was an intense audit of nvmutil that somehow evolved
into writing a new, hardened implementation of mktemp.

it all works, a few memory bugs to solve on bsd, but i don't
see the point in keeping it. mktemp is fine, and nvmutil
already works.

lbutils implemented atomic writes and integrity checking,
in a manner completely overengineered for what it was
actually doing (modifying a few bytes in 8KB GbE files)

just delete it. i'll bring it back if i ever finish the
code. i don't want to leave dead/unfinished code in the tree.

Signed-off-by: Leah Rowe

lbutils: don't use stack memory for path strings

2026-04-21T05:39:50+00:00

Signed-off-by: Leah Rowe

lbutils/file: ignore close err if errno is EINTR

2026-04-01T20:05:08+00:00

but DONT LOOP IT. see comment.

Signed-off-by: Leah Rowe

remove dead code

2026-04-01T19:36:05+00:00

Signed-off-by: Leah Rowe

libreboot-utils: don't loop lseek on EINTR

2026-04-01T18:17:32+00:00

not necessary.

Signed-off-by: Leah Rowe

lbutils/file: don't use undefined USE_OPENAT

2026-04-01T18:09:26+00:00

clang -Weverything told me to

Signed-off-by: Leah Rowe

lbutils/file: don't loop EINTR on close()

2026-04-01T16:43:43+00:00

state is undefined after EINTR. just abort universally.

Signed-off-by: Leah Rowe

lbutils/file: fix implicit conversion on openat2

2026-04-01T16:33:01+00:00

as dictated by clang -Weverything

Signed-off-by: Leah Rowe

lbutils/file: fix overflow check

2026-04-01T16:29:28+00:00

clang -Weverything:

lib/file.c:165:49: warning: implicit conversion changes signedness: 'ssize_t' (aka 'long') to 'size_t' (aka 'unsigned long') [-Wsign-conversion]
  165 |                 if (if_err(rval >= 0 && (size_t)rval > (nrw - rc), EOVERFLOW))

Signed-off-by: Leah Rowe

lbutils/file: remove unused macro

2026-04-01T15:49:19+00:00

not needed here (detected with clang -Weverything)

Signed-off-by: Leah Rowe