diff options
| author | Sergio Durigan Junior <sergiodj@redhat.com> | 2017-09-10 12:16:54 -0400 |
|---|---|---|
| committer | Sergio Durigan Junior <sergiodj@redhat.com> | 2017-09-10 12:33:28 -0400 |
| commit | cc397f3a236509257acfd5cd17c2f15f87a482d7 (patch) | |
| tree | 78d0f666cad6cade770bbb59d66016ade35650d2 /gdb/gdbserver/linux-x86-low.c | |
| parent | 72542b8ee6558677bed21dd880b68dc2e541c9c7 (diff) | |
| download | gdb-cc397f3a236509257acfd5cd17c2f15f87a482d7.zip gdb-cc397f3a236509257acfd5cd17c2f15f87a482d7.tar.gz gdb-cc397f3a236509257acfd5cd17c2f15f87a482d7.tar.bz2 | |
Fix build breakage on GNU/Linux AArch64
This patch fixes the build breakage that has been happening on AArch64
since September 5th. The breakage was introduced by the following
commit:
author Yao Qi <yao.qi@linaro.org>
Tue, 5 Sep 2017 04:54:52 -0400 (09:54 +0100)
committer Yao Qi <yao.qi@linaro.org>
Tue, 5 Sep 2017 04:54:52 -0400 (09:54 +0100)
commit f7000548a2b79d7e5cb924468117ca4245e6b820
Use VEC for target_desc.reg_defs
The build log for this commit can be seen here:
<https://gdb-build.sergiodj.net/builders/Ubuntu-AArch64-native-gdbserver-m64/builds/2696/steps/compile%20gdb/logs/stdio>
And the underlying problem is that the code is not calling the new
function "allocate_target_description" to allocate the "struct
target_desc" using "new" instead of XNEW, which end up not properly
initializing the fields of the structure.
Regtested on BuildBot.
gdb/gdbserver/ChangeLog:
2017-09-10 Sergio Durigan Junior <sergiodj@redhat.com>
* linux-low.c (handle_extended_wait): Use
"allocate_target_description" instead of "XNEW".
* linux-x86-low.c (initialize_low_arch): Likewise.
Diffstat (limited to 'gdb/gdbserver/linux-x86-low.c')
| -rw-r--r-- | gdb/gdbserver/linux-x86-low.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/gdbserver/linux-x86-low.c b/gdb/gdbserver/linux-x86-low.c index f09871a..844a165 100644 --- a/gdb/gdbserver/linux-x86-low.c +++ b/gdb/gdbserver/linux-x86-low.c @@ -2900,7 +2900,7 @@ initialize_low_arch (void) { /* Initialize the Linux target descriptions. */ #ifdef __x86_64__ - tdesc_amd64_linux_no_xml = XNEW (struct target_desc); + tdesc_amd64_linux_no_xml = allocate_target_description (); copy_target_description (tdesc_amd64_linux_no_xml, amd64_linux_read_description (X86_XSTATE_SSE_MASK, false)); @@ -2911,7 +2911,7 @@ initialize_low_arch (void) initialize_low_tdesc (); #endif - tdesc_i386_linux_no_xml = XNEW (struct target_desc); + tdesc_i386_linux_no_xml = allocate_target_description (); copy_target_description (tdesc_i386_linux_no_xml, i386_linux_read_description (X86_XSTATE_SSE_MASK)); tdesc_i386_linux_no_xml->xmltarget = xmltarget_i386_linux_no_xml; |