Simplify PR23110 PE_DEBUG_DATA size checks
authorAlan Modra <amodra@gmail.com>
Tue, 16 Oct 2018 05:59:23 +0000 (16:29 +1030)
committerAlan Modra <amodra@gmail.com>
Tue, 16 Oct 2018 06:11:57 +0000 (16:41 +1030)
The negative size check can be rolled into the "exceeds space left in
section" check if that is done using an unsigned comparison.  We know
that "addr - section->vma" is never larger than section->size since
the section is found by find_section_by_vma.

* peXXigen.c (_bfd_XX_bfd_copy_private_bfd_data_common): Simplify
PE_DEBUG_DATA size checks.

bfd/ChangeLog
bfd/peXXigen.c

index ab665ed08ec385fa070a34d6d43b13afe8606109..9fcc6fc0334de4a5ab3088805d5773106c862432 100644 (file)
@@ -1,3 +1,8 @@
+2018-10-16  Alan Modra  <amodra@gmail.com>
+
+       * peXXigen.c (_bfd_XX_bfd_copy_private_bfd_data_common): Simplify
+       PE_DEBUG_DATA size checks.
+
 2018-10-16  Alan Modra  <amodra@gmail.com>
 
        PR 23781
index 1271d4edbbe54d17361f2330399eb421ad5a31c3..e0b494a2897bee767551ce1de7ce5b1823c3cbe5 100644 (file)
@@ -2984,8 +2984,8 @@ _bfd_XX_bfd_copy_private_bfd_data_common (bfd * ibfd, bfd * obfd)
            (struct external_IMAGE_DEBUG_DIRECTORY *)(data + (addr - section->vma));
 
          /* PR 17512: file: 0f15796a.  */
-         if (ope->pe_opthdr.DataDirectory[PE_DEBUG_DATA].Size + (addr - section->vma)
-             > bfd_get_section_size (section))
+         if ((unsigned long) ope->pe_opthdr.DataDirectory[PE_DEBUG_DATA].Size
+             > section->size - (addr - section->vma))
            {
              /* xgettext:c-format */
              _bfd_error_handler
@@ -2996,16 +2996,6 @@ _bfd_XX_bfd_copy_private_bfd_data_common (bfd * ibfd, bfd * obfd)
              free (data);
              return FALSE;
            }
-         /* PR 23110.  */
-         else if (ope->pe_opthdr.DataDirectory[PE_DEBUG_DATA].Size < 0)
-           {
-             /* xgettext:c-format */
-             _bfd_error_handler
-               (_("%pB: Data Directory size (%#lx) is negative"),
-                obfd, ope->pe_opthdr.DataDirectory[PE_DEBUG_DATA].Size);
-             free (data);
-             return FALSE;
-           }
 
          for (i = 0; i < ope->pe_opthdr.DataDirectory[PE_DEBUG_DATA].Size
                 / sizeof (struct external_IMAGE_DEBUG_DIRECTORY); i++)