Merge "Allow integer_overflow sanitizer path exclusion." am: ad9dde8a86
am: 815c1f5941
Change-Id: Ica0a1e973f7cc6794866d6317ddff39fa72edc9a
This commit is contained in:
@@ -103,15 +103,23 @@ ifneq ($(my_sanitize)$(my_global_sanitize),)
|
|||||||
endif
|
endif
|
||||||
endif
|
endif
|
||||||
|
|
||||||
# Disable integer_overflow if LOCAL_NOSANITIZE=integer.
|
|
||||||
ifneq ($(filter integer_overflow, $(my_global_sanitize) $(my_sanitize)),)
|
ifneq ($(filter integer_overflow, $(my_global_sanitize) $(my_sanitize)),)
|
||||||
|
# Disable integer_overflow in excluded paths.
|
||||||
|
combined_exclude_paths := $(INTEGER_OVERFLOW_EXCLUDE_PATHS) \
|
||||||
|
$(PRODUCT_INTEGER_OVERFLOW_EXCLUDE_PATHS)
|
||||||
|
|
||||||
|
ifneq ($(strip $(foreach dir,$(subst $(comma),$(space),$(combined_exclude_paths)),\
|
||||||
|
$(filter $(dir)%,$(LOCAL_PATH)))),)
|
||||||
|
my_sanitize := $(filter-out integer_overflow,$(my_sanitize))
|
||||||
|
my_sanitize_diag := $(filter-out integer_overflow,$(my_sanitize_diag))
|
||||||
|
endif
|
||||||
|
# Disable integer_overflow if LOCAL_NOSANITIZE=integer.
|
||||||
ifneq ($(filter integer, $(strip $(LOCAL_NOSANITIZE))),)
|
ifneq ($(filter integer, $(strip $(LOCAL_NOSANITIZE))),)
|
||||||
my_sanitize := $(filter-out integer_overflow,$(my_sanitize))
|
my_sanitize := $(filter-out integer_overflow,$(my_sanitize))
|
||||||
my_sanitize_diag := $(filter-out integer_overflow,$(my_sanitize_diag))
|
my_sanitize_diag := $(filter-out integer_overflow,$(my_sanitize_diag))
|
||||||
endif
|
endif
|
||||||
endif
|
endif
|
||||||
|
|
||||||
|
|
||||||
my_nosanitize = $(strip $(LOCAL_NOSANITIZE))
|
my_nosanitize = $(strip $(LOCAL_NOSANITIZE))
|
||||||
ifneq ($(my_nosanitize),)
|
ifneq ($(my_nosanitize),)
|
||||||
my_sanitize := $(filter-out $(my_nosanitize),$(my_sanitize))
|
my_sanitize := $(filter-out $(my_nosanitize),$(my_sanitize))
|
||||||
|
@@ -135,6 +135,7 @@ _product_var_list := \
|
|||||||
PRODUCT_IOT \
|
PRODUCT_IOT \
|
||||||
PRODUCT_SYSTEM_HEADROOM \
|
PRODUCT_SYSTEM_HEADROOM \
|
||||||
PRODUCT_MINIMIZE_JAVA_DEBUG_INFO \
|
PRODUCT_MINIMIZE_JAVA_DEBUG_INFO \
|
||||||
|
PRODUCT_INTEGER_OVERFLOW_EXCLUDE_PATHS \
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
@@ -431,3 +431,7 @@ PRODUCT_SYSTEM_HEADROOM := \
|
|||||||
# Whether to save disk space by minimizing java debug info
|
# Whether to save disk space by minimizing java debug info
|
||||||
PRODUCT_MINIMIZE_JAVA_DEBUG_INFO := \
|
PRODUCT_MINIMIZE_JAVA_DEBUG_INFO := \
|
||||||
$(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_MINIMIZE_JAVA_DEBUG_INFO))
|
$(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_MINIMIZE_JAVA_DEBUG_INFO))
|
||||||
|
|
||||||
|
# Whether any paths are excluded from sanitization when SANITIZE_TARGET=integer_overflow
|
||||||
|
PRODUCT_INTEGER_OVERFLOW_EXCLUDE_PATHS := \
|
||||||
|
$(strip $(PRODUCTS.$(INTERNAL_PRODUCT).PRODUCT_INTEGER_OVERFLOW_EXCLUDE_PATHS))
|
||||||
|
@@ -79,6 +79,7 @@ $(SOONG_VARIABLES): FORCE
|
|||||||
echo ' "CrossHostSecondaryArch": "$(HOST_CROSS_2ND_ARCH)",'; \
|
echo ' "CrossHostSecondaryArch": "$(HOST_CROSS_2ND_ARCH)",'; \
|
||||||
echo ' "Safestack": $(if $(filter true,$(USE_SAFESTACK)),true,false),'; \
|
echo ' "Safestack": $(if $(filter true,$(USE_SAFESTACK)),true,false),'; \
|
||||||
echo ' "EnableCFI": $(if $(filter false,$(ENABLE_CFI)),false,true),'; \
|
echo ' "EnableCFI": $(if $(filter false,$(ENABLE_CFI)),false,true),'; \
|
||||||
|
echo ' "IntegerOverflowExcludePaths": $(call json_list,$(INTEGER_OVERFLOW_EXCLUDE_PATHS) $(PRODUCT_INTEGER_OVERFLOW_EXCLUDE_PATHS)),'; \
|
||||||
echo ' "Device_uses_hwc2": $(if $(filter true,$(TARGET_USES_HWC2)),true,false),'; \
|
echo ' "Device_uses_hwc2": $(if $(filter true,$(TARGET_USES_HWC2)),true,false),'; \
|
||||||
echo ' "Override_rs_driver": "$(OVERRIDE_RS_DRIVER)",'; \
|
echo ' "Override_rs_driver": "$(OVERRIDE_RS_DRIVER)",'; \
|
||||||
echo ' "Treble": $(if $(filter true,$(PRODUCT_FULL_TREBLE)),true,false),'; \
|
echo ' "Treble": $(if $(filter true,$(PRODUCT_FULL_TREBLE)),true,false),'; \
|
||||||
|
Reference in New Issue
Block a user