aboutsummaryrefslogtreecommitdiff
path: root/cmd/acpi.c
diff options
context:
space:
mode:
authorTom Rini <trini@konsulko.com>2022-04-29 11:25:06 -0400
committerTom Rini <trini@konsulko.com>2022-04-29 11:25:06 -0400
commitf7bd9e4936b6e36c2443b9b2ef761e7593511521 (patch)
treef830de0295223e3d938244e60e80a89cdc7f3e91 /cmd/acpi.c
parente95afa56753cebcd20a5114b6d121f281b789006 (diff)
parentd69616e529560ace8cdf40bda91464a88c7ff43a (diff)
Merge branch '2022-04-29-fuzzing-and-asan-fixes'
To quote the author: I've been experimenting with ASAN on sandbox and turned up a few issues that are fixed in this series. Basic ASAN was easy to turn on, but integrating with dlmalloc was messier and fairly intrusive. Even when I had it working, there was only a small redzone between allocations which limits the usefulness. I saw another series on the list by Sean Anderson to enable valgrind which was finding a different set of issues, though there was one overlap that Sean is fixing with "[PATCH] IOMUX: Fix access past end of console_devices". With these issues fixed, I was able to run the dm tests without any ASAN issues. There are a couple of leaks reported at the end, but that's for another day.
Diffstat (limited to 'cmd/acpi.c')
-rw-r--r--cmd/acpi.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/cmd/acpi.c b/cmd/acpi.c
index c543f1e3c2..0e473b415d 100644
--- a/cmd/acpi.c
+++ b/cmd/acpi.c
@@ -178,7 +178,7 @@ static int do_acpi_dump(struct cmd_tbl *cmdtp, int flag, int argc,
printf("Table name '%s' must be four characters\n", name);
return CMD_RET_FAILURE;
}
- str_to_upper(name, sig, -1);
+ str_to_upper(name, sig, ACPI_NAME_LEN);
ret = dump_table_name(sig);
if (ret) {
printf("Table '%.*s' not found\n", ACPI_NAME_LEN, sig);