arm: sanitizer stringop-overflow
authorAlan Modra <amodra@gmail.com>
Wed, 2 Aug 2023 23:30:05 +0000 (09:00 +0930)
committerAlan Modra <amodra@gmail.com>
Thu, 3 Aug 2023 11:49:57 +0000 (21:19 +0930)
In function 'memset',
    inlined from 'create_unwind_entry' at /home/alan/src/binutils-gdb/gas/config/tc-arm.c:27881:3:
/usr/include/bits/string_fortified.h:59:10: error: '__builtin_memset' specified size between 2147483652 and 4294967295 exceeds maximum object size 2147483647 [-Werror=stringop-overflow=]
   59 |   return __builtin___memset_chk (__dest, __ch, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   60 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~

* config/tc-arm.c (create_unwind_entry): Return after bad size,
and bad opcode count.

gas/config/tc-arm.c

index c0f6215fbc1bb90f4b74663ec0986a3ee9a35494..f912fb234b6fdee142fa252e0cb9073638a68d68 100644 (file)
@@ -27828,7 +27828,10 @@ create_unwind_entry (int have_data)
       if (unwind.personality_index == 0)
        {
          if (unwind.opcode_count > 3)
-           as_bad (_("too many unwind opcodes for personality routine 0"));
+           {
+             as_bad (_("too many unwind opcodes for personality routine 0"));
+             return 1;
+           }
 
          if (!have_data)
            {
@@ -27869,7 +27872,10 @@ create_unwind_entry (int have_data)
 
   size = (size + 3) >> 2;
   if (size > 0xff)
-    as_bad (_("too many unwind opcodes"));
+    {
+      as_bad (_("too many unwind opcodes"));
+      return 1;
+    }
 
   frag_align (2, 0, 0);
   record_alignment (now_seg, 2);