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@linaro.org Signed-off-by: Alexander Gendin agendin@matrox.com

Reviewed-by: Heinrich Schuchardt xypron.glpk@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) +