summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorUriel Guajardo <urielguajardo@google.com>2021-04-02 14:21:31 -0700
committerShuah Khan <skhan@linuxfoundation.org>2021-08-13 13:19:06 -0600
commit1195505f5de2adfee5f277433f0b35498467cc64 (patch)
treef484ff3c2589c4b991491fd13a768d470096a910
parentb0d4adaf3b3c4402d9c3b6186e02aa1e4f7985cd (diff)
downloadlinux-stable-1195505f5de2adfee5f277433f0b35498467cc64.tar.gz
linux-stable-1195505f5de2adfee5f277433f0b35498467cc64.tar.bz2
linux-stable-1195505f5de2adfee5f277433f0b35498467cc64.zip
kunit: ubsan integration
Integrates UBSAN into the KUnit testing framework. It fails KUnit tests whenever it reports undefined behavior. When CONFIG_KUNIT=n, nothing is printed or even formatted, so this has no behavioral impact outside of tests. kunit_fail_current_test() effectively does a pr_err() as well, so there's some slight duplication, but it also ensures an error is recorded in the debugfs entry for the running KUnit test. Print a shorter version of the message to make it less spammy. Co-developed-by: Daniel Latypov <dlatypov@google.com> Signed-off-by: Daniel Latypov <dlatypov@google.com> Signed-off-by: Uriel Guajardo <urielguajardo@google.com> Reviewed-by: Alan Maguire <alan.maguire@oracle.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
-rw-r--r--lib/ubsan.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lib/ubsan.c b/lib/ubsan.c
index 26229973049d..bdc380ff5d5c 100644
--- a/lib/ubsan.c
+++ b/lib/ubsan.c
@@ -14,6 +14,7 @@
#include <linux/types.h>
#include <linux/sched.h>
#include <linux/uaccess.h>
+#include <kunit/test-bug.h>
#include "ubsan.h"
@@ -141,6 +142,8 @@ static void ubsan_prologue(struct source_location *loc, const char *reason)
"========================================\n");
pr_err("UBSAN: %s in %s:%d:%d\n", reason, loc->file_name,
loc->line & LINE_MASK, loc->column & COLUMN_MASK);
+
+ kunit_fail_current_test("%s in %s", reason, loc->file_name);
}
static void ubsan_epilogue(void)