[all-commits] [llvm/llvm-project] 6a42cb: [GWP-ASan] Add inbuilt options parser.

Mitch Phillips via All-commits all-commits at lists.llvm.org
Fri Jan 15 13:22:41 PST 2021


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 6a42cbf6d2116b52cb59aa3e23bef93a30cf2dc8
      https://github.com/llvm/llvm-project/commit/6a42cbf6d2116b52cb59aa3e23bef93a30cf2dc8
  Author: Mitch Phillips <31459023+hctim at users.noreply.github.com>
  Date:   2021-01-15 (Fri, 15 Jan 2021)

  Changed paths:
    M compiler-rt/lib/gwp_asan/CMakeLists.txt
    M compiler-rt/lib/gwp_asan/optional/options_parser.cpp
    M compiler-rt/lib/gwp_asan/optional/options_parser.h
    M compiler-rt/lib/gwp_asan/options.inc
    M compiler-rt/lib/gwp_asan/tests/CMakeLists.txt
    A compiler-rt/lib/gwp_asan/tests/options.cpp
    M compiler-rt/lib/scudo/scudo_allocator.cpp
    M compiler-rt/lib/scudo/standalone/CMakeLists.txt
    M compiler-rt/lib/scudo/standalone/combined.h
    M compiler-rt/lib/scudo/standalone/flags.cpp
    M compiler-rt/lib/scudo/standalone/tests/CMakeLists.txt
    M compiler-rt/lib/scudo/standalone/tests/flags_test.cpp
    M compiler-rt/test/scudo/standalone/unit/lit.site.cfg.py.in
    M compiler-rt/tools/gwp_asan/CMakeLists.txt
    A compiler-rt/tools/gwp_asan/options_parser_fuzzer.cpp

  Log Message:
  -----------
  [GWP-ASan] Add inbuilt options parser.

Adds a modified options parser (shamefully pulled from Scudo, which
shamefully pulled it from sanitizer-common) to GWP-ASan. This allows
customers (Android) to parse options strings in a common way.

Depends on D94117.

AOSP side of these patches is staged at:

 - sepolicy (sysprops should only be settable by the shell, in both root and
 unrooted conditions):
 https://android-review.googlesource.com/c/platform/system/sepolicy/+/1517238

 - zygote updates:
 https://android-review.googlesource.com/c/platform/frameworks/base/+/1515009

 - bionic changes to add `gwp_asan.<process_name>` system property, and
 GWP_ASAN_OPTIONS environment variable:
 https://android-review.googlesource.com/c/platform/bionic/+/1514989

Reviewed By: eugenis

Differential Revision: https://reviews.llvm.org/D92696




More information about the All-commits mailing list