mirror of
https://github.com/torvalds/linux.git
synced 2026-10-22 23:01:49 +02:00
gen_init_cpio: stop parsing on lines with a missing argument list
When a line in the cpio list consists of a file type followed only by
whitespace and no newline (e.g. "slink " as the last line of the file),
strtok(NULL, "\n") returns NULL. main() prints an error and sets ec, but
then carries on and passes the NULL args pointer to the type handler,
which hands it to sscanf() and crashes:
AddressSanitizer: SEGV on unknown address 0x000000000000
#4 __isoc99_sscanf
#5 cpio_mkslink_line usr/gen_init_cpio.c:169
Break out of the loop as is already done for the other format error, so
that the error is reported and gen_init_cpio exits with a failure status.
Found by fuzzing gen_init_cpio built with ASan/UBSan.
Reproducer:
printf 'slink ' > list && usr/gen_init_cpio list
Fixes: 1da177e4c3 ("Linux-2.6.12-rc2")
Assisted-by: Claude:claude-opus-5-5 ASan UBSan
Signed-off-by: Leizhen Zhang <lzsx618@gmail.com>
Link: https://patch.msgid.link/20261005085729.872402-1-lzsx618@gmail.com
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
This commit is contained in:
parent
a7d32fa14b
commit
c8dd1aa027
|
|
@ -756,6 +756,7 @@ int main (int argc, char *argv[])
|
|||
"ERROR: incorrect format, newline required line %d: '%s'\n",
|
||||
line_nr, line);
|
||||
ec = -1;
|
||||
break;
|
||||
}
|
||||
|
||||
for (type_idx = 0; file_handler_table[type_idx].type; type_idx++) {
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user