]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
test: unicode: fix a sizeof() vs ARRAY_SIZE() bug
authorDan Carpenter <dan.carpenter@linaro.org>
Thu, 27 Jul 2023 07:12:58 +0000 (10:12 +0300)
committerTom Rini <trini@konsulko.com>
Tue, 8 Aug 2023 21:41:52 +0000 (17:41 -0400)
The u16_strlcat() is in units of u16 not bytes.  So the limit needs to
be ARRAY_SIZE() instead of sizeof().

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
test/unicode_ut.c

index a9356e2b60db217302fd3a3f1147f5add0b2f176..1d0d90c2d734ccb4b681d548eaf8e0623627d7a4 100644 (file)
@@ -807,12 +807,12 @@ static int unicode_test_u16_strlcat(struct unit_test_state *uts)
 
        /* dest and src are empty string */
        memset(buf, 0, sizeof(buf));
-       ret = u16_strlcat(buf, &null_src, sizeof(buf));
+       ret = u16_strlcat(buf, &null_src, ARRAY_SIZE(buf));
        ut_asserteq(0, ret);
 
        /* dest is empty string */
        memset(buf, 0, sizeof(buf));
-       ret = u16_strlcat(buf, src, sizeof(buf));
+       ret = u16_strlcat(buf, src, ARRAY_SIZE(buf));
        ut_asserteq(4, ret);
        ut_assert(!unicode_test_u16_strcmp(buf, src, 40));
 
@@ -820,7 +820,7 @@ static int unicode_test_u16_strlcat(struct unit_test_state *uts)
        memset(buf, 0xCD, (sizeof(buf) - sizeof(u16)));
        buf[39] = 0;
        memcpy(buf, dest, sizeof(dest));
-       ret = u16_strlcat(buf, &null_src, sizeof(buf));
+       ret = u16_strlcat(buf, &null_src, ARRAY_SIZE(buf));
        ut_asserteq(5, ret);
        ut_assert(!unicode_test_u16_strcmp(buf, dest, 40));