Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5b69c06f authored by Vishwath Mohan's avatar Vishwath Mohan
Browse files

Blacklist code for CFI. 

Adds the -fsanitize-blacklist option for CFI, using the built in
blacklist at external/compiler-rt/lib/cfi/cfi_blacklist.txt.

Also refactors the CFI cflags and ldflags into ../soong/cc/makevars.go
to ensure they're consistent across Soong and make projects.

Bug: 30227045
Test: ENABLE_CFI=true m -j40 builds and boots.
Test: The blacklist prevents runtime errors that otherwise occur.
Change-Id: I0c2801ed459a3b9adeb37daff3ca212564801259
parent 22b55e1c

core/config_sanitizers.mk

+2 −2
Original line number Diff line number Diff line
@@ -159,8 +159,8 @@ ifneq ($(filter cfi,$(my_sanitize)),) 
  # LLVM is not set up to do this on a function basis, so force Thumb on the

  # entire module.

  LOCAL_ARM_MODE := thumb

  my_cflags += -flto -fsanitize-cfi-cross-dso -fvisibility=default

  my_ldflags += -flto -fsanitize-cfi-cross-dso -fsanitize=cfi -Wl,-plugin-opt,O1 -Wl,-export-dynamic-symbol=__cfi_check

  my_cflags += $(CFI_EXTRA_CFLAGS)

  my_ldflags += $(CFI_EXTRA_LDFLAGS)

  my_arflags += --plugin $(LLVM_PREBUILTS_PATH)/../lib64/LLVMgold.so

  # Workaround for b/33678192. CFI jumptables need Thumb2 codegen.  Revert when

  # Clang is updated past r290384.