[PATCH] test: cmd: mbr: Fix Smatch static checker warning

Heinrich Schuchardt xypron.glpk at gmx.de
Sat Feb 3 07:55:47 CET 2024


On 2/3/24 03:56, Alexander Gendin wrote:
> This patch fixes Smatch static checker warning:
>          test/cmd/mbr.c:243 mbr_test_run()
>          warn: sizeof(NUMBER)?
>
> Reported-by: Dan Carpenter <dan.carpenter at linaro.org>
> Signed-off-by: Alexander Gendin <agendin at matrox.com>

Reviewed-by: Heinrich Schuchardt <xypron.glpk at gmx.de>

> ---
>   test/cmd/mbr.c | 6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/test/cmd/mbr.c b/test/cmd/mbr.c
> index 46b78e706c..235b363290 100644
> --- a/test/cmd/mbr.c
> +++ b/test/cmd/mbr.c
> @@ -240,7 +240,11 @@ static int mbr_test_run(struct unit_test_state *uts)
>   	ut_assert(ofnode_valid(node));
>   	ut_assertok(lists_bind_fdt(gd->dm_root, node, &dev, NULL, false));
>
> -	mbr_parts_max = sizeof('\0') + 2 +
> +	/*
> +	 * 1 byte for null character
> +	 * 2 reserved bytes
> +	 */
> +	mbr_parts_max = 1 + 2 +
>   		strlen(mbr_parts_header) +
>   		strlen(mbr_parts_p1) +
>   		strlen(mbr_parts_p2) +



More information about the U-Boot mailing list