1998-09-08 Jason Molenda (jsm@bugshack.cygnus.com)

[deliverable/binutils-gdb.git] / bfd / format.c

diff --git a/bfd/format.c b/bfd/format.c
index fc7945ad068a6d3ed8dd9becc8b9dab7216b5a54..fb9fdc0d160c02e8851617f787a600df74023fa7 100644 (file)
--- a/bfd/format.c
+++ b/bfd/format.c
@@ -141,8 +141,8 @@ bfd_check_format_matches (abfd, format, matching)
   if (matching)
     {
       *matching = matching_vector =
-       bfd_xmalloc_by_size_t (sizeof (char *) *
-                              (_bfd_target_vector_entries + 1));
+       (char **) bfd_xmalloc_by_size_t (sizeof (char *) *
+                                        (_bfd_target_vector_entries + 1));
       matching_vector[0] = NULL;
     }
   right_targ = 0;
@@ -177,23 +177,23 @@ bfd_check_format_matches (abfd, format, matching)
     temp = BFD_SEND_FMT (abfd, _bfd_check_format, (abfd));
     if (temp) {                                /* This format checks out as ok! */
       right_targ = temp;
-      match_count++;
       if (matching)
        {
          matching_vector[match_count] = temp->name;
-         matching_vector[match_count] = NULL;
+         matching_vector[match_count + 1] = NULL;
        }
+      match_count++;
       /* If this is the default target, accept it, even if other targets
         might match.  People who want those other targets have to set 
         the GNUTARGET variable.  */
       if (temp == bfd_default_vector[0])
        {
-         match_count = 1;
          if (matching)
            {
              matching_vector[0] = temp->name;
              matching_vector[1] = NULL;
            }
+         match_count = 1;
          break;
        }
 #ifdef GNU960