Message ID | 20201002221527.177500-5-keescook@chromium.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Clean up UBSAN Makefile | expand |
On Fri, Oct 02, 2020 at 03:15:27PM -0700, Kees Cook wrote: > Doing all*config builds attempts build as much as possible. UBSAN_TRAP > effectively short-circuits lib/usban.c, so it should be disabled for > COMPILE_TEST so that the lib/ubsan.c code gets built. > > Signed-off-by: Kees Cook <keescook@chromium.org> Reviewed-by: Nathan Chancellor <natechancellor@gmail.com> > --- > lib/Kconfig.ubsan | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan > index 1fc07f936e06..b5b9da0b635a 100644 > --- a/lib/Kconfig.ubsan > +++ b/lib/Kconfig.ubsan > @@ -14,6 +14,7 @@ if UBSAN > > config UBSAN_TRAP > bool "On Sanitizer warnings, abort the running kernel code" > + depends on !COMPILE_TEST > depends on $(cc-option, -fsanitize-undefined-trap-on-error) > help > Building kernels with Sanitizer features enabled tends to grow > -- > 2.25.1
diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan index 1fc07f936e06..b5b9da0b635a 100644 --- a/lib/Kconfig.ubsan +++ b/lib/Kconfig.ubsan @@ -14,6 +14,7 @@ if UBSAN config UBSAN_TRAP bool "On Sanitizer warnings, abort the running kernel code" + depends on !COMPILE_TEST depends on $(cc-option, -fsanitize-undefined-trap-on-error) help Building kernels with Sanitizer features enabled tends to grow
Doing all*config builds attempts build as much as possible. UBSAN_TRAP effectively short-circuits lib/usban.c, so it should be disabled for COMPILE_TEST so that the lib/ubsan.c code gets built. Signed-off-by: Kees Cook <keescook@chromium.org> --- lib/Kconfig.ubsan | 1 + 1 file changed, 1 insertion(+)