| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sat, 26 Mar 2011 18:25:27 +0100
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Alexey Dobriyan <adobriyan@...il.com>
Cc: akpm@...ux-foundation.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] kstrtox: fix compile warnings in test
On Thu, Mar 24, 2011 at 19:51, Alexey Dobriyan <adobriyan@...il.com> wrote:
> Fix the following warnings:
>
> CC [M] lib/test-kstrtox.o
> lib/test-kstrtox.c: In function 'test_kstrtou64_ok':
> lib/test-kstrtox.c:318: warning: this decimal constant is unsigned only in ISO C90
> ...
>
> Noticed by Geert Uytterhoeven <geert@...ux-m68k.org>
>
> Signed-off-by: Alexey Dobriyan <adobriyan@...il.com>
Acked-by: Geert Uytterhoeven <geert@...ux-m68k.org>
> ---
>
> lib/test-kstrtox.c | 32 ++++++++++++++++----------------
> 1 file changed, 16 insertions(+), 16 deletions(-)
>
> --- a/lib/test-kstrtox.c
> +++ b/lib/test-kstrtox.c
> @@ -315,12 +315,12 @@ static void __init test_kstrtou64_ok(void)
> {"65537", 10, 65537},
> {"2147483646", 10, 2147483646},
> {"2147483647", 10, 2147483647},
> - {"2147483648", 10, 2147483648},
> - {"2147483649", 10, 2147483649},
> - {"4294967294", 10, 4294967294},
> - {"4294967295", 10, 4294967295},
> - {"4294967296", 10, 4294967296},
> - {"4294967297", 10, 4294967297},
> + {"2147483648", 10, 2147483648ULL},
> + {"2147483649", 10, 2147483649ULL},
> + {"4294967294", 10, 4294967294ULL},
> + {"4294967295", 10, 4294967295ULL},
> + {"4294967296", 10, 4294967296ULL},
> + {"4294967297", 10, 4294967297ULL},
> {"9223372036854775806", 10, 9223372036854775806ULL},
> {"9223372036854775807", 10, 9223372036854775807ULL},
> {"9223372036854775808", 10, 9223372036854775808ULL},
> @@ -369,12 +369,12 @@ static void __init test_kstrtos64_ok(void)
> {"65537", 10, 65537},
> {"2147483646", 10, 2147483646},
> {"2147483647", 10, 2147483647},
> - {"2147483648", 10, 2147483648},
> - {"2147483649", 10, 2147483649},
> - {"4294967294", 10, 4294967294},
> - {"4294967295", 10, 4294967295},
> - {"4294967296", 10, 4294967296},
> - {"4294967297", 10, 4294967297},
> + {"2147483648", 10, 2147483648LL},
> + {"2147483649", 10, 2147483649LL},
> + {"4294967294", 10, 4294967294LL},
> + {"4294967295", 10, 4294967295LL},
> + {"4294967296", 10, 4294967296LL},
> + {"4294967297", 10, 4294967297LL},
> {"9223372036854775806", 10, 9223372036854775806LL},
> {"9223372036854775807", 10, 9223372036854775807LL},
> };
> @@ -418,10 +418,10 @@ static void __init test_kstrtou32_ok(void)
> {"65537", 10, 65537},
> {"2147483646", 10, 2147483646},
> {"2147483647", 10, 2147483647},
> - {"2147483648", 10, 2147483648},
> - {"2147483649", 10, 2147483649},
> - {"4294967294", 10, 4294967294},
> - {"4294967295", 10, 4294967295},
> + {"2147483648", 10, 2147483648U},
> + {"2147483649", 10, 2147483649U},
> + {"4294967294", 10, 4294967294U},
> + {"4294967295", 10, 4294967295U},
> };
> TEST_OK(kstrtou32, u32, "%u", test_u32_ok);
> }
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Powered by blists - more mailing lists