Reenable sanitize (#4384)

* turn on sanitize, fix overrun

* allow sanitize override, disable it for shared lib

* probably fix android build

.github/workflows/build-android.yaml

@@ -24,7 +24,7 @@ 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
+        run: make -j4 SANITIZE=no build/lib_rusefi_test
 
       - name: Build Android
         env:

.github/workflows/build-rusEFI-console.yaml

@@ -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?

unit_tests/rules.mk

@@ -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:
 

unit_tests/unit_test_rules.mk

@@ -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