Fix cper header logging
This commit adjusts the cper header logging to solve two problems.
First, the two "invalid header" logs are identical, so it's unclear
which one is correct.
Second, when the header is invalid, add printing of the invalid value.
Change-Id: I241294eb1bbbd4a79dcb27e743842ead8dc21f5f
Signed-off-by: Ed Tanous <etanous@nvidia.com>
diff --git a/cper-parse.c b/cper-parse.c
index edbae56..af7d424 100644
--- a/cper-parse.c
+++ b/cper-parse.c
@@ -38,7 +38,7 @@
if (remaining < sizeof(EFI_COMMON_ERROR_RECORD_HEADER)) {
cper_print_log(
- "Invalid CPER file: Invalid header (incorrect signature).\n");
+ "Invalid CPER file: Invalid header (not enough bytes).\n");
goto fail;
}
@@ -48,7 +48,8 @@
remaining -= sizeof(EFI_COMMON_ERROR_RECORD_HEADER);
if (header->SignatureStart != EFI_ERROR_RECORD_SIGNATURE_START) {
cper_print_log(
- "Invalid CPER file: Invalid header (incorrect signature).\n");
+ "Invalid CPER file: Invalid header (incorrect signature). %x\n",
+ header->SignatureStart);
goto fail;
}
if (header->SectionCount == 0) {