busybox.git/scripts/basic, branch master

fixdep: add fstat error handling

2024-09-27T18:03:30+00:00

When `fstat` fails, `st` is left uninitialised. In our case, Ben Kohler
noticed our release media builds were failing in Gentoo on x86 when building
busybox with occasional SIGBUS. This turned out to be EOVERFLOW (from 32-bit
ino_t) which wasn't being reported because nothing was checking the return value
from `fstat`.

Fix that to avoid UB (use of uninit var) and to give a more friendly
error to the user.

This actually turns out to be fixed already in the kernel from back in
2010 [0] and 2016 [1].

[0] https://github.com/torvalds/linux/commit/a3ba81131aca243bfecfa78c42edec0cd69f72d6
[1] https://github.com/torvalds/linux/commit/46fe94ad18aa7ce6b3dad8c035fb538942020f2b

Reported-by: Ben Kohler 
Signed-off-by: Sam James 
Signed-off-by: Denys Vlasenko

docproc: avoid segfault during file closing

2024-04-13T15:57:46+00:00

In the function find_export_symbols, since the fopen file does not
exit when it fails, there is a dereference problem in fclose(fp),
which will cause a segmentation fault.

Signed-off-by: Yan Zhu 
Signed-off-by: Denys Vlasenko

fixdep: avoid underflow when end of entry doesn't coincide with EOF

2023-02-27T12:09:44+00:00

Bug: https://bugs.gentoo.org/893776
Closes: https://bugs.busybox.net/show_bug.cgi?id=15326
Signed-off-by: Arsen Arsenović 
Signed-off-by: Denys Vlasenko

fix "warning array subscript has type 'char'"

2021-04-14T17:12:43+00:00

Signed-off-by: Denys Vlasenko

build system: avoid build failure during bisection

2021-04-13T16:26:46+00:00

Commit 4bdc914ff (build system: fix compiler warnings) added a
test on the return value of fgets() in split-include.c.

During bisection it's possible to go back to a state where a
configuration value didn't exist.  This results in an empty
include file corresponding to the missing feature.  If a
subsequent bisection returns to a state where the feature exists
split-include treats the empty file as an error and the build
fails.

Add a call to ferror() to distinguish between fgets() failing
due to an error and due to there being no data to read.

Signed-off-by: Ron Yorston 
Signed-off-by: Denys Vlasenko

build system: fix compiler warnings

2019-01-06T19:12:16+00:00

Signed-off-by: Denys Vlasenko

typo fixes

2015-05-25T11:46:36+00:00

Signed-off-by: Maninder Singh 
Signed-off-by: Denys Vlasenko

build system: stop including alloca.h, stdlib.h provides it

2014-01-09T10:02:46+00:00

Signed-off-by: Denys Vlasenko

suppress "warning: implicit declaration of function 'alloca'"

2011-09-15T08:37:04+00:00

Signed-off-by: Denys Vlasenko

Fix double words in comments. No code changes

2011-04-16T15:33:43+00:00

Signed-off-by: Marek Polacek 
Signed-off-by: Denys Vlasenko