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 296e8612468565d21e213013d9a2c288275be2d5..dacfcca16a0bf98ef7a9bc6b9c37669e1bc93f10 100644 (file)
@@ -1,3 +1,7 @@
+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
index e950e8d17e84e27c1072a880fc032377965a1c4d..8832ef4ee51048648852fe84d8d256079b2f5d4c 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);