Skip to content

Commit

Permalink
[sanitizer_common] Fix forward: limit llvm#98200 to Linux only
Browse files Browse the repository at this point in the history
My patch broke Solaris: llvm#98200 (comment)
Fixing forward by limiting the change to Linux only.
  • Loading branch information
thurstond committed Jul 10, 2024
1 parent 1819323 commit 6789e1b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion compiler-rt/lib/sanitizer_common/sanitizer_linux.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,6 @@ void BlockSignals(__sanitizer_sigset_t *oldset) {
// If this signal is blocked, such calls cannot be handled and the process may
// hang.
internal_sigdelset(&set, 31);
# endif

// Don't block synchronous signals
internal_sigdelset(&set, SIGSEGV);
Expand All @@ -180,6 +179,7 @@ void BlockSignals(__sanitizer_sigset_t *oldset) {
internal_sigdelset(&set, SIGABRT);
internal_sigdelset(&set, SIGFPE);
internal_sigdelset(&set, SIGPIPE);
# endif

SetSigProcMask(&set, oldset);
}
Expand Down

0 comments on commit 6789e1b

Please sign in to comment.