linux-kernel - Re: [PATCH] kasan: add missing Kconfig dependencies

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: <CAK8P3a1944yQ5rUG8zAYB5O5CQ=G-khyRGkwQmn-fZ6_-jMKcQ@mail.gmail.com>
Date:   Mon, 10 Dec 2018 22:26:15 +0100
From:   Arnd Bergmann <arnd@...db.de>
To:     Andrey Konovalov <andreyknvl@...gle.com>
Cc:     Andrew Morton <akpm@...ux-foundation.org>,
        Andrey Ryabinin <aryabinin@...tuozzo.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>, gor@...ux.ibm.com,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] kasan: add missing Kconfig dependencies

On Mon, Dec 10, 2018 at 10:15 PM Andrey Konovalov <andreyknvl@...gle.com> wrote:
>
> On Mon, Dec 10, 2018 at 9:59 PM Arnd Bergmann <arnd@...db.de> wrote:
> >
> > Building with CONFIG_KASAN=y when neither KASAN_GENERIC nor KASAN_SW_TAGS
> > can be selected leads to a build error:
> >
> > arch/arm64/mm/kasan_init.c: In function 'kasan_pte_populate':
> > arch/arm64/mm/kasan_init.c:106:28: error: 'KASAN_SHADOW_INIT' undeclared (first use in this function); did you mean 'KASAN_SHADOW_END'?
> >
> > We should really only allow enabling KASAN when all its dependencies
> > are met as it was before the introduction of CONFIG_KASAN_SW_TAGS.
> >
> > Fixes: b8440cc16515 ("kasan: add CONFIG_KASAN_GENERIC and CONFIG_KASAN_SW_TAGS")
> > Signed-off-by: Arnd Bergmann <arnd@...db.de>
>
> Hi Arnd,
>
> Is this a patch against linux-next? The issue has been addressed by
> v13 of the patchset, which is now in the mm tree and should end up in
> the linux-next soon AFAIU.

Yes, I tested on today's linux-next. I'll drop the patch tomorrow then from
my test tree.

       Arnd