Skip to content

Commit

Permalink
[sanitizer] [Darwin] Disable InstallAtForkHandler
Browse files Browse the repository at this point in the history
This is a followup to d01be3c.
  • Loading branch information
Azharuddin Mohammed committed Dec 18, 2023
1 parent 6c1f56f commit 2a03854
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion compiler-rt/lib/asan/asan_posix.cpp
Expand Up @@ -149,7 +149,7 @@ void PlatformTSDDtor(void *tsd) {
#endif

void InstallAtForkHandler() {
# if SANITIZER_SOLARIS || SANITIZER_NETBSD
# if SANITIZER_SOLARIS || SANITIZER_NETBSD || SANITIZER_APPLE
return; // FIXME: Implement FutexWait.
# endif
auto before = []() {
Expand Down
2 changes: 1 addition & 1 deletion compiler-rt/lib/lsan/lsan_posix.cpp
Expand Up @@ -101,7 +101,7 @@ void InstallAtExitCheckLeaks() {
}

void InstallAtForkHandler() {
# if SANITIZER_SOLARIS || SANITIZER_NETBSD
# if SANITIZER_SOLARIS || SANITIZER_NETBSD || SANITIZER_APPLE
return; // FIXME: Implement FutexWait.
# endif
auto before = []() {
Expand Down
Expand Up @@ -15,6 +15,7 @@
// FIXME: Requires `FutexWait` implementation. See __asan::InstallAtForkHandler.
// UNSUPPORTED: target={{.*solaris.*}}
// UNSUPPORTED: target={{.*netbsd.*}}
// UNSUPPORTED: target={{.*apple.*}}

// Forking in multithread environment is unsupported. However we already have
// some workarounds, and will add more, so this is the test.
Expand Down

1 comment on commit 2a03854

@vitalybuka
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is "// UNSUPPORTED: darwin" on line 4

Please sign in to comment.