This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Don't intercept LFS64 symbols on musl
ClosedPublic

Authored by MaskRay on Jan 7 2023, 12:21 AM.

Details

Summary

LFS64 symbols in musl are for glibc-ABI-compat and not intended for linking
(correct usage will not create LFS64 references). The next release 1.2.4 will
disallow linking against LFS64 symbols[1].

For sanitizers, let's just remove LFS64 interceptors. In case of erroneous LFS64
references, asan/tsan will detect fewer problems and msan may have false
positives.

[1]: https://git.musl-libc.org/cgit/musl/commit/?id=246f1c811448f37a44b41cd8df8d0ef9736d95f4

Diff Detail

Event Timeline

MaskRay created this revision.Jan 7 2023, 12:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 7 2023, 12:21 AM
MaskRay requested review of this revision.Jan 7 2023, 12:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 7 2023, 12:21 AM
MaskRay updated this revision to Diff 487065.Jan 7 2023, 12:29 AM

fix statfs

thesamesam accepted this revision.Jan 7 2023, 12:36 AM
thesamesam added a reviewer: raj.khem.
This revision is now accepted and ready to land.Jan 7 2023, 12:37 AM
This revision was landed with ongoing or failed builds.Jan 9 2023, 12:43 PM
This revision was automatically updated to reflect the committed changes.