commit 37bc1904abdee1d49df54e946887cf120d8c0f3c
author Fangrui Song <i@maskray.me> Sat Nov 06 22:26:05 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Sat Nov 06 23:30:56 2021 -0700
tree 6dfefa7d81104a53ae1b8362703d49118013d16b
parent afc273f6d49f1392b2d6b744e1def0f0f256efbc

[sanitizer][aarch64] Add cast to drop reliance on the type of uc_mcontext.__reserved

https://sourceware.org/bugzilla/show_bug.cgi?id=22742
uc_mcontext.__reserved probably should not be considered user visible API but
unfortunate it is: it is the only way to access cpu states of some Linux
asm/sigcontext.h extensions. That said, the declaration may be
long double __reserved[256]; (used by musl)
instead of
unsigned char __reserved[4096] __attribute__((__aligned__(16))); (glibc)
to avoid dependency on a GNU variable attribute.

GitOrigin-RevId: 70986ea3d6aeacb5d10bfe3b75757611d4e7a379

lib/sanitizer_common/sanitizer_linux.cpp

diff --git a/lib/sanitizer_common/sanitizer_linux.cpp b/lib/sanitizer_common/sanitizer_linux.cpp
index 826c6d3..d171e91 100644
--- a/lib/sanitizer_common/sanitizer_linux.cpp
+++ b/lib/sanitizer_common/sanitizer_linux.cpp
@@ -1773,7 +1773,7 @@
 
 static bool Aarch64GetESR(ucontext_t *ucontext, u64 *esr) {
   static const u32 kEsrMagic = 0x45535201;
-  u8 *aux = ucontext->uc_mcontext.__reserved;
+  u8 *aux = reinterpret_cast<u8 *>(ucontext->uc_mcontext.__reserved);
   while (true) {
     _aarch64_ctx *ctx = (_aarch64_ctx *)aux;
     if (ctx->size == 0) break;