From: Pali Rohár Date: Sun, 29 Jan 2023 13:33:36 +0000 (+0100) Subject: tools: kwbimage: Fix generating secure boot data image signature X-Git-Tag: baikal/mips/sdk6.2~4^2~3^2~175^2~41^2~16 X-Git-Url: https://git.baikalelectronics.ru/sdk/?a=commitdiff_plain;h=a1139fdeaa01a5647d797f1c8698ccb4d884ba34;p=uboot.git tools: kwbimage: Fix generating secure boot data image signature Secure boot data image signature is calculated from the data image without trailing 4-bit checksum. Commit 77bf89a1f015 ("tools: kwbimage: Simplify aligning and calculating checksum") unintentionally broke this calculation when it increased payloadsz variable by 4 bytes which was propagated also into the add_secure_header_v1() function. Fix this issue by decreasing size of buffer by 4 bytes from which is calculated secure boot data image signature. Fixes: 77bf89a1f015 ("tools: kwbimage: Simplify aligning and calculating checksum") Signed-off-by: Pali Rohár --- diff --git a/tools/kwbimage.c b/tools/kwbimage.c index b32f845b7e..a8a59c154b 100644 --- a/tools/kwbimage.c +++ b/tools/kwbimage.c @@ -1355,7 +1355,7 @@ static int add_secure_header_v1(struct image_tool_params *params, uint8_t *image if (kwb_sign_csk_with_kak(params, secure_hdr, csk)) return 1; - if (kwb_sign_and_verify(csk, image_ptr, image_size, + if (kwb_sign_and_verify(csk, image_ptr, image_size - 4, &secure_hdr->imgsig, "image") < 0) return 1;