diff options
author | Andy Lutomirski <luto@kernel.org> | 2020-03-19 14:30:56 -0700 |
---|---|---|
committer | Borislav Petkov <bp@suse.de> | 2020-03-20 11:20:04 +0100 |
commit | 07f24dc95daca49b8a2e804edc024dd4e91610ac (patch) | |
tree | 354b59093889111beb67683cb41256e007b10585 /tools/testing/selftests/x86/test_vdso.c | |
parent | fb33c6510d5595144d585aa194d377cf74d31911 (diff) |
selftests/x86/vdso: Fix no-vDSO segfaults
test_vdso would try to call a NULL pointer if the vDSO was missing.
vdso_restorer_32 hit a genuine failure: trying to use the
kernel-provided signal restorer doesn't work if the vDSO is missing.
Skip the test if the vDSO is missing, since the test adds no particular
value in that case.
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Andy Lutomirski <luto@kernel.org>
Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lkml.kernel.org/r/618ea7b8c55b10d08b1cb139e9a3a957934b8647.1584653439.git.luto@kernel.org
Diffstat (limited to 'tools/testing/selftests/x86/test_vdso.c')
-rw-r--r-- | tools/testing/selftests/x86/test_vdso.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tools/testing/selftests/x86/test_vdso.c b/tools/testing/selftests/x86/test_vdso.c index 35edd61d1663..42052db0f870 100644 --- a/tools/testing/selftests/x86/test_vdso.c +++ b/tools/testing/selftests/x86/test_vdso.c @@ -259,6 +259,11 @@ static void test_one_clock_gettime(int clock, const char *name) static void test_clock_gettime(void) { + if (!vdso_clock_gettime) { + printf("[SKIP]\tNo vDSO, so skipping clock_gettime() tests\n"); + return; + } + for (int clock = 0; clock < sizeof(clocknames) / sizeof(clocknames[0]); clock++) { test_one_clock_gettime(clock, clocknames[clock]); |