daily update
[deliverable/binutils-gdb.git] / bfd / format.c
index 6afcb60d6ed0b911f5efd44f634e128cd387f8fb..8cc7b7a1670b039881626143ad0ef837e30b27aa 100644 (file)
@@ -1,5 +1,5 @@
 /* Generic BFD support for file formats.
-   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1999, 2000, 2001
+   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1999, 2000, 2001, 2002
    Free Software Foundation, Inc.
    Written by Cygnus Support.
 
@@ -52,7 +52,7 @@ FUNCTION
        bfd_check_format
 
 SYNOPSIS
-       boolean bfd_check_format(bfd *abfd, bfd_format format);
+       bfd_boolean bfd_check_format(bfd *abfd, bfd_format format);
 
 DESCRIPTION
        Verify if the file attached to the BFD @var{abfd} is compatible
@@ -67,7 +67,7 @@ DESCRIPTION
        matches, it is used.  If not, exactly one target must recognize
        the file, or an error results.
 
-       The function returns <<true>> on success, otherwise <<false>>
+       The function returns <<TRUE>> on success, otherwise <<FALSE>>
        with one of the following error codes:
 
        o <<bfd_error_invalid_operation>> -
@@ -85,7 +85,7 @@ DESCRIPTION
        more than one backend recognised the file format.
 */
 
-boolean
+bfd_boolean
 bfd_check_format (abfd, format)
      bfd *abfd;
      bfd_format format;
@@ -98,10 +98,10 @@ FUNCTION
        bfd_check_format_matches
 
 SYNOPSIS
-       boolean bfd_check_format_matches(bfd *abfd, bfd_format format, char ***matching);
+       bfd_boolean bfd_check_format_matches(bfd *abfd, bfd_format format, char ***matching);
 
 DESCRIPTION
-       Like <<bfd_check_format>>, except when it returns false with
+       Like <<bfd_check_format>>, except when it returns FALSE with
        <<bfd_errno>> set to <<bfd_error_file_ambiguously_recognized>>.  In that
        case, if @var{matching} is not NULL, it will be filled in with
        a NULL-terminated list of the names of the formats that matched,
@@ -112,45 +112,48 @@ DESCRIPTION
        should free it.
 */
 
-boolean
+bfd_boolean
 bfd_check_format_matches (abfd, format, matching)
      bfd *abfd;
      bfd_format format;
      char ***matching;
 {
   extern const bfd_target binary_vec;
-  const bfd_target * const *target, *save_targ, *right_targ;
-  char **matching_vector = NULL;
+  const bfd_target * const *target;
+  const bfd_target **matching_vector = NULL;
+  const bfd_target *save_targ, *right_targ, *ar_right_targ;
   int match_count;
+  int ar_match_index;
 
-  if (!bfd_read_p (abfd) ||
-      ((int)(abfd->format) < (int)bfd_unknown) ||
-      ((int)(abfd->format) >= (int)bfd_type_end))
+  if (!bfd_read_p (abfd)
+      || (unsigned int) abfd->format >= (unsigned int) bfd_type_end)
     {
       bfd_set_error (bfd_error_invalid_operation);
-      return false;
+      return FALSE;
     }
 
   if (abfd->format != bfd_unknown)
-    return (abfd->format == format)? true: false;
+    return abfd->format == format;
 
   /* Since the target type was defaulted, check them
      all in the hope that one will be uniquely recognized.  */
   save_targ = abfd->xvec;
   match_count = 0;
+  ar_match_index = _bfd_target_vector_entries;
 
   if (matching)
     {
-      matching_vector =
-       (char **) bfd_malloc (sizeof (char *) *
-                             (_bfd_target_vector_entries + 1));
+      bfd_size_type amt;
+
+      *matching = NULL;
+      amt = sizeof (*matching_vector) * 2 * _bfd_target_vector_entries;
+      matching_vector = (const bfd_target **) bfd_malloc (amt);
       if (!matching_vector)
-       return false;
-      matching_vector[0] = NULL;
-      *matching = matching_vector;
+       return FALSE;
     }
 
   right_targ = 0;
+  ar_right_targ = 0;
 
   /* Presume the answer is yes.  */
   abfd->format = format;
@@ -158,8 +161,8 @@ bfd_check_format_matches (abfd, format, matching)
   /* If the target type was explicitly specified, just check that target.  */
   if (!abfd->target_defaulted)
     {
-      if (bfd_seek (abfd, (file_ptr)0, SEEK_SET) != 0) /* rewind! */
-       return false;
+      if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0)        /* rewind! */
+       return FALSE;
 
       right_targ = BFD_SEND_FMT (abfd, _bfd_check_format, (abfd));
 
@@ -168,9 +171,9 @@ bfd_check_format_matches (abfd, format, matching)
          abfd->xvec = right_targ;      /* Set the target as returned.  */
 
          if (matching)
-           free (matching_vector);
+           free ((PTR) matching_vector);
 
-         return true;                  /* File position has moved, BTW.  */
+         return TRUE;                  /* File position has moved, BTW.  */
        }
 
       /* For a long time the code has dropped through to check all
@@ -191,25 +194,26 @@ bfd_check_format_matches (abfd, format, matching)
          abfd->format = bfd_unknown;
 
          if (matching)
-           free (matching_vector);
+           free ((PTR) matching_vector);
 
          bfd_set_error (bfd_error_file_not_recognized);
 
-         return false;
+         return FALSE;
        }
     }
 
   for (target = bfd_target_vector; *target != NULL; target++)
     {
       const bfd_target *temp;
+      bfd_error_type err;
 
       if (*target == &binary_vec)
        continue;
 
       abfd->xvec = *target;    /* Change BFD's target temporarily */
 
-      if (bfd_seek (abfd, (file_ptr)0, SEEK_SET) != 0)
-       return false;
+      if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0)
+       return FALSE;
 
       /* If _bfd_check_format neglects to set bfd_error, assume
         bfd_error_wrong_format.  We didn't used to even pay any
@@ -223,27 +227,20 @@ bfd_check_format_matches (abfd, format, matching)
        {               /* This format checks out as ok!  */
          right_targ = temp;
 
-         if (matching)
-           {
-             matching_vector[match_count] = temp->name;
-             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])
            {
-             if (matching)
-               {
-                 matching_vector[0] = temp->name;
-                 matching_vector[1] = NULL;
-               }
              match_count = 1;
              break;
            }
+
+         if (matching)
+           matching_vector[match_count] = temp;
+
+         match_count++;
+
 #ifdef GNU960
          /* Big- and little-endian b.out archives look the same, but it
             doesn't matter: there is no difference in their headers, and
@@ -254,16 +251,67 @@ bfd_check_format_matches (abfd, format, matching)
          break;
 #endif
        }
-      else if (bfd_get_error () != bfd_error_wrong_format)
+      else if ((err = bfd_get_error ()) == bfd_error_wrong_object_format
+              || err == bfd_error_file_ambiguously_recognized)
+       {
+         /* An archive with objects of the wrong type, or an
+            ambiguous match.  We want this target to match if we get
+            no better matches.  */
+         if (ar_right_targ != bfd_default_vector[0])
+           ar_right_targ = *target;
+         if (matching)
+           matching_vector[ar_match_index] = *target;
+         ar_match_index++;
+       }
+      else if (err != bfd_error_wrong_format)
        {
          abfd->xvec = save_targ;
          abfd->format = bfd_unknown;
 
-         if (matching && bfd_get_error ()
-             != bfd_error_file_ambiguously_recognized)
-           free (matching_vector);
+         if (matching)
+           free ((PTR) matching_vector);
 
-         return false;
+         return FALSE;
+       }
+    }
+
+  if (match_count == 0)
+    {
+      /* Try partial matches.  */
+      right_targ = ar_right_targ;
+      if (right_targ == bfd_default_vector[0])
+       {
+         match_count = 1;
+       }
+      else
+       {
+         match_count = ar_match_index - _bfd_target_vector_entries;
+         if (matching && match_count > 1)
+           {
+             memcpy (matching_vector,
+                     matching_vector + _bfd_target_vector_entries,
+                     sizeof (*matching_vector) * match_count);
+           }
+       }
+    }
+
+  if (match_count > 1 && bfd_associated_vector != NULL)
+    {
+      const bfd_target * const *assoc = bfd_associated_vector;
+
+      while ((right_targ = *assoc++) != NULL)
+       {
+         int i = match_count;
+
+         while (--i >= 0)
+           if (matching_vector[i] == right_targ)
+             break;
+
+         if (i >= 0)
+           {
+             match_count = 1;
+             break;
+           }
        }
     }
 
@@ -272,9 +320,9 @@ bfd_check_format_matches (abfd, format, matching)
       abfd->xvec = right_targ;         /* Change BFD's target permanently.  */
 
       if (matching)
-       free (matching_vector);
+       free ((PTR) matching_vector);
 
-      return true;                     /* File position has moved, BTW.  */
+      return TRUE;                     /* File position has moved, BTW.  */
     }
 
   abfd->xvec = save_targ;              /* Restore original target type.  */
@@ -285,12 +333,27 @@ bfd_check_format_matches (abfd, format, matching)
       bfd_set_error (bfd_error_file_not_recognized);
 
       if (matching)
-       free (matching_vector);
+       free ((PTR) matching_vector);
     }
   else
-    bfd_set_error (bfd_error_file_ambiguously_recognized);
+    {
+      bfd_set_error (bfd_error_file_ambiguously_recognized);
+
+      if (matching)
+       {
+         *matching = (char **) matching_vector;
+         matching_vector[match_count] = NULL;
+         /* Return target names.  This is a little nasty.  Maybe we
+            should do another bfd_malloc?  */
+         while (--match_count >= 0)
+           {
+             const char *name = matching_vector[match_count]->name;
+             *(const char **) &matching_vector[match_count] = name;
+           }
+       }
+    }
 
-  return false;
+  return FALSE;
 }
 
 /*
@@ -298,7 +361,7 @@ FUNCTION
        bfd_set_format
 
 SYNOPSIS
-       boolean bfd_set_format(bfd *abfd, bfd_format format);
+       bfd_boolean bfd_set_format(bfd *abfd, bfd_format format);
 
 DESCRIPTION
        This function sets the file format of the BFD @var{abfd} to the
@@ -307,21 +370,20 @@ DESCRIPTION
        is not open for writing, then an error occurs.
 */
 
-boolean
+bfd_boolean
 bfd_set_format (abfd, format)
      bfd *abfd;
      bfd_format format;
 {
-  if (bfd_read_p (abfd) ||
-      ((int)abfd->format < (int)bfd_unknown) ||
-      ((int)abfd->format >= (int)bfd_type_end))
+  if (bfd_read_p (abfd)
+      || (unsigned int) abfd->format >= (unsigned int) bfd_type_end)
     {
       bfd_set_error (bfd_error_invalid_operation);
-      return false;
+      return FALSE;
     }
 
   if (abfd->format != bfd_unknown)
-    return (abfd->format == format) ? true : false;
+    return abfd->format == format;
 
   /* Presume the answer is yes.  */
   abfd->format = format;
@@ -329,10 +391,10 @@ bfd_set_format (abfd, format)
   if (!BFD_SEND_FMT (abfd, _bfd_set_format, (abfd)))
     {
       abfd->format = bfd_unknown;
-      return false;
+      return FALSE;
     }
 
-  return true;
+  return TRUE;
 }
 
 /*
@@ -340,7 +402,7 @@ FUNCTION
        bfd_format_string
 
 SYNOPSIS
-       CONST char *bfd_format_string(bfd_format format);
+       const char *bfd_format_string(bfd_format format);
 
 DESCRIPTION
        Return a pointer to a const string
@@ -348,7 +410,7 @@ DESCRIPTION
        depending upon the value of @var{format}.
 */
 
-CONST char *
+const char *
 bfd_format_string (format)
      bfd_format format;
 {
This page took 0.027719 seconds and 4 git commands to generate.