Reenable sanitize (#4365)

* turn on sanitize, fix overrun

* allow sanitize override, disable it for shared lib
This commit is contained in:
Matthew Kennedy 2022-07-21 22:19:54 -07:00 committed by GitHub
parent c5d32ad831
commit d87be38401
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 10 additions and 8 deletions

View File

@ -31,8 +31,8 @@ jobs:
- name: Build Native Unit Tests as shared library
working-directory: ./unit_tests/
# we have a unit test of JNI thus we need to build shared library
run: make -j4
# we have a unit test of JNI thus we need to build shared library
run: make -j4 SANITIZE=no build/lib_rusefi_test
- name: Test console
# at the moment 'jar' task does not depend on tests?! maybe because tests take some time?

View File

@ -89,7 +89,7 @@ VPATH = $(SRCPATHS)
# Makefile rules
#
all: $(OBJS) $(BINARY_OUTPUT) $(SHARED_OUTPUT) MAKE_ALL_RULE_HOOK
all: $(OBJS) $(BINARY_OUTPUT) MAKE_ALL_RULE_HOOK
MAKE_ALL_RULE_HOOK:

View File

@ -14,11 +14,13 @@ PCHSUB = unit_tests
include $(PROJECT_DIR)/rusefi_rules.mk
ifneq ($(OS),Windows_NT)
# at the moment lib asan breaks JNI library
SANITIZE = no
else
SANITIZE = no
# User may want to pass in a forced value for SANITIZE
ifeq ($(SANITIZE),)
ifneq ($(OS),Windows_NT)
SANITIZE = yes
else
SANITIZE = no
endif
endif
IS_MAC = no