From: Kaushlendra Kumar Date: Fri, 12 Sep 2025 20:06:08 +0000 (+0200) Subject: ACPICA: acpidump: fix return values in ap_is_valid_checksum() X-Git-Tag: v6.18-rc1~151^2~4^2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=54ba9071a04b0d05086826c442c2de5830fce5bf;p=thirdparty%2Fkernel%2Fstable.git ACPICA: acpidump: fix return values in ap_is_valid_checksum() The function ap_is_valid_checksum() has a boolean name suggesting it should return TRUE/FALSE, but incorrectly returns AE_OK on success and has no explicit return on failure, leading to undefined behavior. Fix by returning proper values: - FALSE when checksum validation fails - TRUE when checksum validation succeeds Link: https://github.com/acpica/acpica/commit/479ba862 Signed-off-by: Kaushlendra Kumar Signed-off-by: Rafael J. Wysocki --- diff --git a/tools/power/acpi/tools/acpidump/apdump.c b/tools/power/acpi/tools/acpidump/apdump.c index bf30143efbdcb..7a6223aa703c3 100644 --- a/tools/power/acpi/tools/acpidump/apdump.c +++ b/tools/power/acpi/tools/acpidump/apdump.c @@ -86,9 +86,10 @@ u8 ap_is_valid_checksum(struct acpi_table_header *table) if (ACPI_FAILURE(status)) { fprintf(stderr, "%4.4s: Warning: wrong checksum in table\n", table->signature); + return (FALSE); } - return (AE_OK); + return (TRUE); } /******************************************************************************