Summary: | llvm-runtimes/compiler-rt-sanitizers: clang -fsanitize=memory (msan) broken w/ sys-libs/musl-1.2.4 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Sam James <sam> |
Component: | Current packages | Assignee: | LLVM support project <llvm> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | sam |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://bugs.gentoo.org/show_bug.cgi?id=905914 https://reviews.llvm.org/D150925 https://bugs.gentoo.org/show_bug.cgi?id=928936 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 903611 | ||
Attachments: | 0001-msan-Don-t-intercept-LFS-prlimit64-getrlimi64-on-mus.patch |
Description
Sam James
![]() ![]() ![]() ![]() I think we need to guard all the LFS functions with a glibc check in msan, starting at e.g. https://github.com/llvm/llvm-project/blob/ed90cf1873aa696f7c99ea2eafc540206c2391e1/compiler-rt/lib/msan/msan_interceptors.cpp#L847. I'll look at it later. Created attachment 861837 [details, diff]
0001-msan-Don-t-intercept-LFS-prlimit64-getrlimi64-on-mus.patch
I need to test this more before sending it upstream.
(In reply to Sam James from comment #2) > Created attachment 861837 [details, diff] [details, diff] > 0001-msan-Don-t-intercept-LFS-prlimit64-getrlimi64-on-mus.patch > > I need to test this more before sending it upstream. We could always add your patch conditionally musl-only ... It can't be far off the target since MaskRay approved it. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6aa6d6441b56e5053007153e39960e203f34130c commit 6aa6d6441b56e5053007153e39960e203f34130c Author: Andreas K. Hüttel <dilfridge@gentoo.org> AuthorDate: 2024-03-07 17:33:36 +0000 Commit: Andreas K. Hüttel <dilfridge@gentoo.org> CommitDate: 2024-03-07 17:34:25 +0000 profiles, musl: stable-use-mask sys-libs/compiler-rt-sanitizers[msan] Bug: https://bugs.gentoo.org/906603 Signed-off-by: Andreas K. Hüttel <dilfridge@gentoo.org> profiles/features/musl/package.use.stable.mask | 4 ++++ 1 file changed, 4 insertions(+) |