From: Marcin Kościelnicki Date: Fri, 22 Jan 2016 13:25:02 +0000 (+0100) Subject: gdb.trace: Fix write_inferior_data_ptr on 32-bit big-endian machines. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=99e8eb11cfcdde8cba6755ed4613c3cb079dfaa4;p=binutils-gdb.git gdb.trace: Fix write_inferior_data_ptr on 32-bit big-endian machines. Noticed and tested on 31-bit s390. This bug caused completely broken fast tracepoints. gdb/gdbserver/ChangeLog: * tracepoint.c (write_inferior_data_ptr): Cast to uintptr_t, so that it works properly on big-endian machines where sizeof (CORE_ADDR) != sizeof (void *). --- diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 9c42fcf97ff..6ddd9ce91a1 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,9 @@ +2016-01-22 Marcin Kościelnicki + + * tracepoint.c (write_inferior_data_ptr): Cast to uintptr_t, so that + it works properly on big-endian machines where sizeof (CORE_ADDR) + != sizeof (void *). + 2016-01-21 Pedro Alves * Makefile.in (COMPILER_CFLAGS, CXXFLAGS): New. diff --git a/gdb/gdbserver/tracepoint.c b/gdb/gdbserver/tracepoint.c index 40d0da93a61..0671999ef90 100644 --- a/gdb/gdbserver/tracepoint.c +++ b/gdb/gdbserver/tracepoint.c @@ -5931,14 +5931,15 @@ compile_tracepoint_condition (struct tracepoint *tpoint, *jump_entry += 16; } -/* We'll need to adjust these when we consider bi-arch setups, and big - endian machines. */ +/* We'll need to adjust these when we consider bi-arch setups. */ static int write_inferior_data_ptr (CORE_ADDR where, CORE_ADDR ptr) { + uintptr_t pptr = ptr; + return write_inferior_memory (where, - (unsigned char *) &ptr, sizeof (void *)); + (unsigned char *) &pptr, sizeof pptr); } /* The base pointer of the IPA's heap. This is the only memory the