Add missing parts of ChangeLog entry for my change of 2000/02/19 00:59:26 to coff...
[deliverable/binutils-gdb.git] / bfd / format.c
index 310a305c903b506a2d69824b59090ddb2e19d111..c9f1f9cd4be3f6f2c521956114a386b7470eefdb 100644 (file)
@@ -1,5 +1,5 @@
 /* Generic BFD support for file formats.
-   Copyright (C) 1990-1991 Free Software Foundation, Inc.
+   Copyright (C) 1990, 91, 92, 93, 94, 95, 1999 Free Software Foundation, Inc.
    Written by Cygnus Support.
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -16,7 +16,7 @@ GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 along with this program; if not, write to the Free Software
-Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 /*
 SECTION
@@ -45,7 +45,7 @@ SECTION
 #include "libbfd.h"
 
 /* IMPORT from targets.c.  */
-extern char *matching_vector[];
+extern const size_t _bfd_target_vector_entries;
 
 /*
 FUNCTION
@@ -70,34 +70,63 @@ DESCRIPTION
        The function returns <<true>> on success, otherwise <<false>>
        with one of the following error codes:  
 
-       o <<invalid_operation>> -
+       o <<bfd_error_invalid_operation>> -
        if <<format>> is not one of <<bfd_object>>, <<bfd_archive>> or
        <<bfd_core>>.
 
-       o <<system_call_error>> -
+       o <<bfd_error_system_call>> -
        if an error occured during a read - even some file mismatches
-       can cause system_call_errors.
+       can cause bfd_error_system_calls.
 
        o <<file_not_recognised>> -
        none of the backends recognised the file format.
 
-       o <<file_ambiguously_recognized>> -
+       o <<bfd_error_file_ambiguously_recognized>> -
        more than one backend recognised the file format.
+*/
+
+boolean
+bfd_check_format (abfd, format)
+     bfd *abfd;
+     bfd_format format;
+{
+  return bfd_check_format_matches (abfd, format, NULL);
+}
+
+/*
+FUNCTION
+       bfd_check_format_matches
 
+SYNOPSIS
+       boolean bfd_check_format_matches(bfd *abfd, bfd_format format, char ***matching);
+
+DESCRIPTION
+       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,
+       allocated with <<malloc>>.
+       Then the user may choose a format and try again.
+
+       When done with the list that @var{matching} points to, the caller
+       should free it.  
 */
 
 boolean
-DEFUN(bfd_check_format,(abfd, format),
-      bfd *abfd AND
-      bfd_format format)
+bfd_check_format_matches (abfd, format, matching)
+     bfd *abfd;
+     bfd_format format;
+     char ***matching;
 {
-  bfd_target **target, *save_targ, *right_targ;
+  extern const bfd_target binary_vec;
+  const bfd_target * const *target, *save_targ, *right_targ;
+  char **matching_vector = NULL;
   int match_count;
 
   if (!bfd_read_p (abfd) ||
       ((int)(abfd->format) < (int)bfd_unknown) ||
       ((int)(abfd->format) >= (int)bfd_type_end)) {
-    bfd_error = invalid_operation;
+    bfd_set_error (bfd_error_invalid_operation);
     return false;
   }
 
@@ -110,7 +139,16 @@ DEFUN(bfd_check_format,(abfd, format),
 
   save_targ = abfd->xvec;
   match_count = 0;
-  matching_vector[0] = NULL;
+  if (matching)
+    {
+      matching_vector =
+       (char **) bfd_malloc (sizeof (char *) *
+                             (_bfd_target_vector_entries + 1));
+      if (!matching_vector)
+       return false;
+      matching_vector[0] = NULL;
+      *matching = matching_vector;
+    }
   right_targ = 0;
 
 
@@ -120,37 +158,73 @@ DEFUN(bfd_check_format,(abfd, format),
   /* If the target type was explicitly specified, just check that target.  */
 
   if (!abfd->target_defaulted) {
-    bfd_seek (abfd, (file_ptr)0, SEEK_SET);    /* rewind! */
-
+    if (bfd_seek (abfd, (file_ptr)0, SEEK_SET) != 0)   /* rewind! */
+      return false;
     right_targ = BFD_SEND_FMT (abfd, _bfd_check_format, (abfd));
     if (right_targ) {
       abfd->xvec = right_targ;         /* Set the target as returned */
+      if (matching)
+       free (matching_vector);
       return true;                     /* File position has moved, BTW */
     }
+
+    /* For a long time the code has dropped through to check all
+       targets if the specified target was wrong.  I don't know why,
+       and I'm reluctant to change it.  However, in the case of an
+       archive, it can cause problems.  If the specified target does
+       not permit archives (e.g., the binary target), then we should
+       not allow some other target to recognize it as an archive, but
+       should instead allow the specified target to recognize it as an
+       object.  When I first made this change, it broke the PE target,
+       because the specified pei-i386 target did not recognize the
+       actual pe-i386 archive.  Since there may be other problems of
+       this sort, I changed this test to check only for the binary
+       target.  */
+
+    if (format == bfd_archive && save_targ == &binary_vec)
+      {
+       abfd->xvec = save_targ;
+       abfd->format = bfd_unknown;
+       if (matching)
+         free (matching_vector);
+       bfd_set_error (bfd_error_file_not_recognized);
+       return false;
+      }
   }
 
-  for (target = target_vector; *target != NULL; target++) {
-    bfd_target *temp;
+  for (target = bfd_target_vector; *target != NULL; target++) {
+    const bfd_target *temp;
+
+    if (*target == &binary_vec)
+      continue;
 
     abfd->xvec = *target;      /* Change BFD's target temporarily */
-    bfd_seek (abfd, (file_ptr)0, SEEK_SET);
-    /* If _bfd_check_format neglects to set bfd_error, assume wrong_format.
+    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 attention to bfd_error, so I suspect
        that some _bfd_check_format might have this problem.  */
-    bfd_error = wrong_format;
+    bfd_set_error (bfd_error_wrong_format);
     temp = BFD_SEND_FMT (abfd, _bfd_check_format, (abfd));
     if (temp) {                                /* This format checks out as ok! */
       right_targ = temp;
-      matching_vector[match_count++] = temp->name;
-      matching_vector[match_count] = NULL;
+      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 == default_vector[0])
+      if (temp == bfd_default_vector[0])
        {
+         if (matching)
+           {
+             matching_vector[0] = temp->name;
+             matching_vector[1] = NULL;
+           }
          match_count = 1;
-         matching_vector[0] = temp->name;
-         matching_vector[1] = NULL;
          break;
        }
 #ifdef GNU960
@@ -162,45 +236,35 @@ DEFUN(bfd_check_format,(abfd, format),
        */
       break;
 #endif
-    } else if (bfd_error != wrong_format) {
+    } else if (bfd_get_error () != 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);
       return false;
     }
   }
 
   if (match_count == 1) {
     abfd->xvec = right_targ;           /* Change BFD's target permanently */
+    if (matching)
+      free (matching_vector);
     return true;                       /* File position has moved, BTW */
   }
 
   abfd->xvec = save_targ;              /* Restore original target type */
   abfd->format = bfd_unknown;          /* Restore original format */
-  bfd_error = ((match_count == 0) ? file_not_recognized :
-              file_ambiguously_recognized);
+  if (match_count == 0)
+    {
+      bfd_set_error (bfd_error_file_not_recognized);
+      if (matching)
+       free (matching_vector);
+    }
+  else
+    bfd_set_error (bfd_error_file_ambiguously_recognized);
   return false;
 }
 
-/*
-FUNCTION
-       bfd_matching_formats
-
-SYNOPSIS
-       char **bfd_matching_formats();
-
-DESCRIPTION
-       If a call to <<bfd_check_format>> returns
-       <<file_ambiguously_recognized>>, you can call this function
-       afterward to return a NULL-terminated list of the names of
-       the formats that matched.
-       Then you can choose one and try again.  */
-
-char **
-bfd_matching_formats ()
-{
-  return &matching_vector[0];
-}
-
 /*
 FUNCTION
        bfd_set_format
@@ -217,15 +281,15 @@ DESCRIPTION
 */
 
 boolean
-DEFUN(bfd_set_format,(abfd, format),
-      bfd *abfd AND
-      bfd_format format)
+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)) {
-    bfd_error = invalid_operation;
+    bfd_set_error (bfd_error_invalid_operation);
     return false;
   }
 
@@ -258,8 +322,8 @@ DESCRIPTION
 */
 
 CONST char *
-DEFUN(bfd_format_string,(format),
-     bfd_format format)
+bfd_format_string (format)
+     bfd_format format;
 {
   if (((int)format <(int) bfd_unknown) 
       || ((int)format >=(int) bfd_type_end)) 
This page took 0.026416 seconds and 4 git commands to generate.