Aarch64 SVE: Fix stack smashing when calling functions
authorAlan Hayward <alan.hayward@arm.com>
Mon, 17 Sep 2018 14:28:53 +0000 (15:28 +0100)
committerAlan Hayward <alan.hayward@arm.com>
Tue, 18 Sep 2018 15:24:27 +0000 (16:24 +0100)
Using "call" on a function that passes arguments via float registers can cause
gdb to overflow buffers.

Ensure enough memory is reserved to hold a full FP register.

This fixes gdb.base/callfuncs.exp for Aarch64 SVE.

2018-09-18  Alan Hayward  <alan.hayward@arm.com>

* aarch64-tdep.c (pass_in_v): Use register size.
(aarch64_extract_return_value): Likewise.
(aarch64_store_return_value): Likewise.

gdb/ChangeLog
gdb/aarch64-tdep.c

index ddd2782d99c0893350c79d85f6e6a2eed4adb74b..790cd3b1eed0e1582306a7e95a3b234657446866 100644 (file)
@@ -1,3 +1,9 @@
+2018-09-18  Alan Hayward  <alan.hayward@arm.com>
+
+       * aarch64-tdep.c (pass_in_v): Use register size.
+       (aarch64_extract_return_value): Likewise.
+       (aarch64_store_return_value): Likewise.
+
 2018-09-18  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
        * utils.c (dump_core) [HAVE_SETRLIMIT]: Cast RLIM_INFINITY to
index d2e6ac64d54c30c02bb42a48a79b086dbaaf2717..90b6deb0eade924ec1297d72f1b454df7c5957bb 100644 (file)
@@ -1366,7 +1366,9 @@ pass_in_v (struct gdbarch *gdbarch,
   if (info->nsrn < 8)
     {
       int regnum = AARCH64_V0_REGNUM + info->nsrn;
-      gdb_byte reg[V_REGISTER_SIZE];
+      /* Enough space for a full vector register.  */
+      gdb_byte reg[register_size (gdbarch, regnum)];
+      gdb_assert (len <= sizeof (reg));
 
       info->argnum++;
       info->nsrn++;
@@ -1937,7 +1939,9 @@ aarch64_extract_return_value (struct type *type, struct regcache *regs,
       for (int i = 0; i < elements; i++)
        {
          int regno = AARCH64_V0_REGNUM + i;
-         bfd_byte buf[V_REGISTER_SIZE];
+         /* Enough space for a full vector register.  */
+         gdb_byte buf[register_size (gdbarch, regno)];
+         gdb_assert (len <= sizeof (buf));
 
          if (aarch64_debug)
            {
@@ -2047,7 +2051,9 @@ aarch64_store_return_value (struct type *type, struct regcache *regs,
       for (int i = 0; i < elements; i++)
        {
          int regno = AARCH64_V0_REGNUM + i;
-         bfd_byte tmpbuf[V_REGISTER_SIZE];
+         /* Enough space for a full vector register.  */
+         gdb_byte tmpbuf[register_size (gdbarch, regno)];
+         gdb_assert (len <= sizeof (tmpbuf));
 
          if (aarch64_debug)
            {