Fix a memory access violation triggeed by a fuzzed binary.
authorNick Clifton <nickc@redhat.com>
Tue, 16 Dec 2014 14:17:15 +0000 (14:17 +0000)
committerNick Clifton <nickc@redhat.com>
Tue, 16 Dec 2014 14:17:15 +0000 (14:17 +0000)
PR binutils/17512
* format.c (bfd_check_format_matches): Check for a matching vector
before using match priorities.
* mach-o.c (bfd_mach_o_canonicalize_one_reloc): Fix off-by-one
errors with previous delta.

bfd/ChangeLog
bfd/format.c
bfd/mach-o.c

index 561c6031973f0be3c74a936f29df53b6acecb15c..6152f510c3809af053db549fec8e6426a78f89b7 100644 (file)
@@ -1,3 +1,11 @@
+2014-12-16  Nick Clifton  <nickc@redhat.com>
+
+       PR binutils/17512
+       * format.c (bfd_check_format_matches): Check for a matching vector
+       before using match priorities.
+       * mach-o.c (bfd_mach_o_canonicalize_one_reloc): Fix off-by-one
+       errors with previous delta.
+
 2014-12-15  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR ld/17713
index c4bc94403864d70fcc0f698afe22adc0fcdc065f..f0d1e668c77d49406b3933b66327fccaa67742c1 100644 (file)
@@ -402,7 +402,7 @@ bfd_check_format_matches (bfd *abfd, bfd_format format, char ***matching)
   /* We still have more than one equally good match, and at least some
      of the targets support match priority.  Choose the first of the
      best matches.  */
-  if (match_count > 1 && best_count != match_count)
+  if (matching_vector && match_count > 1 && best_count != match_count)
     {
       int i;
 
index 31ffa84966c2e015b824ed1fb649015b25be8514..61d60db54a2a6d93b3ff4b78ea681193b0dab064 100644 (file)
@@ -1350,7 +1350,7 @@ bfd_mach_o_canonicalize_one_reloc (bfd *abfd,
       if (reloc.r_extern)
        {
          /* PR 17512: file: 8396-1185-0.004.  */
-         if (num >= bfd_get_symcount (abfd))
+         if (bfd_get_symcount (abfd) > 0 && num > bfd_get_symcount (abfd))
            sym = bfd_und_section_ptr->symbol_ptr_ptr;
          else
            /* An external symbol number.  */
@@ -1368,7 +1368,7 @@ bfd_mach_o_canonicalize_one_reloc (bfd *abfd,
       else
         {
          /* PR 17512: file: 006-2964-0.004.  */
-         if (num >= mdata->nsects)
+         if (num > mdata->nsects)
            return -1;
          
          /* A section number.  */
@@ -1400,6 +1400,7 @@ bfd_mach_o_canonicalize_one_reloc (bfd *abfd,
 
   if (!(*bed->_bfd_mach_o_swap_reloc_in)(res, &reloc))
     return -1;
+
   return 0;
 }
 
@@ -1414,6 +1415,7 @@ bfd_mach_o_canonicalize_relocs (bfd *abfd, unsigned long filepos,
 
   /* Allocate and read relocs.  */
   native_size = count * BFD_MACH_O_RELENT_SIZE;
+
   native_relocs =
     (struct mach_o_reloc_info_external *) bfd_malloc (native_size);
   if (native_relocs == NULL)