From deb9b03af9aff54a3dee390cbd5ee63b0b6fcc26 Mon Sep 17 00:00:00 2001 From: Paul Menzel Date: Wed, 8 Mar 2017 10:51:52 +0100 Subject: cbfstool/ifwitool: Remove unnecessary assignment Fix the warning below. ``` util/cbfstool/ifwitool.c:551:2: warning: Value stored to 'offset' is never read offset = read_member(data, offset, sizeof(h->fit_tool_version), ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ``` Found-by: scan-build from clang 3.8.1 Change-Id: I6c322a335a371a20561b32e04e7dcc7310dab607 Signed-off-by: Paul Menzel Reviewed-on: https://review.coreboot.org/18667 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- util/cbfstool/ifwitool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/util/cbfstool/ifwitool.c b/util/cbfstool/ifwitool.c index f6ca2a848e..32d09ca6c8 100644 --- a/util/cbfstool/ifwitool.c +++ b/util/cbfstool/ifwitool.c @@ -548,8 +548,8 @@ static void bpdt_read_header(void *data, struct bpdt_header *h, &h->xor_redundant_block); offset = read_member(data, offset, sizeof(h->ifwi_version), &h->ifwi_version); - offset = read_member(data, offset, sizeof(h->fit_tool_version), - &h->fit_tool_version); + read_member(data, offset, sizeof(h->fit_tool_version), + &h->fit_tool_version); bpdt_validate_header(h, name); bpdt_print_header(h, name); -- cgit v1.2.3