From 9a1c0e68ba8c764025c2aefe442387ad8a5f92e7 Mon Sep 17 00:00:00 2001 From: Alexander Amelkin Date: Fri, 21 May 2021 02:39:16 +0300 Subject: [PATCH] ekanalyzer: Fix internal use area off-by-one bug The length of internal use area printed by ekanalyzer didn't take in account the version byte. The data printed was also 1 byte longer than needed. Signed-off-by: Alexander Amelkin --- lib/ipmi_ekanalyzer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/ipmi_ekanalyzer.c b/lib/ipmi_ekanalyzer.c index 8a3e5a5..dbc76b4 100644 --- a/lib/ipmi_ekanalyzer.c +++ b/lib/ipmi_ekanalyzer.c @@ -2488,6 +2488,8 @@ ipmi_ek_display_fru_header_detail(char *filename) */ if (next_offset < UINT8_MAX) { len = (next_offset - header.offset.internal) * FACTOR_OFFSET; + --len; /* First byte of internal use area is version and we've + already read it */ } else { struct stat fs;