gdb/linux-record: Fix msghdr parsing on 64-bit targets
authorMarcin Kościelnicki <koriakin@0x04.net>
Fri, 30 Oct 2015 15:51:58 +0000 (15:51 +0000)
committerPedro Alves <palves@redhat.com>
Fri, 30 Oct 2015 15:51:58 +0000 (15:51 +0000)
The code failed to account for padding between the int and subsequent
pointer present on 64-bit architectures.

gdb/ChangeLog:

* linux-record.c (record_linux_msghdr): Fix msg_namelen handling.

gdb/ChangeLog
gdb/linux-record.c

index 296e861..dacfcca 100644 (file)
@@ -1,5 +1,9 @@
 2015-10-30  Marcin Kościelnicki  <koriakin@0x04.net>
 
+       * linux-record.c (record_linux_msghdr): Fix msg_namelen handling.
+
+2015-10-30  Marcin Kościelnicki  <koriakin@0x04.net>
+
        * aarch64-linux-tdep.c (aarch64_linux_init_abi): Remove
        size_dirent{,64}, add size_old_dirent.
        * amd64-linux-tdep.c (amd64_linux_init_abi): Remove size_dirent{,64},
index e950e8d..8832ef4 100644 (file)
@@ -162,7 +162,9 @@ record_linux_msghdr (struct regcache *regcache,
                                       tdep->size_int,
                                       byte_order)))
     return -1;
-  a += tdep->size_int;
+  /* We have read an int, but skip size_pointer bytes to account for alignment
+     of the next field on 64-bit targets. */
+  a += tdep->size_pointer;
 
   /* msg_iov msg_iovlen */
   addr = extract_unsigned_integer (a, tdep->size_pointer, byte_order);