| Summary: | Make memcheck/tests/linux/sigqueue usable with musl | ||
|---|---|---|---|
| Product: | [Developer tools] valgrind | Reporter: | Stefan Maksimovic <stefan.maksimovic> |
| Component: | general | Assignee: | Julian Seward <jseward> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | mark, mips32r2 |
| Priority: | NOR | ||
| Version First Reported In: | 3.15 SVN | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: | proposed solution | ||
Did anyone have a chance to take a look at this in the meantime? Everyone OK to submit this? I think this is fine to go in. This is only in some debug code, so shouldn't impact the test. But musl is a little pedantic, they actually do support that field, they just call it differently from the glibc and linux uapi header for some reason they added an extra underscore... |
Created attachment 125733 [details] proposed solution Remove offsetof(siginfo_t, _sifields) from the test. "_sifields" isn't mandatory field of struct siginfo_t so it shouldn't be used in regular user program. There is no functional impact on test.