From 75ca6a193d34f05fee66290028321a97e0a74910 Mon Sep 17 00:00:00 2001 From: Michael Forney Date: Tue, 10 Mar 2020 16:30:20 -0700 Subject: Use public fpregset_t type to define fpstate_t The _libc_fpstate struct tag is an implementation detail of glibc, and musl uses a different struct tag, _fpstate. Unfortunately, the public type fpregset_t is a pointer type, so is not suitable for allocating storage, or referring to constant storage. Instead, we can use std::remove_pointer::type to refer to the pointed-to type, regardless of the struct tag. Bug: google-breakpad:631 Change-Id: Iaf47f15b2d834dd8de839431f65a481e9b0c7f9e Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/2096171 Reviewed-by: Mike Frysinger --- src/client/linux/dump_writer_common/ucontext_reader.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/client/linux/dump_writer_common/ucontext_reader.h') diff --git a/src/client/linux/dump_writer_common/ucontext_reader.h b/src/client/linux/dump_writer_common/ucontext_reader.h index f830618f..8e74a8a5 100644 --- a/src/client/linux/dump_writer_common/ucontext_reader.h +++ b/src/client/linux/dump_writer_common/ucontext_reader.h @@ -34,6 +34,7 @@ #include #include "client/linux/dump_writer_common/raw_context_cpu.h" +#include "client/linux/minidump_writer/minidump_writer.h" #include "common/memory_allocator.h" #include "google_breakpad/common/minidump_format.h" @@ -50,7 +51,7 @@ struct UContextReader { // info: the collection of register structures. #if defined(__i386__) || defined(__x86_64) static void FillCPUContext(RawContextCPU *out, const ucontext_t *uc, - const struct _libc_fpstate* fp); + const fpstate_t* fp); #elif defined(__aarch64__) static void FillCPUContext(RawContextCPU *out, const ucontext_t *uc, const struct fpsimd_context* fpregs); -- cgit v1.2.1