diff options
| author | Rainer Orth <ro@gcc.gnu.org> | 2024-07-30 08:59:12 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-30 08:59:12 +0200 |
| commit | 94394ca980f8ecbd845155d2170cfd865e4d62dc (patch) | |
| tree | 5599e9be2d7066a7c7a5471e52e6b1a1d8ceb300 | |
| parent | 1c53b907bd6348138a59da270836fc9b4c161a07 (diff) | |
| download | llvm-94394ca980f8ecbd845155d2170cfd865e4d62dc.zip llvm-94394ca980f8ecbd845155d2170cfd865e4d62dc.tar.gz llvm-94394ca980f8ecbd845155d2170cfd865e4d62dc.tar.bz2 | |
[sanitizer_common] Fix signal_line.cpp on SPARC (#100535)
```
SanitizerCommon-ubsan-sparc-Linux :: Linux/signal_line.cpp
```
currently `FAIL`s on Linux/sparc64 (32 and 64-bit) for `n == 2`. Instead
of the expected `SIGSEGV`, the test dies with `SIGBUS`. `strace` reveals
that this is due to a unaligned access:
```
--- SIGBUS {si_signo=SIGBUS, si_code=BUS_ADRALN, si_addr=0x1} ---
```
which is to be expected on a strict-alignment target like SPARC. Fixed
by changing the invalid pointer to be better aligned.
Tested on `sparc64-unknown-linux-gnu` and `x86_64-pc-linux-gnu`.
| -rw-r--r-- | compiler-rt/test/sanitizer_common/TestCases/Linux/signal_line.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/compiler-rt/test/sanitizer_common/TestCases/Linux/signal_line.cpp b/compiler-rt/test/sanitizer_common/TestCases/Linux/signal_line.cpp index 208ece3..f1afd85 100644 --- a/compiler-rt/test/sanitizer_common/TestCases/Linux/signal_line.cpp +++ b/compiler-rt/test/sanitizer_common/TestCases/Linux/signal_line.cpp @@ -20,7 +20,8 @@ int main(int argc, char **argv) { // CHECK1: SUMMARY: [[SAN]]: SEGV {{.*}}signal_line.cpp:[[@LINE-2]]:[[TAB]] in main if (n == 2) - *((volatile int *)0x1) = __LINE__; + // Allow for strict-alignment targets that require natural alignment. + *((volatile int *)0x8) = __LINE__; // CHECK2: #{{[0-9]+ .*}}main {{.*}}signal_line.cpp:[[@LINE-1]]:[[TAB:[0-9]+]] // CHECK2: SUMMARY: [[SAN]]: SEGV {{.*}}signal_line.cpp:[[@LINE-2]]:[[TAB]] in main } |