[PATCH v2 2/3] test: Fix size_t formats in strlcat test

Minkyu Kang promsoft at gmail.com
Fri Jul 30 06:58:33 CEST 2021


Dear Simon Glass,

On Mon, 19 Jul 2021 at 10:02, Simon Glass <sjg at chromium.org> wrote:

> Some of the printf() formats do not match their types. Fix this.
>
> Reported-by: Minkyu Kang <promsoft at gmail.com>
>

I modified email address to mk7.kang at samsung.com : )

Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Fix some size_t warnings with recent compilers
>
>  test/lib/strlcat.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/test/lib/strlcat.c b/test/lib/strlcat.c
> index ee61684c405..a0ec037388b 100644
> --- a/test/lib/strlcat.c
> +++ b/test/lib/strlcat.c
> @@ -48,7 +48,7 @@ static int do_test_strlcat(struct unit_test_state *uts,
> int line, size_t align1,
>         if (expected != actual) {
>                 ut_failf(uts, __FILE__, line, __func__,
>                          "strlcat(s2, s1, 2) == len2 < n ? min(len1 +
> len2, n) : n",
> -                        "Expected %#lx (%ld), got %#lx (%ld)",
> +                        "Expected %#zx (%zd), got %#zx (%zd)",
>                          expected, expected, actual, actual);
>                 return CMD_RET_FAILURE;
>         }
> @@ -66,7 +66,7 @@ static int do_test_strlcat(struct unit_test_state *uts,
> int line, size_t align1,
>         if (len2 < n && s2[i] != '\0') {
>                 ut_failf(uts, __FILE__, line, __func__,
>                          "n < len1 && s2[len2 + n] == '\\0'",
> -                        "Expected s2[%ld] = '\\0', got %d ('%c')",
> +                        "Expected s2[%zd] = '\\0', got %d ('%c')",
>                          i, s2[i], s2[i]);
>                 return CMD_RET_FAILURE;
>         }
> --
> 2.32.0.402.g57bb445576-goog
>
>
applied to u-boot-samsung.

-- 
Thanks,
Minkyu Kang.


More information about the U-Boot mailing list