[all-commits] [llvm/llvm-project] 7adb7a: [GWP-ASan] Various test fixes. (#94938)

[Mitch Phillips via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7adb7aa494247f2492f6207289ad90cb48807517
      https://github.com/llvm/llvm-project/commit/7adb7aa494247f2492f6207289ad90cb48807517
  Author: Mitch Phillips <31459023+hctim at users.noreply.github.com>
  Date:   2024-06-13 (Thu, 13 Jun 2024)

  Changed paths:
    M compiler-rt/lib/gwp_asan/guarded_pool_allocator.cpp
    M compiler-rt/lib/gwp_asan/optional/segv_handler_posix.cpp
    M compiler-rt/lib/gwp_asan/tests/harness.cpp
    M compiler-rt/lib/gwp_asan/tests/harness.h
    M compiler-rt/lib/gwp_asan/tests/late_init.cpp

  Log Message:
  -----------
  [GWP-ASan] Various test fixes. (#94938)

When running some tests with --gtest_repeat=100 --gtest_shuffle, I
encountered some problems because the allocator wasn't torn down
completely, and the singleton pointer ended up pointing to a
use-after-scope'd object.

This patch has a couple of fixes and niceties:
 1. Removing the once-init stuff from tests, now that it's implicitly
    done in GuardedPoolAllocator::installAtFork() anyway.
 2. Calling uninitTestOnly() in the late_init test.
 3. Resetting the HasReportedBadPoolAccess when the signal handlers are
    installed (allowing for --gtest_repeat w/ recoverable mode).
 4. Adding a check and resetting the singleton pointer in
    uninitTestOnly().



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications