linux-kernel - Re: [PATCH] Makefile.extrawarn: add -Wformat-insufficient-args for clang build

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK7LNAR3YGoQU6ZTZmC84C1OoH0rPinjoyPDXCD0BDPoRS4NDA@mail.gmail.com>
Date:   Fri, 22 Jul 2022 13:20:30 +0900
From:   Masahiro Yamada <masahiroy@...nel.org>
To:     Youngmin Nam <youngmin.nam@...sung.com>
Cc:     Michal Marek <michal.lkml@...kovi.net>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        hosung0.kim@...sung.com, d7271.choe@...sung.com
Subject: Re: [PATCH] Makefile.extrawarn: add -Wformat-insufficient-args for
 clang build

On Sat, Jul 16, 2022 at 5:17 PM Youngmin Nam <youngmin.nam@...sung.com> wrote:
>
> The -Wformat-insufficient-args for clang is useful to detect the situation
> when the total number is unmatched between format specifiers and arguments.
>
> Originally, this option is enabled by default(Link[1]), but it is disabled by
> -Wno-format explicitly so that we can't detect this unmatched situation.
>
> We can enable it by adding this option after -Wno-format.
>
> Link[1]: https://releases.llvm.org/13.0.0/tools/clang/docs/DiagnosticsReference.html#wformat-insufficient-args
> Signed-off-by: Youngmin Nam <youngmin.nam@...sung.com>
> ---


Please let me hold on this patch because
I'd rather go straight to the removal of  -Wno-format.

https://lore.kernel.org/linux-kbuild/CAFhGd8pk+0XEz0tMiJcwMM7B3NYF=yF4cHW8A-6-81SgpKFPNw@mail.gmail.com/T/#m4becf6ed91f25217b59a840ed1829f36e49fe347



>  scripts/Makefile.extrawarn | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn
> index f5f0d6f09053..c23d7c286bad 100644
> --- a/scripts/Makefile.extrawarn
> +++ b/scripts/Makefile.extrawarn
> @@ -48,6 +48,7 @@ else
>  ifdef CONFIG_CC_IS_CLANG
>  KBUILD_CFLAGS += -Wno-initializer-overrides
>  KBUILD_CFLAGS += -Wno-format
> +KBUILD_CFLAGS += -Wformat-insufficient-args
>  KBUILD_CFLAGS += -Wno-sign-compare
>  KBUILD_CFLAGS += -Wno-format-zero-length
>  KBUILD_CFLAGS += $(call cc-disable-warning, pointer-to-enum-cast)
> --
> 2.34.0
>


-- 
Best Regards
Masahiro Yamada