Skip to content

Commit d6a0e0b

Browse files
committed
ubsan: integer-overflow: depend on BROKEN to keep this out of CI
Depending on !COMPILE_TEST isn't sufficient to keep this feature out of CI because we can't stop it from being included in randconfig builds. This feature is still highly experimental, and is developed in lock-step with Clang's Overflow Behavior Types[1]. Depend on BROKEN to keep it from being enabled by anyone not expecting it. Link: https://discourse.llvm.org/t/rfc-v2-clang-introduce-overflowbehaviortypes-for-wrapping-and-non-wrapping-arithmetic/86507 [1] Reported-by: kernel test robot <[email protected]> Closes: https://lore.kernel.org/oe-lkp/[email protected] Fixes: 557f8c5 ("ubsan: Reintroduce signed overflow sanitizer") Acked-by: Eric Biggers <[email protected]> Link: https://lore.kernel.org/r/[email protected] Reviewed-by: Nathan Chancellor <[email protected]> Acked-by: Marco Elver <[email protected]> Signed-off-by: Kees Cook <[email protected]>
1 parent 368556d commit d6a0e0b

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

lib/Kconfig.ubsan

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -118,6 +118,8 @@ config UBSAN_UNREACHABLE
118118

119119
config UBSAN_INTEGER_WRAP
120120
bool "Perform checking for integer arithmetic wrap-around"
121+
# This is very experimental so drop the next line if you really want it
122+
depends on BROKEN
121123
depends on !COMPILE_TEST
122124
depends on $(cc-option,-fsanitize-undefined-ignore-overflow-pattern=all)
123125
depends on $(cc-option,-fsanitize=signed-integer-overflow)

0 commit comments

Comments
 (0)