[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fwts-devel
Subject:    ACK: [PATCH 07/10] dmicheck: fix wrong length for reserved fields in type 13
From:       ivanhu <ivan.hu () canonical ! com>
Date:       2020-05-28 6:31:33
Message-ID: 1d036fd8-1e67-8d68-e35d-3002a170874b () canonical ! com
[Download RAW message or body]



On 5/21/20 10:37 AM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  src/dmi/dmicheck/dmicheck.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/dmi/dmicheck/dmicheck.c b/src/dmi/dmicheck/dmicheck.c
> index d163bed3..f23c353b 100644
> --- a/src/dmi/dmicheck/dmicheck.c
> +++ b/src/dmi/dmicheck/dmicheck.c
> @@ -1501,7 +1501,7 @@ static void dmicheck_entry(fwts_framework *fw,
>  			dmi_reserved_bits_check(fw, table, addr, "Flags", hdr, sizeof(uint8_t), 0x5, 1, 7);
>  			if (hdr->length < 0x15)
>  				break;
> -			for (i = 0x6; i < 0x15; i++)
> +			for (i = 0x6; i < 15; i++)
>  				dmi_reserved_uint8_check(fw, table, addr, "Reserved", hdr, i);
>  			if (hdr->length < 0x16)
>  				break;
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>

-- 
fwts-devel mailing list
fwts-devel@lists.ubuntu.com
Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/fwts-devel

[prev in list] [next in list] [prev in thread] [next in thread]