Date: Fri, 22 Nov 2019 10:07:29 +0100 From: Dmitry Vyukov <dvyukov@...gle.com> To: Kees Cook <keescook@...omium.org> Cc: Andrew Morton <akpm@...ux-foundation.org>, Andrey Ryabinin <aryabinin@...tuozzo.com>, Elena Petrova <lenaptr@...gle.com>, Alexander Potapenko <glider@...gle.com>, Linus Torvalds <torvalds@...ux-foundation.org>, Dan Carpenter <dan.carpenter@...cle.com>, "Gustavo A. R. Silva" <gustavo@...eddedor.com>, Arnd Bergmann <arnd@...db.de>, Ard Biesheuvel <ard.biesheuvel@...aro.org>, kasan-dev <kasan-dev@...glegroups.com>, LKML <linux-kernel@...r.kernel.org>, kernel-hardening@...ts.openwall.com, syzkaller <syzkaller@...glegroups.com> Subject: Re: [PATCH v2 0/3] ubsan: Split out bounds checker On Thu, Nov 21, 2019 at 7:15 PM Kees Cook <keescook@...omium.org> wrote: > > v2: > - clarify Kconfig help text (aryabinin) > - add reviewed-by > - aim series at akpm, which seems to be where ubsan goes through? > v1: https://firstname.lastname@example.org > > This splits out the bounds checker so it can be individually used. This > is expected to be enabled in Android and hopefully for syzbot. Includes > LKDTM tests for behavioral corner-cases (beyond just the bounds checker). > > -Kees +syzkaller mailing list This is great! I wanted to enable UBSAN on syzbot for a long time. And it's _probably_ not lots of work. But it was stuck on somebody actually dedicating some time specifically for it. Kees, or anybody else interested, could you provide relevant configs that (1) useful for kernel, (2) we want 100% cleanliness, (3) don't fire all the time even without fuzzing? Anything else required to enable UBSAN? I don't see anything. syzbot uses gcc 8.something, which I assume should be enough (but we can upgrade if necessary). > Kees Cook (3): > ubsan: Add trap instrumentation option > ubsan: Split "bounds" checker from other options > lkdtm/bugs: Add arithmetic overflow and array bounds checks > > drivers/misc/lkdtm/bugs.c | 75 ++++++++++++++++++++++++++++++++++++++ > drivers/misc/lkdtm/core.c | 3 ++ > drivers/misc/lkdtm/lkdtm.h | 3 ++ > lib/Kconfig.ubsan | 42 +++++++++++++++++++-- > lib/Makefile | 2 + > scripts/Makefile.ubsan | 16 ++++++-- > 6 files changed, 134 insertions(+), 7 deletions(-) > > -- > 2.17.1
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.