Module Name: src
Committed By: mgorny
Date: Tue Jan 29 19:59:10 UTC 2019
Modified Files:
src/tests/usr.bin/cc: Makefile t_ubsan_int_add_overflow.sh
t_ubsan_int_divzero.sh t_ubsan_int_neg_overflow.sh
t_ubsan_int_sub_overflow.sh t_ubsan_vla_out_of_bounds.sh
Added Files:
src/tests/usr.bin/cc: ubsan_common.subr
Log Message:
Move common UBSAN test case code into ubsan_common.subr
Reviewed by <kamil>
To generate a diff of this commit:
cvs rdiff -u -r1.5 -r1.6 src/tests/usr.bin/cc/Makefile
cvs rdiff -u -r1.3 -r1.4 src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh \
src/tests/usr.bin/cc/t_ubsan_int_divzero.sh \
src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh \
src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh
cvs rdiff -u -r1.4 -r1.5 src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh
cvs rdiff -u -r0 -r1.1 src/tests/usr.bin/cc/ubsan_common.subr
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/tests/usr.bin/cc/Makefile
diff -u src/tests/usr.bin/cc/Makefile:1.5 src/tests/usr.bin/cc/Makefile:1.6
--- src/tests/usr.bin/cc/Makefile:1.5 Tue Jan 29 19:56:37 2019
+++ src/tests/usr.bin/cc/Makefile Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.5 2019/01/29 19:56:37 mgorny Exp $
+# $NetBSD: Makefile,v 1.6 2019/01/29 19:59:10 mgorny Exp $
.include <bsd.own.mk>
@@ -12,17 +12,23 @@ ASAN_TESTS+= t_asan_off_by_one
ASAN_TESTS+= t_asan_poison
ASAN_TESTS+= t_asan_uaf
+UBSAN_TESTS= #
+UBSAN_TESTS+= t_ubsan_int_add_overflow
+UBSAN_TESTS+= t_ubsan_int_neg_overflow
+UBSAN_TESTS+= t_ubsan_vla_out_of_bounds
+UBSAN_TESTS+= t_ubsan_int_sub_overflow
+UBSAN_TESTS+= t_ubsan_int_divzero
+
TESTS_SH= #
TESTS_SH+= $(ASAN_TESTS)
-TESTS_SH+= t_ubsan_int_add_overflow
-TESTS_SH+= t_ubsan_int_sub_overflow
-TESTS_SH+= t_ubsan_int_neg_overflow
-TESTS_SH+= t_ubsan_int_divzero
-TESTS_SH+= t_ubsan_vla_out_of_bounds
+TESTS_SH+= $(UBSAN_TESTS)
TESTS_SH+= t_hello
.for test in ${ASAN_TESTS}
TESTS_SH_SRC_${test}= asan_common.subr ${test}.sh
.endfor
+.for test in ${UBSAN_TESTS}
+TESTS_SH_SRC_${test}= ubsan_common.subr ${test}.sh
+.endfor
.include <bsd.test.mk>
Index: src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh
diff -u src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh:1.3 src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh:1.4
--- src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh:1.3 Mon Jul 16 07:25:58 2018
+++ src/tests/usr.bin/cc/t_ubsan_int_add_overflow.sh Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# Copyright (c) 2018 The NetBSD Foundation, Inc.
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
# All rights reserved.
#
# This code is derived from software contributed to The NetBSD Foundation
@@ -26,172 +26,23 @@
# POSSIBILITY OF SUCH DAMAGE.
#
-test_target()
-{
- SUPPORT='n'
- if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
- SUPPORT='y'
- fi
-
- if ! echo __clang__ | cc -E - | grep -q __clang__; then
- SUPPORT='y'
- fi
-}
-
-atf_test_case int_add_overflow
-int_add_overflow_head() {
- atf_set "descr" "Test Undefined Behavior for int addition overflows"
- atf_set "require.progs" "cc"
-}
-
-atf_test_case int_add_overflow_profile
-int_add_overflow_profile_head() {
- atf_set "descr" "Test Undefined Behavior for int addition overflows with profiling option"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_add_overflow_pic
-int_add_overflow_pic_head() {
- atf_set "descr" "Test Undefined Behavior for int addition overflows with position independent code (PIC) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_add_overflow_pie
-int_add_overflow_pie_head() {
- atf_set "descr" "Test Undefined Behavior for int addition overflows with position independent execution (PIE) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_add_overflow32
-int_add_overflow32_head() {
- atf_set "descr" "Test Undefined Behavior for int addition overflows in NetBSD_32 emulation"
- atf_set "require.progs" "cc file diff cat"
-}
-
-
-int_add_overflow_body(){
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MAX; l+=argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test test.c
- atf_check -e match:"signed integer overflow" ./test
-}
-
-int_add_overflow_profile_body(){
- cat > test.c << EOF
+UBSAN_CODE='
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MAX; l+=argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -pg test.c
- atf_check -e match:"signed integer overflow" ./test
-}
-
-int_add_overflow_pic_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
int help(int);
-int main(int argc, char **argv) {volatile int k = help(argc); return k;}
-EOF
-
- cat > pic.c << EOF
-#include <stdlib.h>
-#include <stdio.h>
-#include <limits.h>
+#ifndef PIC_MAIN
int help(int count) {volatile int l = INT_MAX; l+= count; return l;}
-EOF
-
- cc -fsanitize=undefined -fPIC -shared -o libtest.so pic.c
- cc -o test test.c -fsanitize=undefined -L. -ltest
-
- export LD_LIBRARY_PATH=.
- atf_check -e match:"signed integer overflow" ./test
-}
-
-int_add_overflow_pie_body(){
-
- #check whether -pie flag is supported on this architecture
- if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
- atf_set_skip "cc -pie not supported on this architecture"
- fi
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MAX; l+= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -fpie -pie test.c
- atf_check -e match:"signed integer overflow" ./test
-}
-
-
-int_add_overflow32_body(){
-
- # check what this architecture is, after all
- if ! cc -dM -E - < /dev/null | grep -F -q _LP64; then
- atf_skip "This is not a 64 bit architecture"
- fi
- if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
- atf_skip "cc -m32 Not supported on this architecture"
- else
- if grep -F -q _LP64 ./def32; then
- atf_fail "cc -m32 Does not generate NetBSD32 binaries"
- fi
- fi
-
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MAX; l+= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o md32 -m32 test.c
- cc -fsanitize=undefined -o md64 test.c
- file -b ./md32 > ./ftype32
- file -b ./md64 > ./ftype64
- if diff ./ftype32 ./ftype64 >/dev/null; then
- atf_fail "Generated 32bit binaries do not differ from 64bit ones"
- fi
- echo "32bit binaries on this platform are:"
- cat ./ftype32
- echo "64bit binaries are on the other hand:"
- cat ./ftype64
- atf_check -e match:"signed integer overflow" ./md32
-
- # Another test with profile 32bit binaries, just to make sure everything has been thoroughly done
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MAX; l+= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -pg -m32 -o test test.c
- atf_check -e match:"signed integer overflow" ./test
-}
+#endif
+#ifndef PIC_FOO
+int main(int argc, char **argv) {volatile int l = INT_MAX; l+=argc; return l;}
+#endif
+'
-atf_test_case target_not_supported
-target_not_supported_head()
-{
- atf_set "descr" "Test forced skip"
-}
+ubsan_test_case int_add_overflow "int addition overflows" \
+ "signed integer overflow"
atf_init_test_cases()
{
- test_target
- test $SUPPORT = 'n' && {
- atf_add_test_case target_not_supported
- return 0
- }
- atf_add_test_case int_add_overflow
-# atf_add_test_case int_add_overflow_profile
- atf_add_test_case int_add_overflow_pie
- atf_add_test_case int_add_overflow_pic
-# atf_add_test_case int_add_overflow32
+ ubsan_add_test_cases int_add_overflow
}
Index: src/tests/usr.bin/cc/t_ubsan_int_divzero.sh
diff -u src/tests/usr.bin/cc/t_ubsan_int_divzero.sh:1.3 src/tests/usr.bin/cc/t_ubsan_int_divzero.sh:1.4
--- src/tests/usr.bin/cc/t_ubsan_int_divzero.sh:1.3 Mon Jul 16 07:25:58 2018
+++ src/tests/usr.bin/cc/t_ubsan_int_divzero.sh Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# Copyright (c) 2018 The NetBSD Foundation, Inc.
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
# All rights reserved.
#
# This code is derived from software contributed to The NetBSD Foundation
@@ -26,165 +26,21 @@
# POSSIBILITY OF SUCH DAMAGE.
#
-test_target()
-{
- SUPPORT='n'
- if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
- SUPPORT='y'
- fi
-
- if ! echo __clang__ | cc -E - | grep -q __clang__; then
- SUPPORT='y'
- fi
-}
-
-atf_test_case int_divzero
-int_divzero_head() {
- atf_set "descr" "Test Undefined Behavior for int division with zero"
- atf_set "require.progs" "cc"
-}
-
-atf_test_case int_divzero_profile
-int_divzero_profile_head() {
- atf_set "descr" "Test Undefined Behavior for int division with zero with profiling option"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_divzero_pic
-int_divzero_pic_head() {
- atf_set "descr" "Test Undefined Behavior for int division with zero with position independent code (PIC) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_divzero_pie
-int_divzero_pie_head() {
- atf_set "descr" "Test Undefined Behavior for int division with zero with position independent execution (PIE) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_divzero32
-int_divzero32_head() {
- atf_set "descr" "Test Undefined Behavior for int division with zero in NetBSD_32 emulation"
- atf_set "require.progs" "cc file diff cat"
-}
-
-
-int_divzero_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = argc; volatile int k = 0; l/= k; return l;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -o test test.c
- atf_check -s signal:8 -e match:"division by zero" ./test
-}
-
-int_divzero_profile_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = argc; volatile int k = 0; l/= k; return l;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -o test -pg test.c
- atf_check -s signal:8 -e match:"division by zero" ./test
-}
-
-int_divzero_pic_body(){
- cat > test.c << EOF
+UBSAN_CODE='
#include <stdio.h>
#include <stdlib.h>
int help(int);
-int main(int argc, char **argv) {return help(argc);}
-EOF
-
- cat > pic.c << EOF
-#include <stdlib.h>
-#include <stdio.h>
+#ifndef PIC_MAIN
int help(int count) {volatile int l = count; volatile int k = 0; return l/k;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -fPIC -shared -o libtest.so pic.c
- cc -o test test.c -fsanitize=integer-divide-by-zero -L. -ltest
-
- export LD_LIBRARY_PATH=.
- atf_check -s signal:8 -e match:"division by zero" ./test
-}
-int_divzero_pie_body(){
-
- #check whether -pie flag is supported on this architecture
- if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
- atf_set_skip "cc -pie not supported on this architecture"
- fi
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = argc; int k = 0; l/= k; return l;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -o test -fpie -pie test.c
- atf_check -s signal:8 -e match:"division by zero" ./test
-}
-
-
-int_divzero32_body(){
-
- # check what this architecture is, after all
- if ! cc -dM -E - < /dev/null | grep -F -q _LP64; then
- atf_skip "This is not a 64 bit architecture"
- fi
- if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
- atf_skip "cc -m32 Not supported on this architecture"
- else
- if grep -F -q _LP64 ./def32; then
- atf_fail "cc -m32 Does not generate NetBSD32 binaries"
- fi
- fi
-
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {int l = argc; int k = 0; l/= k; return l;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -o md32 -m32 test.c
- cc -fsanitize=integer-divide-by-zero -o md64 test.c
- file -b ./md32 > ./ftype32
- file -b ./md64 > ./ftype64
- if diff ./ftype32 ./ftype64 >/dev/null; then
- atf_fail "Generated 32bit binaries do not differ from 64bit ones"
- fi
- echo "32bit binaries on this platform are:"
- cat ./ftype32
- echo "64bit binaries are on the other hand:"
- cat ./ftype64
- atf_check -s signal:8 -e match:"division by zero" ./md32
-
- # Another test with profile 32bit binaries, just to make sure everything has been thoroughly done
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {int l = argc; int k = 0; l /= k; return l;}
-EOF
-
- cc -fsanitize=integer-divide-by-zero -pg -m32 -o test test.c
- atf_check -s signal:8 -e match:"division by zero" ./test
-}
+#endif
+#ifndef PIC_FOO
+int main(int argc, char **argv) {return help(argc);}
+#endif
+'
-atf_test_case target_not_supported
-target_not_supported_head()
-{
- atf_set "descr" "Test forced skip"
-}
+ubsan_test_case int_divzero "int division with zero" "division by zero"
atf_init_test_cases()
{
- test_target
- test $SUPPORT = 'n' && {
- atf_add_test_case target_not_supported
- return 0
- }
- atf_add_test_case int_divzero
-# atf_add_test_case int_divzero_profile
- atf_add_test_case int_divzero_pie
- atf_add_test_case int_divzero_pic
-# atf_add_test_case int_divzero32
+ ubsan_add_test_cases int_divzero
}
Index: src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh
diff -u src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh:1.3 src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh:1.4
--- src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh:1.3 Mon Jul 16 07:25:58 2018
+++ src/tests/usr.bin/cc/t_ubsan_int_neg_overflow.sh Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# Copyright (c) 2018 The NetBSD Foundation, Inc.
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
# All rights reserved.
#
# This code is derived from software contributed to The NetBSD Foundation
@@ -26,172 +26,22 @@
# POSSIBILITY OF SUCH DAMAGE.
#
-test_target()
-{
- SUPPORT='n'
- if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
- SUPPORT='y'
- fi
-
- if ! echo __clang__ | cc -E - | grep -q __clang__; then
- SUPPORT='y'
- fi
-}
-
-atf_test_case int_neg_overflow
-int_neg_overflow_head() {
- atf_set "descr" "Test Undefined Behavior for int negation overflows"
- atf_set "require.progs" "cc"
-}
-
-atf_test_case int_neg_overflow_profile
-int_neg_overflow_profile_head() {
- atf_set "descr" "Test Undefined Behavior for int negation overflows with profiling option"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_neg_overflow_pic
-int_neg_overflow_pic_head() {
- atf_set "descr" "Test Undefined Behavior for int negation overflows with position independent code (PIC) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_neg_overflow_pie
-int_neg_overflow_pie_head() {
- atf_set "descr" "Test Undefined Behavior for int negation overflows with position independent execution (PIE) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_neg_overflow32
-int_neg_overflow32_head() {
- atf_set "descr" "Test Undefined Behavior for int negation overflows in NetBSD_32 emulation"
- atf_set "require.progs" "cc file diff cat"
-}
-
-
-int_neg_overflow_body(){
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -o test test.c
- atf_check -e match:"negation of" ./test
-}
-
-int_neg_overflow_profile_body(){
- cat > test.c << EOF
+UBSAN_CODE='
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -pg test.c
- atf_check -e match:"negation of" ./test
-}
-
-int_neg_overflow_pic_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
int help(int);
-int main(int argc, char **argv) {return help(argc);}
-EOF
-
- cat > pic.c << EOF
-#include <stdlib.h>
-#include <stdio.h>
-#include <limits.h>
+#ifndef PIC_MAIN
int help(int count) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -fPIC -shared -o libtest.so pic.c
- cc -o test test.c -fsanitize=undefined -L. -ltest
-
- export LD_LIBRARY_PATH=.
- atf_check -e match:"negation of" ./test
-}
-
-int_neg_overflow_pie_body(){
-
- #check whether -pie flag is supported on this architecture
- if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
- atf_set_skip "cc -pie not supported on this architecture"
- fi
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -fpie -pie test.c
- atf_check -e match:"negation of" ./test
-}
-
-
-int_neg_overflow32_body(){
-
- # check what this architecture is, after all
- if ! cc -dM -E - < /dev/null | grep -F -q _LP64; then
- atf_skip "This is not a 64 bit architecture"
- fi
- if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
- atf_skip "cc -m32 Not supported on this architecture"
- else
- if grep -F -q _LP64 ./def32; then
- atf_fail "cc -m32 Does not generate NetBSD32 binaries"
- fi
- fi
-
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -o md32 -m32 test.c
- cc -fsanitize=undefined -o md64 test.c
- file -b ./md32 > ./ftype32
- file -b ./md64 > ./ftype64
- if diff ./ftype32 ./ftype64 >/dev/null; then
- atf_fail "Generated 32bit binaries do not differ from 64bit ones"
- fi
- echo "32bit binaries on this platform are:"
- cat ./ftype32
- echo "64bit binaries are on the other hand:"
- cat ./ftype64
- atf_check -e match:"negation of" ./md32
-
- # Another test with profile 32bit binaries, just to make sure everything has been thoroughly done
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l = -l; return l;}
-EOF
-
- cc -fsanitize=undefined -pg -m32 -o test test.c
- atf_check -e match:"negation of" ./test
-}
+#endif
+#ifndef PIC_FOO
+int main(int argc, char **argv) {return help(argc);}
+#endif
+'
-atf_test_case target_not_supported
-target_not_supported_head()
-{
- atf_set "descr" "Test forced skip"
-}
+ubsan_test_case int_neg_overflow "int negation overflows" "negation of"
atf_init_test_cases()
{
- test_target
- test $SUPPORT = 'n' && {
- atf_add_test_case target_not_supported
- return 0
- }
- atf_add_test_case int_neg_overflow
-# atf_add_test_case int_neg_overflow_profile
- atf_add_test_case int_neg_overflow_pie
- atf_add_test_case int_neg_overflow_pic
-# atf_add_test_case int_neg_overflow32
+ ubsan_add_test_cases int_neg_overflow
}
Index: src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh
diff -u src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh:1.3 src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh:1.4
--- src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh:1.3 Mon Jul 16 07:25:58 2018
+++ src/tests/usr.bin/cc/t_ubsan_int_sub_overflow.sh Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# Copyright (c) 2018 The NetBSD Foundation, Inc.
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
# All rights reserved.
#
# This code is derived from software contributed to The NetBSD Foundation
@@ -26,172 +26,23 @@
# POSSIBILITY OF SUCH DAMAGE.
#
-test_target()
-{
- SUPPORT='n'
- if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
- SUPPORT='y'
- fi
-
- if ! echo __clang__ | cc -E - | grep -q __clang__; then
- SUPPORT='y'
- fi
-}
-
-atf_test_case int_sub_overflow
-int_sub_overflow_head() {
- atf_set "descr" "Test Undefined Behavior for int subtraction overflows"
- atf_set "require.progs" "cc"
-}
-
-atf_test_case int_sub_overflow_profile
-int_sub_overflow_profile_head() {
- atf_set "descr" "Test Undefined Behavior for int subtraction overflows with profiling option"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_sub_overflow_pic
-int_sub_overflow_pic_head() {
- atf_set "descr" "Test Undefined Behavior for int subtraction overflows with position independent code (PIC) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_sub_overflow_pie
-int_sub_overflow_pie_head() {
- atf_set "descr" "Test Undefined Behavior for int subtraction overflows with position independent execution (PIE) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case int_sub_overflow32
-int_sub_overflow32_head() {
- atf_set "descr" "Test Undefined Behavior for int subtraction overflows in NetBSD_32 emulation"
- atf_set "require.progs" "cc file diff cat"
-}
-
-
-int_sub_overflow_body(){
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l-=argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test test.c
- atf_check -s ignore -e match:"signed integer overflow" ./test
-}
-
-int_sub_overflow_profile_body(){
- cat > test.c << EOF
+UBSAN_CODE='
#include <limits.h>
#include <stdio.h>
#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l-=argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -pg test.c
- atf_check -s ignore -e match:"signed integer overflow" ./test
-}
-
-int_sub_overflow_pic_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
int help(int);
-int main(int argc, char **argv) {volatile int k = help(argc); return k;}
-EOF
-
- cat > pic.c << EOF
-#include <stdlib.h>
-#include <stdio.h>
-#include <limits.h>
+#ifndef PIC_MAIN
int help(int count) {volatile int l = INT_MIN; l-= count; return l;}
-EOF
-
- cc -fsanitize=undefined -fPIC -shared -o libtest.so pic.c
- cc -o test test.c -fsanitize=undefined -L. -ltest
-
- export LD_LIBRARY_PATH=.
- atf_check -s ignore -e match:"signed integer overflow" ./test
-}
-
-int_sub_overflow_pie_body(){
-
- #check whether -pie flag is supported on this architecture
- if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
- atf_set_skip "cc -pie not supported on this architecture"
- fi
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l-= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o test -fpie -pie test.c
- atf_check -s ignore -e match:"signed integer overflow" ./test
-}
-
-
-int_sub_overflow32_body(){
-
- # check what this architecture is, after all
- if ! cc -dM -E - < /dev/null | grep -F -q _LP64; then
- atf_skip "This is not a 64 bit architecture"
- fi
- if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
- atf_skip "cc -m32 Not supported on this architecture"
- else
- if grep -F -q _LP64 ./def32; then
- atf_fail "cc -m32 Does not generate NetBSD32 binaries"
- fi
- fi
-
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l-= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -o md32 -m32 test.c
- cc -fsanitize=undefined -o md64 test.c
- file -b ./md32 > ./ftype32
- file -b ./md64 > ./ftype64
- if diff ./ftype32 ./ftype64 >/dev/null; then
- atf_fail "Generated 32bit binaries do not differ from 64bit ones"
- fi
- echo "32bit binaries on this platform are:"
- cat ./ftype32
- echo "64bit binaries are on the other hand:"
- cat ./ftype64
- atf_check -s ignore -e match:"signed integer overflow" ./md32
-
- # Another test with profile 32bit binaries, just to make sure everything has been thoroughly done
- cat > test.c << EOF
-#include <limits.h>
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int l = INT_MIN; l-= argc; return l;}
-EOF
-
- cc -fsanitize=undefined -pg -m32 -o test test.c
- atf_check -s ignore -e match:"signed integer overflow" ./test
-}
+#endif
+#ifndef PIC_FOO
+int main(int argc, char **argv) {volatile int k = help(argc); return k;}
+#endif
+'
-atf_test_case target_not_supported
-target_not_supported_head()
-{
- atf_set "descr" "Test forced skip"
-}
+ubsan_test_case int_sub_overflow "int subtraction overflows" \
+ "signed integer overflow"
atf_init_test_cases()
{
- test_target
- test $SUPPORT = 'n' && {
- atf_add_test_case target_not_supported
- return 0
- }
- atf_add_test_case int_sub_overflow
-# atf_add_test_case int_sub_overflow_profile
- atf_add_test_case int_sub_overflow_pie
- atf_add_test_case int_sub_overflow_pic
-# atf_add_test_case int_sub_overflow32
+ ubsan_add_test_cases int_sub_overflow
}
Index: src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh
diff -u src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh:1.4 src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh:1.5
--- src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh:1.4 Mon Jul 16 07:25:58 2018
+++ src/tests/usr.bin/cc/t_ubsan_vla_out_of_bounds.sh Tue Jan 29 19:59:10 2019
@@ -1,4 +1,4 @@
-# Copyright (c) 2018 The NetBSD Foundation, Inc.
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
# All rights reserved.
#
# This code is derived from software contributed to The NetBSD Foundation
@@ -26,169 +26,22 @@
# POSSIBILITY OF SUCH DAMAGE.
#
-test_target()
-{
- SUPPORT='n'
- if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
- SUPPORT='y'
- fi
-
- if ! echo __clang__ | cc -E - | grep -q __clang__; then
- SUPPORT='y'
- fi
-}
-
-atf_test_case vla_out_of_bounds
-vla_out_of_bounds_head() {
- atf_set "descr" "Test Undefined Behavior for vla (Variable Length Array) out of bounds"
- atf_set "require.progs" "cc"
-}
-
-atf_test_case vla_out_of_bounds_profile
-vla_out_of_bounds_profile_head() {
- atf_set "descr" "Test Undefined Behavior for vla (Variable Length Array) out of bounds with profiling option"
- atf_set "require.progs" "cc"
-}
-atf_test_case vla_out_of_bounds_pic
-vla_out_of_bounds_pic_head() {
- atf_set "descr" "Test Undefined Behavior for vla (Variable Length Array) out of bounds with position independent code (PIC) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case vla_out_of_bounds_pie
-vla_out_of_bounds_pie_head() {
- atf_set "descr" "Test Undefined Behavior for vla (Variable Length Array) out of bounds with position independent execution (PIE) flag"
- atf_set "require.progs" "cc"
-}
-atf_test_case vla_out_of_bounds32
-vla_out_of_bounds32_head() {
- atf_set "descr" "Test Undefined Behavior for vla (Variable Length Array) out of bounds in NetBSD_32 emulation"
- atf_set "require.progs" "cc file diff cat"
-}
-
-
-vla_out_of_bounds_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int val1 = argc, val2 = argc+1; volatile int arr[val1]; arr[val2] = argc; exit(0);}
-EOF
-
- cc -fsanitize=undefined -o test test.c
- atf_check -e match:"out of bounds" ./test
-}
-
-vla_out_of_bounds_profile_body(){
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int val1 = argc, val2 = argc+1; volatile int arr[val1]; arr[val2] = argc; exit(0);}
-EOF
-
- cc -fsanitize=undefined -o test -pg test.c
- atf_check -e match:"out of bounds" ./test
-}
-
-vla_out_of_bounds_pic_body(){
- cat > test.c << EOF
+UBSAN_CODE='
#include <stdio.h>
#include <stdlib.h>
void help(int);
-int main(int argc, char **argv) {help(argc); exit(0);}
-EOF
-
- cat > pic.c << EOF
-#include <stdlib.h>
-#include <stdio.h>
-#include <limits.h>
+#ifndef PIC_MAIN
void help(int count) {volatile int val1 = count, val2 = count+1; volatile int arr[val1]; arr[val2] = count; }
-EOF
-
- cc -fsanitize=undefined -fPIC -shared -o libtest.so pic.c
- cc -o test test.c -fsanitize=undefined -L. -ltest
-
- export LD_LIBRARY_PATH=.
- atf_check -e match:"out of bounds" ./test
-}
-
-vla_out_of_bounds_pie_body(){
-
- #check whether -pie flag is supported on this architecture
- if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
- atf_set_skip "cc -pie not supported on this architecture"
- fi
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-
-int main(int argc, char **argv) {volatile int val1 = argc, val2 = argc+1; volatile int arr[val1]; arr[val2] = argc; exit(0);}
-EOF
-
- cc -fsanitize=undefined -o test -fpie -pie test.c
- atf_check -e match:"out of bounds" ./test
-}
-
-
-vla_out_of_bounds32_body(){
-
- # check what this architecture is, after all
- if ! cc -dM -E - < /dev/null | grep -F -q _LP64; then
- atf_skip "This is not a 64 bit architecture"
- fi
- if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
- atf_skip "cc -m32 Not supported on this architecture"
- else
- if grep -F -q _LP64 ./def32; then
- atf_fail "cc -m32 Does not generate NetBSD32 binaries"
- fi
- fi
-
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-int main(int argc, char **argv) {volatile int val1 = argc, val2 = argc+1; volatile int arr[val1]; arr[val2] = argc; exit(0);}
-EOF
-
- cc -fsanitize=undefined -o md32 -m32 test.c
- cc -fsanitize=undefined -o md64 test.c
- file -b ./md32 > ./ftype32
- file -b ./md64 > ./ftype64
- if diff ./ftype32 ./ftype64 >/dev/null; then
- atf_fail "Generated 32bit binaries do not differ from 64bit ones"
- fi
- echo "32bit binaries on this platform are:"
- cat ./ftype32
- echo "64bit binaries are on the other hand:"
- cat ./ftype64
- atf_check -e match:"out of bounds" ./md32
-
- # Another test with profile 32bit binaries, just to make sure everything has been thoroughly done
- cat > test.c << EOF
-#include <stdio.h>
-#include <stdlib.h>
-
-int main(int argc, char **argv) { volatile int val1 = argc, val2 = argc+1; volatile int arr[val1]; arr[val2] = argc; exit(0);}
-EOF
-
- cc -fsanitize=undefined -pg -m32 -o test test.c
- atf_check -e match:"out of bounds" ./test
-}
+#endif
+#ifndef PIC_FOO
+int main(int argc, char **argv) {help(argc); exit(0);}
+#endif
+'
-atf_test_case target_not_supported
-target_not_supported_head()
-{
- atf_set "descr" "Test forced skip"
-}
+ubsan_test_case vla_out_of_bounds \
+ "vla (Variable Length Array) out of bounds" "out of bounds"
atf_init_test_cases()
{
- test_target
- test $SUPPORT = 'n' && {
- atf_add_test_case target_not_supported
- return 0
- }
- atf_add_test_case vla_out_of_bounds
-# atf_add_test_case vla_out_of_bounds_profile
- atf_add_test_case vla_out_of_bounds_pie
- atf_add_test_case vla_out_of_bounds_pic
-# atf_add_test_case vla_out_of_bounds32
+ ubsan_add_test_cases vla_out_of_bounds
}
Added files:
Index: src/tests/usr.bin/cc/ubsan_common.subr
diff -u /dev/null src/tests/usr.bin/cc/ubsan_common.subr:1.1
--- /dev/null Tue Jan 29 19:59:10 2019
+++ src/tests/usr.bin/cc/ubsan_common.subr Tue Jan 29 19:59:10 2019
@@ -0,0 +1,162 @@
+# $NetBSD: ubsan_common.subr,v 1.1 2019/01/29 19:59:10 mgorny Exp $
+#
+# Copyright (c) 2018, 2019 The NetBSD Foundation, Inc.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions
+# are met:
+# 1. Redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer.
+# 2. Redistributions in binary form must reproduce the above copyright
+# notice, this list of conditions and the following disclaimer in the
+# documentation and/or other materials provided with the distribution.
+#
+# THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
+# ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+# TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+# PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+# BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+# CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+# POSSIBILITY OF SUCH DAMAGE.
+#
+
+test_target()
+{
+ SUPPORT='n'
+ if ! echo __GNUC__ | cc -E - | grep -q __GNUC__; then
+ SUPPORT='y'
+ fi
+
+ if ! echo __clang__ | cc -E - | grep -q __clang__; then
+ SUPPORT='y'
+ fi
+}
+
+atf_test_case target_not_supported
+target_not_supported_head()
+{
+ atf_set "descr" "Test forced skip"
+}
+
+target_not_supported_body()
+{
+ atf_skip "Target is not supported"
+}
+
+# Add a new test case, with head & body.
+# asan_test_case <test-name> <description> <check-output>
+ubsan_test_case() {
+ atf_test_case "$1"
+ eval "$1_head() {
+ atf_set 'descr' 'Test Undefined Behavior for $2'
+ atf_set 'require.progs' 'cc'
+ }"
+
+ atf_test_case "$1_profile"
+ eval "$1_head() {
+ atf_set 'descr' 'Test Undefined Behavior for $2 with profiling option'
+ atf_set 'require.progs' 'cc'
+ }"
+
+ atf_test_case "$1_pic"
+ eval "$1_head() {
+ atf_set 'descr' 'Test Undefined Behavior for $2 with position independent code (PIC) flag'
+ atf_set 'require.progs' 'cc'
+ }"
+
+ atf_test_case "$1_pie"
+ eval "$1_head() {
+ atf_set 'descr' 'Test Undefined Behavior for $2 with position independent execution (PIE) flag'
+ atf_set 'require.progs' 'cc'
+ }"
+
+ atf_test_case "${1}32"
+ eval "$1_head() {
+ atf_set 'descr' 'Test Undefined Behavior for $2 in NetBSD_32 emulation'
+ atf_set 'require.progs' 'cc file diff cat'
+ }"
+
+ eval "$1_body() {
+ echo \"\$UBSAN_CODE\" > test.c
+ cc -fsanitize=undefined -o test test.c
+ # note: ignoring exit status due to inconsistency between gcc/clang
+ # (and between individual tests)
+ atf_check -s ignore -e match:'$3' ./test
+ }
+
+ $1_profile_body() {
+ echo \"\$UBSAN_CODE\" > test.c
+ cc -fsanitize=undefined -o test -pg test.c
+ atf_check -s ignore -e match:'$3' ./test
+ }
+
+ $1_pic_body() {
+ echo \"\$UBSAN_CODE\" > test.c
+ cc -DPIC_FOO -fsanitize=undefined -fPIC -shared -o libtest.so test.c
+ cc -DPIC_MAIN -o test test.c -fsanitize=undefined -L. -ltest
+
+ export LD_LIBRARY_PATH=.
+ atf_check -s ignore -e match:'$3' ./test
+ }
+
+ $1_pie_body() {
+ # check whether this arch supports -pice
+ if ! cc -pie -dM -E - < /dev/null 2>/dev/null >/dev/null; then
+ atf_set_skip 'cc -pie not supported on this architecture'
+ fi
+ echo \"\$UBSAN_CODE\" > test.c
+ cc -fsanitize=undefined -o test -fpie -pie test.c
+ atf_check -s ignore -e match:'$3' ./test
+ }
+
+ ${1}32_body() {
+ # check whether this arch is 64bit
+ if ! cc -dM -E - < /dev/null | fgrep -q _LP64; then
+ atf_skip 'this is not a 64 bit architecture'
+ fi
+ if ! cc -m32 -dM -E - < /dev/null 2>/dev/null > ./def32; then
+ atf_skip 'cc -m32 not supported on this architecture'
+ else
+ if fgrep -q _LP64 ./def32; then
+ atf_fail 'cc -m32 does not generate netbsd32 binaries'
+ fi
+ fi
+
+ echo \"\$UBSAN_CODE\" > test.c
+ cc -fsanitize=undefined -o df32 -m32 test.c
+ cc -fsanitize=undefined -o df64 test.c
+ file -b ./df32 > ./ftype32
+ file -b ./df64 > ./ftype64
+ if diff ./ftype32 ./ftype64 >/dev/null; then
+ atf_fail 'generated binaries do not differ'
+ fi
+ echo '32bit binaries on this platform are:'
+ cat ./ftype32
+ echo 'While native (64bit) binaries are:'
+ cat ./ftype64
+ atf_check -s ignore -e match:'$3' ./df32
+
+# and another test with profile 32bit binaries
+ cc -fsanitize=undefined -o test -pg -m32 test.c
+ atf_check -s ignore -e match:'$3' ./test
+ }"
+}
+
+ubsan_add_test_cases() {
+ test_target
+ test $SUPPORT = 'n' && {
+ atf_add_test_case target_not_supported
+ return 0
+ }
+
+ atf_add_test_case "$1"
+# atf_add_test_case "$1_profile"
+ atf_add_test_case "$1_pic"
+ atf_add_test_case "$1_pie"
+# atf_add_test_case "${1}32"
+}