From b0499167243e867d8b4648cbf800548557ddb2b2 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Thu, 17 Oct 2019 16:59:11 +0200 Subject: [PATCH] fwtool: do not omit final 16 byte when image does not contain signature The fwutil command will interpret the final 16 byte of a given firmware image files as "struct fwimage_trailer". In case these bytes do look like a valid trailer, we must ensure that we print them out along with the remainder of the image to not accidentally truncate non-trailer-images by 16 bytes when they're piped through fwtool, e.g. as part of an image verification command sequence. Some command sequences pipe images through fwtool in order to strip any possible metadata, certificate or signature trailers and do not expect bare images without any of that metadata to get truncated as other non- fwtool specific metadata is expected at the end of the file, e.g. an information block with an md5sum in case of the combined image format. Signed-off-by: Jo-Philipp Wich --- Makefile | 2 +- src/fwtool.c | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 901081c..283be8d 100644 --- a/Makefile +++ b/Makefile @@ -8,7 +8,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=fwtool -PKG_RELEASE:=1 +PKG_RELEASE:=2 PKG_FLAGS:=nonshared diff --git a/src/fwtool.c b/src/fwtool.c index 3adc1e0..89e8951 100644 --- a/src/fwtool.c +++ b/src/fwtool.c @@ -328,12 +328,14 @@ extract_data(const char *name) if (extract_tail(&dbuf, &tr, sizeof(tr))) break; - data_len = be32_to_cpu(tr.size) - sizeof(tr); if (tr.magic != cpu_to_be32(FWIMAGE_MAGIC)) { msg("Data not found\n"); + metadata_keep = true; break; } + data_len = be32_to_cpu(tr.size) - sizeof(tr); + if (be32_to_cpu(tr.crc32) != tail_crc32(&dbuf, crc32)) { msg("CRC error\n"); break; -- 2.30.2