[all-commits] [llvm/llvm-project] 25fd36: [sanitizer_common] AND signals in BlockSignals ins...

[Thurston Dang via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 25fd366d6a7d40266ff27c134ed8beb0a90cc33b
      https://github.com/llvm/llvm-project/commit/25fd366d6a7d40266ff27c134ed8beb0a90cc33b
  Author: Thurston Dang <thurston at google.com>
  Date:   2024-10-31 (Thu, 31 Oct 2024)

  Changed paths:
    M compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
    M compiler-rt/lib/sanitizer_common/tests/CMakeLists.txt
    A compiler-rt/lib/sanitizer_common/tests/sanitizer_block_signals.cpp

  Log Message:
  -----------
  [sanitizer_common] AND signals in BlockSignals instead of deleting (#113443)

My earlier patch https://github.com/llvm/llvm-project/pull/98200 caused a regression because it unconditionally unblocked synchronous signals, even if the user program had deliberately blocked them. This patch fixes the issue by checking the current signal mask, as suggested by Vitaly. It also adds tests.

Fixes #113385

---------

Co-authored-by: Vitaly Buka <vitalybuka at gmail.com>



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