Add SH2E support
[deliverable/binutils-gdb.git] / bfd / peicode.h
index 22ddb2a359627d46cba8e2d87e75892f2c6d4963..2dff0ac3f68607779b273cb61055213d12769758 100644 (file)
@@ -1,5 +1,6 @@
 /* Support for the generic parts of PE/PEI, for BFD.
-   Copyright 1995, 1996, 1997, 1998, 1999, 2000 Free Software Foundation, Inc.
+   Copyright 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002
+   Free Software Foundation, Inc.
    Written by Cygnus Solutions.
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -57,7 +58,7 @@ PE/PEI rearrangement (and code added): Donn Terry
 
 #include "libpei.h"
 
-static boolean (*pe_saved_coff_bfd_print_private_bfd_data)
+static bfd_boolean (*pe_saved_coff_bfd_print_private_bfd_data)
     PARAMS ((bfd *, PTR)) =
 #ifndef coff_bfd_print_private_bfd_data
      NULL;
@@ -66,10 +67,10 @@ static boolean (*pe_saved_coff_bfd_print_private_bfd_data)
 #undef coff_bfd_print_private_bfd_data
 #endif
 
-static boolean pe_print_private_bfd_data PARAMS ((bfd *, PTR));
+static bfd_boolean pe_print_private_bfd_data PARAMS ((bfd *, PTR));
 #define coff_bfd_print_private_bfd_data pe_print_private_bfd_data
 
-static boolean (*pe_saved_coff_bfd_copy_private_bfd_data)
+static bfd_boolean (*pe_saved_coff_bfd_copy_private_bfd_data)
     PARAMS ((bfd *, bfd *)) =
 #ifndef coff_bfd_copy_private_bfd_data
      NULL;
@@ -78,7 +79,7 @@ static boolean (*pe_saved_coff_bfd_copy_private_bfd_data)
 #undef coff_bfd_copy_private_bfd_data
 #endif
 
-static boolean pe_bfd_copy_private_bfd_data PARAMS ((bfd *, bfd *));
+static bfd_boolean pe_bfd_copy_private_bfd_data PARAMS ((bfd *, bfd *));
 #define coff_bfd_copy_private_bfd_data pe_bfd_copy_private_bfd_data
 
 #define coff_mkobject      pe_mkobject
@@ -90,7 +91,7 @@ static unsigned int coff_swap_reloc_out PARAMS ((bfd *, PTR, PTR));
 #endif
 static void coff_swap_filehdr_in PARAMS ((bfd *, PTR, PTR));
 static void coff_swap_scnhdr_in PARAMS ((bfd *, PTR, PTR));
-static boolean pe_mkobject PARAMS ((bfd *));
+static bfd_boolean pe_mkobject PARAMS ((bfd *));
 static PTR pe_mkobject_hook PARAMS ((bfd *, PTR, PTR));
 
 #ifdef COFF_IMAGE_WITH_PE
@@ -103,29 +104,29 @@ typedef struct
   bfd_byte *           data;
   struct bfd_in_memory * bim;
   unsigned short        magic;
-  
+
   arelent *            reltab;
   unsigned int                 relcount;
 
   coff_symbol_type *   sym_cache;
   coff_symbol_type *   sym_ptr;
   unsigned int         sym_index;
-  
+
   unsigned int *       sym_table;
   unsigned int *       table_ptr;
-  
+
   combined_entry_type * native_syms;
   combined_entry_type * native_ptr;
 
   coff_symbol_type **  sym_ptr_table;
   coff_symbol_type **  sym_ptr_ptr;
-  
+
   unsigned int         sec_index;
 
   char *                string_table;
   char *                string_ptr;
   char *               end_string_ptr;
-  
+
   SYMENT *              esym_table;
   SYMENT *              esym_ptr;
 
@@ -138,7 +139,7 @@ static void               pe_ILF_make_a_reloc     PARAMS ((pe_ILF_vars *, bfd_vm
 static void               pe_ILF_make_a_symbol    PARAMS ((pe_ILF_vars *, const char *, const char *, asection_ptr, flagword));
 static void               pe_ILF_save_relocs      PARAMS ((pe_ILF_vars *, asection_ptr));
 static void              pe_ILF_make_a_symbol_reloc  PARAMS ((pe_ILF_vars *, bfd_vma, bfd_reloc_code_real_type, struct symbol_cache_entry **, unsigned int));
-static boolean            pe_ILF_build_a_bfd      PARAMS ((bfd *, unsigned short, bfd_byte *, bfd_byte *, unsigned int, unsigned int));
+static bfd_boolean        pe_ILF_build_a_bfd      PARAMS ((bfd *, unsigned int, bfd_byte *, bfd_byte *, unsigned int, unsigned int));
 static const bfd_target * pe_ILF_object_p         PARAMS ((bfd *));
 static const bfd_target * pe_bfd_object_p        PARAMS ((bfd *));
 #endif /* COFF_IMAGE_WITH_PE */
@@ -155,18 +156,16 @@ coff_swap_reloc_in (abfd, src, dst)
   RELOC *reloc_src = (RELOC *) src;
   struct internal_reloc *reloc_dst = (struct internal_reloc *) dst;
 
-  reloc_dst->r_vaddr = bfd_h_get_32(abfd, (bfd_byte *)reloc_src->r_vaddr);
-  reloc_dst->r_symndx = bfd_h_get_signed_32(abfd, (bfd_byte *) reloc_src->r_symndx);
+  reloc_dst->r_vaddr = H_GET_32 (abfd, reloc_src->r_vaddr);
+  reloc_dst->r_symndx = H_GET_S32 (abfd, reloc_src->r_symndx);
 
-  reloc_dst->r_type = bfd_h_get_16(abfd, (bfd_byte *) reloc_src->r_type);
+  reloc_dst->r_type = H_GET_16 (abfd, reloc_src->r_type);
 
 #ifdef SWAP_IN_RELOC_OFFSET
-  reloc_dst->r_offset = SWAP_IN_RELOC_OFFSET(abfd,
-                                            (bfd_byte *) reloc_src->r_offset);
+  reloc_dst->r_offset = SWAP_IN_RELOC_OFFSET (abfd, reloc_src->r_offset);
 #endif
 }
 
-
 static unsigned int
 coff_swap_reloc_out (abfd, src, dst)
      bfd       *abfd;
@@ -175,19 +174,16 @@ coff_swap_reloc_out (abfd, src, dst)
 {
   struct internal_reloc *reloc_src = (struct internal_reloc *)src;
   struct external_reloc *reloc_dst = (struct external_reloc *)dst;
-  bfd_h_put_32(abfd, reloc_src->r_vaddr, (bfd_byte *) reloc_dst->r_vaddr);
-  bfd_h_put_32(abfd, reloc_src->r_symndx, (bfd_byte *) reloc_dst->r_symndx);
+  H_PUT_32 (abfd, reloc_src->r_vaddr, reloc_dst->r_vaddr);
+  H_PUT_32 (abfd, reloc_src->r_symndx, reloc_dst->r_symndx);
 
-  bfd_h_put_16(abfd, reloc_src->r_type, (bfd_byte *)
-              reloc_dst->r_type);
+  H_PUT_16 (abfd, reloc_src->r_type, reloc_dst->r_type);
 
 #ifdef SWAP_OUT_RELOC_OFFSET
-  SWAP_OUT_RELOC_OFFSET(abfd,
-                       reloc_src->r_offset,
-                       (bfd_byte *) reloc_dst->r_offset);
+  SWAP_OUT_RELOC_OFFSET (abfd, reloc_src->r_offset, reloc_dst->r_offset);
 #endif
 #ifdef SWAP_OUT_RELOC_EXTRA
-  SWAP_OUT_RELOC_EXTRA(abfd,reloc_src, reloc_dst);
+  SWAP_OUT_RELOC_EXTRA(abfd, reloc_src, reloc_dst);
 #endif
   return RELSZ;
 }
@@ -201,33 +197,13 @@ coff_swap_filehdr_in (abfd, src, dst)
 {
   FILHDR *filehdr_src = (FILHDR *) src;
   struct internal_filehdr *filehdr_dst = (struct internal_filehdr *) dst;
-  filehdr_dst->f_magic = bfd_h_get_16(abfd, (bfd_byte *) filehdr_src->f_magic);
-  filehdr_dst->f_nscns = bfd_h_get_16(abfd, (bfd_byte *)filehdr_src-> f_nscns);
-  filehdr_dst->f_timdat = bfd_h_get_32(abfd, (bfd_byte *)filehdr_src-> f_timdat);
+  filehdr_dst->f_magic = H_GET_16 (abfd, filehdr_src->f_magic);
+  filehdr_dst->f_nscns = H_GET_16 (abfd, filehdr_src-> f_nscns);
+  filehdr_dst->f_timdat = H_GET_32 (abfd, filehdr_src-> f_timdat);
 
-  filehdr_dst->f_nsyms = bfd_h_get_32(abfd, (bfd_byte *)filehdr_src-> f_nsyms);
-  filehdr_dst->f_flags = bfd_h_get_16(abfd, (bfd_byte *)filehdr_src-> f_flags);
-  filehdr_dst->f_symptr = bfd_h_get_32 (abfd, (bfd_byte *) filehdr_src->f_symptr);
-
-#ifdef COFF_IMAGE_WITH_PE
-  /* There are really two magic numbers involved; the magic number
-     that says this is a NT executable (PEI) and the magic number that
-     determines the architecture.  The former is DOSMAGIC, stored in
-     the e_magic field.  The latter is stored in the f_magic field.
-     If the NT magic number isn't valid, the architecture magic number
-     could be mimicked by some other field (specifically, the number
-     of relocs in section 3).  Since this routine can only be called
-     correctly for a PEI file, check the e_magic number here, and, if
-     it doesn't match, clobber the f_magic number so that we don't get
-     a false match.  */
-#if 0
-  /* We can't assume that the PE header is at offset 0x80.  When it
-     isn't, the DOS header isn't read correctly, so we can't assume
-     e_magic is set even for valid PE files. */
-  if (bfd_h_get_16 (abfd, (bfd_byte *) filehdr_src->e_magic) != DOSMAGIC)
-    filehdr_dst->f_magic = -1;
-#endif
-#endif
+  filehdr_dst->f_nsyms = H_GET_32 (abfd, filehdr_src-> f_nsyms);
+  filehdr_dst->f_flags = H_GET_16 (abfd, filehdr_src-> f_flags);
+  filehdr_dst->f_symptr = H_GET_32 (abfd, filehdr_src->f_symptr);
 
   /* Other people's tools sometimes generate headers with an nsyms but
      a zero symptr.  */
@@ -237,14 +213,13 @@ coff_swap_filehdr_in (abfd, src, dst)
       filehdr_dst->f_flags |= F_LSYMS;
     }
 
-  filehdr_dst->f_opthdr = bfd_h_get_16(abfd, 
-                                      (bfd_byte *)filehdr_src-> f_opthdr);
+  filehdr_dst->f_opthdr = H_GET_16 (abfd, filehdr_src-> f_opthdr);
 }
 
 #ifdef COFF_IMAGE_WITH_PE
-#define coff_swap_filehdr_out _bfd_pei_only_swap_filehdr_out
+# define coff_swap_filehdr_out _bfd_XXi_only_swap_filehdr_out
 #else
-#define coff_swap_filehdr_out _bfd_pe_only_swap_filehdr_out
+# define coff_swap_filehdr_out _bfd_pe_only_swap_filehdr_out
 #endif
 
 static void
@@ -256,37 +231,28 @@ coff_swap_scnhdr_in (abfd, ext, in)
   SCNHDR *scnhdr_ext = (SCNHDR *) ext;
   struct internal_scnhdr *scnhdr_int = (struct internal_scnhdr *) in;
 
-  memcpy(scnhdr_int->s_name, scnhdr_ext->s_name, sizeof(scnhdr_int->s_name));
-  scnhdr_int->s_vaddr =
-    GET_SCNHDR_VADDR (abfd, (bfd_byte *) scnhdr_ext->s_vaddr);
-  scnhdr_int->s_paddr =
-    GET_SCNHDR_PADDR (abfd, (bfd_byte *) scnhdr_ext->s_paddr);
-  scnhdr_int->s_size =
-    GET_SCNHDR_SIZE (abfd, (bfd_byte *) scnhdr_ext->s_size);
-  scnhdr_int->s_scnptr =
-    GET_SCNHDR_SCNPTR (abfd, (bfd_byte *) scnhdr_ext->s_scnptr);
-  scnhdr_int->s_relptr =
-    GET_SCNHDR_RELPTR (abfd, (bfd_byte *) scnhdr_ext->s_relptr);
-  scnhdr_int->s_lnnoptr =
-    GET_SCNHDR_LNNOPTR (abfd, (bfd_byte *) scnhdr_ext->s_lnnoptr);
-  scnhdr_int->s_flags = bfd_h_get_32(abfd, (bfd_byte *) scnhdr_ext->s_flags);
+  memcpy(scnhdr_int->s_name, scnhdr_ext->s_name, sizeof (scnhdr_int->s_name));
+  scnhdr_int->s_vaddr = GET_SCNHDR_VADDR (abfd, scnhdr_ext->s_vaddr);
+  scnhdr_int->s_paddr = GET_SCNHDR_PADDR (abfd, scnhdr_ext->s_paddr);
+  scnhdr_int->s_size = GET_SCNHDR_SIZE (abfd, scnhdr_ext->s_size);
+  scnhdr_int->s_scnptr = GET_SCNHDR_SCNPTR (abfd, scnhdr_ext->s_scnptr);
+  scnhdr_int->s_relptr = GET_SCNHDR_RELPTR (abfd, scnhdr_ext->s_relptr);
+  scnhdr_int->s_lnnoptr = GET_SCNHDR_LNNOPTR (abfd, scnhdr_ext->s_lnnoptr);
+  scnhdr_int->s_flags = H_GET_32 (abfd, scnhdr_ext->s_flags);
 
   /* MS handles overflow of line numbers by carrying into the reloc
      field (it appears).  Since it's supposed to be zero for PE
      *IMAGE* format, that's safe.  This is still a bit iffy.  */
 #ifdef COFF_IMAGE_WITH_PE
-  scnhdr_int->s_nlnno =
-    (bfd_h_get_16 (abfd, (bfd_byte *) scnhdr_ext->s_nlnno)
-     + (bfd_h_get_16 (abfd, (bfd_byte *) scnhdr_ext->s_nreloc) << 16));
+  scnhdr_int->s_nlnno = (H_GET_16 (abfd, scnhdr_ext->s_nlnno)
+                        + (H_GET_16 (abfd, scnhdr_ext->s_nreloc) << 16));
   scnhdr_int->s_nreloc = 0;
 #else
-  scnhdr_int->s_nreloc = bfd_h_get_16 (abfd,
-                                      (bfd_byte *) scnhdr_ext->s_nreloc);
-  scnhdr_int->s_nlnno = bfd_h_get_16 (abfd,
-                                     (bfd_byte *) scnhdr_ext->s_nlnno);
+  scnhdr_int->s_nreloc = H_GET_16 (abfd, scnhdr_ext->s_nreloc);
+  scnhdr_int->s_nlnno = H_GET_16 (abfd, scnhdr_ext->s_nlnno);
 #endif
 
-  if (scnhdr_int->s_vaddr != 0) 
+  if (scnhdr_int->s_vaddr != 0)
     {
       scnhdr_int->s_vaddr += pe_data (abfd)->pe_opthdr.ImageBase;
       scnhdr_int->s_vaddr &= 0xffffffff;
@@ -295,7 +261,8 @@ coff_swap_scnhdr_in (abfd, ext, in)
 #ifndef COFF_NO_HACK_SCNHDR_SIZE
   /* If this section holds uninitialized data, use the virtual size
      (stored in s_paddr) instead of the physical size.  */
-  if ((scnhdr_int->s_flags & IMAGE_SCN_CNT_UNINITIALIZED_DATA) != 0)
+  if ((scnhdr_int->s_flags & IMAGE_SCN_CNT_UNINITIALIZED_DATA) != 0
+      && (scnhdr_int->s_paddr > 0))
     {
       scnhdr_int->s_size = scnhdr_int->s_paddr;
       /* This code used to set scnhdr_int->s_paddr to 0.  However,
@@ -306,16 +273,17 @@ coff_swap_scnhdr_in (abfd, ext, in)
 #endif
 }
 
-static boolean
+static bfd_boolean
 pe_mkobject (abfd)
      bfd * abfd;
 {
   pe_data_type *pe;
-  abfd->tdata.pe_obj_data = 
-    (struct pe_tdata *) bfd_zalloc (abfd, sizeof (pe_data_type));
+  bfd_size_type amt = sizeof (pe_data_type);
+
+  abfd->tdata.pe_obj_data = (struct pe_tdata *) bfd_zalloc (abfd, amt);
 
   if (abfd->tdata.pe_obj_data == 0)
-    return false;
+    return FALSE;
 
   pe = pe_data (abfd);
 
@@ -323,7 +291,15 @@ pe_mkobject (abfd)
 
   /* in_reloc_p is architecture dependent.  */
   pe->in_reloc_p = in_reloc_p;
-  return true;
+
+#ifdef PEI_FORCE_MINIMUM_ALIGNMENT
+  pe->force_minimum_alignment = 1;
+#endif
+#ifdef PEI_TARGET_SUBSYSTEM
+  pe->target_subsystem = PEI_TARGET_SUBSYSTEM;
+#endif
+
+  return TRUE;
 }
 
 /* Create the COFF backend specific information.  */
@@ -336,7 +312,7 @@ pe_mkobject_hook (abfd, filehdr, aouthdr)
   struct internal_filehdr *internal_f = (struct internal_filehdr *) filehdr;
   pe_data_type *pe;
 
-  if (pe_mkobject (abfd) == false)
+  if (! pe_mkobject (abfd))
     return NULL;
 
   pe = pe_data (abfd);
@@ -367,27 +343,27 @@ pe_mkobject_hook (abfd, filehdr, aouthdr)
     abfd->flags |= HAS_DEBUG;
 
 #ifdef COFF_IMAGE_WITH_PE
-  if (aouthdr) 
+  if (aouthdr)
     pe->pe_opthdr = ((struct internal_aouthdr *)aouthdr)->pe;
 #endif
 
-#ifdef ARM 
+#ifdef ARM
   if (! _bfd_coff_arm_set_private_flags (abfd, internal_f->f_flags))
     coff_data (abfd) ->flags = 0;
 #endif
-  
+
   return (PTR) pe;
 }
 
-static boolean
+static bfd_boolean
 pe_print_private_bfd_data (abfd, vfile)
      bfd *abfd;
      PTR vfile;
 {
   FILE *file = (FILE *) vfile;
 
-  if (!_bfd_pe_print_private_bfd_data_common (abfd, vfile))
-    return false;
+  if (!_bfd_XX_print_private_bfd_data_common (abfd, vfile))
+    return FALSE;
 
   if (pe_saved_coff_bfd_print_private_bfd_data != NULL)
     {
@@ -396,35 +372,35 @@ pe_print_private_bfd_data (abfd, vfile)
       return pe_saved_coff_bfd_print_private_bfd_data (abfd, vfile);
     }
 
-  return true;
+  return TRUE;
 }
 
 /* Copy any private info we understand from the input bfd
    to the output bfd.  */
 
-static boolean
+static bfd_boolean
 pe_bfd_copy_private_bfd_data (ibfd, obfd)
      bfd *ibfd, *obfd;
 {
-  if (!_bfd_pe_bfd_copy_private_bfd_data_common (ibfd, obfd))
-    return false;
+  if (!_bfd_XX_bfd_copy_private_bfd_data_common (ibfd, obfd))
+    return FALSE;
 
   if (pe_saved_coff_bfd_copy_private_bfd_data)
     return pe_saved_coff_bfd_copy_private_bfd_data (ibfd, obfd);
 
-  return true;
+  return TRUE;
 }
 
 #define coff_bfd_copy_private_section_data \
-  _bfd_pe_bfd_copy_private_section_data
+  _bfd_XX_bfd_copy_private_section_data
 
-#define coff_get_symbol_info _bfd_pe_get_symbol_info
+#define coff_get_symbol_info _bfd_XX_get_symbol_info
 
 #ifdef COFF_IMAGE_WITH_PE
 \f
 /* Code to handle Microsoft's Image Library Format.
    Also known as LINK6 format.
-   Documentation about this format can be found at: 
+   Documentation about this format can be found at:
 
    http://msdn.microsoft.com/library/specs/pecoff_section8.htm  */
 
@@ -450,7 +426,7 @@ pe_bfd_copy_private_bfd_data (ibfd, obfd)
 #define NUM_ILF_RELOCS         8
 #define NUM_ILF_SECTIONS        6
 #define NUM_ILF_SYMS           (2 + NUM_ILF_SECTIONS)
-  
+
 #define SIZEOF_ILF_SYMS                (NUM_ILF_SYMS * sizeof (* vars.sym_cache))
 #define SIZEOF_ILF_SYM_TABLE   (NUM_ILF_SYMS * sizeof (* vars.sym_table))
 #define SIZEOF_ILF_NATIVE_SYMS (NUM_ILF_SYMS * sizeof (* vars.native_syms))
@@ -468,7 +444,7 @@ pe_bfd_copy_private_bfd_data (ibfd, obfd)
 #define SIZEOF_IDATA6          (2 + strlen (symbol_name) + 1 + 1)
 #define SIZEOF_IDATA7          (strlen (source_dll) + 1 + 1)
 #define SIZEOF_ILF_SECTIONS     (NUM_ILF_SECTIONS * sizeof (struct coff_section_tdata))
-  
+
 #define ILF_DATA_SIZE                          \
       sizeof (* vars.bim)                      \
     + SIZEOF_ILF_SYMS                          \
@@ -487,7 +463,6 @@ pe_bfd_copy_private_bfd_data (ibfd, obfd)
     + SIZEOF_ILF_SECTIONS                      \
     + MAX_TEXT_SECTION_SIZE
 
-
 /* Create an empty relocation against the given symbol.  */
 static void
 pe_ILF_make_a_symbol_reloc (pe_ILF_vars *                 vars,
@@ -501,7 +476,7 @@ pe_ILF_make_a_symbol_reloc (pe_ILF_vars *                 vars,
 
   entry = vars->reltab + vars->relcount;
   internal = vars->int_reltab + vars->relcount;
-  
+
   entry->address     = address;
   entry->addend      = 0;
   entry->howto       = bfd_reloc_type_lookup (vars->abfd, reloc);
@@ -515,9 +490,9 @@ pe_ILF_make_a_symbol_reloc (pe_ILF_vars *                 vars,
   internal->r_extern = 0;
   internal->r_offset = 0;
 #endif
-  
+
   vars->relcount ++;
-  
+
   BFD_ASSERT (vars->relcount <= NUM_ILF_RELOCS);
 }
 
@@ -543,7 +518,7 @@ pe_ILF_save_relocs (pe_ILF_vars * vars,
     abort ();
 
   coff_section_data (vars->abfd, sec)->relocs = vars->int_reltab;
-  coff_section_data (vars->abfd, sec)->keep_relocs = true;
+  coff_section_data (vars->abfd, sec)->keep_relocs = TRUE;
 
   sec->relocation  = vars->reltab;
   sec->reloc_count = vars->relcount;
@@ -553,7 +528,7 @@ pe_ILF_save_relocs (pe_ILF_vars * vars,
   vars->int_reltab += vars->relcount;
   vars->relcount   = 0;
 
-  BFD_ASSERT ((bfd_byte *)vars->int_reltab < (bfd_byte *)vars->string_table);
+  BFD_ASSERT ((bfd_byte *) vars->int_reltab < (bfd_byte *) vars->string_table);
 }
 
 /* Create a global symbol and add it to the relevant tables.  */
@@ -573,8 +548,8 @@ pe_ILF_make_a_symbol (pe_ILF_vars *  vars,
     sclass = C_STAT;
   else
     sclass = C_EXT;
-  
-#ifdef THUMBPEMAGIC  
+
+#ifdef THUMBPEMAGIC
   if (vars->magic == THUMBPEMAGIC)
     {
       if (extra_flags & BSF_FUNCTION)
@@ -587,7 +562,7 @@ pe_ILF_make_a_symbol (pe_ILF_vars *  vars,
 #endif
 
   BFD_ASSERT (vars->sym_index < NUM_ILF_SYMS);
-  
+
   sym = vars->sym_ptr;
   ent = vars->native_ptr;
   esym = vars->esym_ptr;
@@ -597,10 +572,11 @@ pe_ILF_make_a_symbol (pe_ILF_vars *  vars,
 
   if (section == NULL)
     section = (asection_ptr) & bfd_und_section;
-  
+
   /* Initialise the external symbol.  */
-  bfd_h_put_32 (vars->abfd, vars->string_ptr - vars->string_table, (bfd_byte *) esym->e.e.e_offset);
-  bfd_h_put_16 (vars->abfd, section->target_index, (bfd_byte *) esym->e_scnum);
+  H_PUT_32 (vars->abfd, vars->string_ptr - vars->string_table,
+           esym->e.e.e_offset);
+  H_PUT_16 (vars->abfd, section->target_index, esym->e_scnum);
   esym->e_sclass[0] = sclass;
 
   /* The following initialisations are unnecessary - the memory is
@@ -611,12 +587,12 @@ pe_ILF_make_a_symbol (pe_ILF_vars *  vars,
   esym->e_type = T_NULL;
   esym->e_numaux = 0;
 #endif
-  
+
   /* Initialise the internal symbol structure.  */
   ent->u.syment.n_sclass          = sclass;
   ent->u.syment.n_scnum           = section->target_index;
   ent->u.syment._n._n_n._n_offset = (long) sym;
-  
+
 #if 0 /* See comment above.  */
   ent->u.syment.n_value  = 0;
   ent->u.syment.n_flags  = 0;
@@ -624,23 +600,23 @@ pe_ILF_make_a_symbol (pe_ILF_vars *  vars,
   ent->u.syment.n_numaux = 0;
   ent->fix_value         = 0;
 #endif
-  
+
   sym->symbol.the_bfd = vars->abfd;
   sym->symbol.name    = vars->string_ptr;
   sym->symbol.flags   = BSF_EXPORT | BSF_GLOBAL | extra_flags;
   sym->symbol.section = section;
   sym->native         = ent;
-  
+
 #if 0 /* See comment above.  */
   sym->symbol.value   = 0;
   sym->symbol.udata.i = 0;
-  sym->done_lineno    = false;
+  sym->done_lineno    = FALSE;
   sym->lineno         = NULL;
 #endif
-  
+
   * vars->table_ptr = vars->sym_index;
   * vars->sym_ptr_ptr = sym;
-  
+
   /* Adjust pointers for the next symbol.  */
   vars->sym_index ++;
   vars->sym_ptr ++;
@@ -662,48 +638,48 @@ pe_ILF_make_a_section (pe_ILF_vars * vars,
 {
   asection_ptr sec;
   flagword     flags;
-  
+
   sec = bfd_make_section_old_way (vars->abfd, name);
   if (sec == NULL)
     return NULL;
-  
+
   flags = SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD | SEC_KEEP | SEC_IN_MEMORY;
-  
+
   bfd_set_section_flags (vars->abfd, sec, flags | extra_flags);
-     
+
   bfd_set_section_alignment (vars->abfd, sec, 2);
-  
+
   /* Check that we will not run out of space.  */
   BFD_ASSERT (vars->data + size < vars->bim->buffer + vars->bim->size);
-  
+
   /* Set the section size and contents.  The actual
      contents are filled in by our parent.  */
-  bfd_set_section_size (vars->abfd, sec, size);
+  bfd_set_section_size (vars->abfd, sec, (bfd_size_type) size);
   sec->contents = vars->data;
   sec->target_index = vars->sec_index ++;
 
   /* Advance data pointer in the vars structure.  */
   vars->data += size;
-  
+
   /* Skip the padding byte if it was not needed.
      The logic here is that if the string length is odd,
      then the entire string length, including the null byte,
      is even and so the extra, padding byte, is not needed.  */
   if (size & 1)
     vars->data --;
-  
+
   /* Create a coff_section_tdata structure for our use.  */
   sec->used_by_bfd = (struct coff_section_tdata *) vars->data;
   vars->data += sizeof (struct coff_section_tdata);
 
   BFD_ASSERT (vars->data <= vars->bim->buffer + vars->bim->size);
-  
+
   /* Create a symbol to refer to this section.  */
   pe_ILF_make_a_symbol (vars, "", name, sec, BSF_LOCAL);
 
   /* Cache the index to the symbol in the coff_section_data structure.  */
   coff_section_data (vars->abfd, sec)->i = vars->sym_index - 1;
-  
+
   return sec;
 }
 
@@ -734,7 +710,7 @@ static jump_table jtab[] =
     8, 2
   },
 #endif
-  
+
 #ifdef  MC68MAGIC
   { MC68MAGIC, { /* XXX fill me in */ }, 0, 0 },
 #endif
@@ -745,7 +721,7 @@ static jump_table jtab[] =
     16, 0
   },
 #endif
-  
+
 #ifdef  SH_ARCH_MAGIC_WINCE
   { SH_ARCH_MAGIC_WINCE,
     { 0x01, 0xd0, 0x02, 0x60, 0x2b, 0x40,
@@ -753,7 +729,7 @@ static jump_table jtab[] =
     12, 8
   },
 #endif
-  
+
 #ifdef  ARMPEMAGIC
   { ARMPEMAGIC,
     { 0x00, 0xc0, 0x9f, 0xe5, 0x00, 0xf0,
@@ -761,7 +737,7 @@ static jump_table jtab[] =
     12, 8
   },
 #endif
-  
+
 #ifdef  THUMBPEMAGIC
   { THUMBPEMAGIC,
     { 0x40, 0xb4, 0x02, 0x4e, 0x36, 0x68, 0xb4, 0x46,
@@ -777,14 +753,14 @@ static jump_table jtab[] =
 #endif
 
 /* Build a full BFD from the information supplied in a ILF object.  */
-static boolean
+static bfd_boolean
 pe_ILF_build_a_bfd (bfd *           abfd,
-                   unsigned short  magic,
+                   unsigned int    magic,
                    bfd_byte *      symbol_name,
                    bfd_byte *      source_dll,
                    unsigned int    ordinal,
                    unsigned int    types)
-{ 
+{
   bfd_byte *               ptr;
   pe_ILF_vars              vars;
   struct internal_filehdr  internal_f;
@@ -803,17 +779,17 @@ pe_ILF_build_a_bfd (bfd *           abfd,
     case IMPORT_CODE:
     case IMPORT_DATA:
       break;
-      
+
     case IMPORT_CONST:
       /* XXX code yet to be written.  */
       _bfd_error_handler (_("%s: Unhandled import type; %x"),
-                         bfd_get_filename (abfd), import_type);
-      return false;
-      
+                         bfd_archive_filename (abfd), import_type);
+      return FALSE;
+
     default:
       _bfd_error_handler (_("%s: Unrecognised import type; %x"),
-                         bfd_get_filename (abfd), import_type);
-      return false;
+                         bfd_archive_filename (abfd), import_type);
+      return FALSE;
     }
 
   switch (import_name_type)
@@ -823,37 +799,37 @@ pe_ILF_build_a_bfd (bfd *           abfd,
     case IMPORT_NAME_NOPREFIX:
     case IMPORT_NAME_UNDECORATE:
       break;
-      
+
     default:
       _bfd_error_handler (_("%s: Unrecognised import name type; %x"),
-                         bfd_get_filename (abfd), import_name_type);
-      return false;
+                         bfd_archive_filename (abfd), import_name_type);
+      return FALSE;
     }
 
   /* Initialise local variables.
-     
+
      Note these are kept in a structure rather than being
      declared as statics since bfd frowns on global variables.
-     
+
      We are going to construct the contents of the BFD in memory,
      so allocate all the space that we will need right now.  */
-  ptr = bfd_zalloc (abfd, ILF_DATA_SIZE);
+  ptr = bfd_zalloc (abfd, (bfd_size_type) ILF_DATA_SIZE);
   if (ptr == NULL)
-    return false;
+    return FALSE;
 
   /* Create a bfd_in_memory structure.  */
   vars.bim = (struct bfd_in_memory *) ptr;
   vars.bim->buffer = ptr;
   vars.bim->size   = ILF_DATA_SIZE;
   ptr += sizeof (* vars.bim);
-  
+
   /* Initialise the pointers to regions of the memory and the
      other contents of the pe_ILF_vars structure as well.  */
   vars.sym_cache = (coff_symbol_type *) ptr;
   vars.sym_ptr   = (coff_symbol_type *) ptr;
   vars.sym_index = 0;
   ptr += SIZEOF_ILF_SYMS;
-  
+
   vars.sym_table = (unsigned int *) ptr;
   vars.table_ptr = (unsigned int *) ptr;
   ptr += SIZEOF_ILF_SYM_TABLE;
@@ -865,11 +841,11 @@ pe_ILF_build_a_bfd (bfd *           abfd,
   vars.sym_ptr_table = (coff_symbol_type **) ptr;
   vars.sym_ptr_ptr   = (coff_symbol_type **) ptr;
   ptr += SIZEOF_ILF_SYM_PTR_TABLE;
-  
+
   vars.esym_table = (SYMENT *) ptr;
   vars.esym_ptr   = (SYMENT *) ptr;
   ptr += SIZEOF_ILF_EXT_SYMS;
-  
+
   vars.reltab   = (arelent *) ptr;
   vars.relcount = 0;
   ptr += SIZEOF_ILF_RELOCS;
@@ -881,14 +857,14 @@ pe_ILF_build_a_bfd (bfd *           abfd,
   vars.string_ptr   = ptr + STRING_SIZE_SIZE;
   ptr += SIZEOF_ILF_STRINGS;
   vars.end_string_ptr = ptr;
-  
+
   /* The remaining space in bim->buffer is used
      by the pe_ILF_make_a_section() function.  */
   vars.data = ptr;
   vars.abfd = abfd;
   vars.sec_index = 0;
   vars.magic = magic;
-  
+
   /* Create the initial .idata$<n> sections:
      [.idata$2:  Import Directory Table -- not needed]
      .idata$4:  Import Lookup Table
@@ -899,26 +875,26 @@ pe_ILF_build_a_bfd (bfd *           abfd,
   id4 = pe_ILF_make_a_section (& vars, ".idata$4", SIZEOF_IDATA4, 0);
   id5 = pe_ILF_make_a_section (& vars, ".idata$5", SIZEOF_IDATA5, 0);
   if (id4 == NULL || id5 == NULL)
-    return false;
-  
+    return FALSE;
+
   /* Fill in the contents of these sections.  */
   if (import_name_type == IMPORT_ORDINAL)
     {
       if (ordinal == 0)
        /* XXX - treat as IMPORT_NAME ??? */
        abort ();
-      
+
       * (unsigned int *) id4->contents = ordinal | 0x80000000;
       * (unsigned int *) id5->contents = ordinal | 0x80000000;
     }
   else
     {
       char * symbol;
-      
+
       /* Create .idata$6 - the Hint Name Table.  */
       id6 = pe_ILF_make_a_section (& vars, ".idata$6", SIZEOF_IDATA6, 0);
       if (id6 == NULL)
-       return false;
+       return FALSE;
 
       /* If necessary, trim the import symbol name.  */
       symbol = symbol_name;
@@ -936,27 +912,27 @@ pe_ILF_build_a_bfd (bfd *           abfd,
 
          * symbol = 0;
        }
-      
+
       id6->contents[0] = ordinal & 0xff;
       id6->contents[1] = ordinal >> 8;
-      
+
       strcpy (id6->contents + 2, symbol);
     }
 
   if (import_name_type != IMPORT_ORDINAL)
     {
-      pe_ILF_make_a_reloc (& vars, 0, BFD_RELOC_RVA, id6);
-      pe_ILF_save_relocs (& vars, id4);
-      
-      pe_ILF_make_a_reloc (& vars, 0, BFD_RELOC_RVA, id6);
-      pe_ILF_save_relocs (& vars, id5);
+      pe_ILF_make_a_reloc (&vars, (bfd_vma) 0, BFD_RELOC_RVA, id6);
+      pe_ILF_save_relocs (&vars, id4);
+
+      pe_ILF_make_a_reloc (&vars, (bfd_vma) 0, BFD_RELOC_RVA, id6);
+      pe_ILF_save_relocs (&vars, id5);
     }
 
   /* Create extra sections depending upon the type of import we are dealing with.  */
   switch (import_type)
     {
       int i;
-      
+
     case IMPORT_CODE:
       /* Create a .text section.
         First we need to look up its contents in the jump table.  */
@@ -974,7 +950,7 @@ pe_ILF_build_a_bfd (bfd *           abfd,
       /* Create the .text section.  */
       text = pe_ILF_make_a_section (& vars, ".text", jtab[i].size, SEC_CODE);
       if (text == NULL)
-       return false;
+       return FALSE;
 
       /* Copy in the jump code.  */
       memcpy (text->contents, jtab[i].data, jtab[i].size);
@@ -983,22 +959,25 @@ pe_ILF_build_a_bfd (bfd *           abfd,
       pe_ILF_make_a_symbol (& vars, "__imp_", symbol_name, id5, 0);
       imp_sym   = vars.sym_ptr_ptr - 1;
       imp_index = vars.sym_index - 1;
-    
+
       /* Create a reloc for the data in the text section.  */
-#ifdef MIPS_ARCH_MAGIC_WINCE      
+#ifdef MIPS_ARCH_MAGIC_WINCE
       if (magic == MIPS_ARCH_MAGIC_WINCE)
        {
-         pe_ILF_make_a_symbol_reloc (& vars, 0, BFD_RELOC_HI16_S,
-                                     (struct symbol_cache_entry **) imp_sym, imp_index);
-         pe_ILF_make_a_reloc (& vars, 0, BFD_RELOC_LO16, text);
-         pe_ILF_make_a_symbol_reloc (& vars, 4, BFD_RELOC_LO16,
-                                     (struct symbol_cache_entry **) imp_sym, imp_index);
+         pe_ILF_make_a_symbol_reloc (&vars, (bfd_vma) 0, BFD_RELOC_HI16_S,
+                                     (struct symbol_cache_entry **) imp_sym,
+                                     imp_index);
+         pe_ILF_make_a_reloc (&vars, (bfd_vma) 0, BFD_RELOC_LO16, text);
+         pe_ILF_make_a_symbol_reloc (&vars, (bfd_vma) 4, BFD_RELOC_LO16,
+                                     (struct symbol_cache_entry **) imp_sym,
+                                     imp_index);
        }
       else
 #endif
-       pe_ILF_make_a_symbol_reloc (& vars, jtab[i].offset, BFD_RELOC_32,
-                                   (asymbol **) imp_sym, imp_index);
-      
+       pe_ILF_make_a_symbol_reloc (&vars, (bfd_vma) jtab[i].offset,
+                                   BFD_RELOC_32, (asymbol **) imp_sym,
+                                   imp_index);
+
       pe_ILF_save_relocs (& vars, text);
       break;
 
@@ -1009,29 +988,29 @@ pe_ILF_build_a_bfd (bfd *           abfd,
       /* XXX code not yet written.  */
       abort ();
     }
-  
+
   /* Initialise the bfd.  */
   memset (& internal_f, 0, sizeof (internal_f));
-  
+
   internal_f.f_magic  = magic;
   internal_f.f_symptr = 0;
   internal_f.f_nsyms  = 0;
   internal_f.f_flags  = F_AR32WR | F_LNNO; /* XXX is this correct ?  */
-  
-  if (   ! bfd_set_start_address (abfd, 0)
+
+  if (   ! bfd_set_start_address (abfd, (bfd_vma) 0)
       || ! bfd_coff_set_arch_mach_hook (abfd, & internal_f))
-    return false;
+    return FALSE;
 
   if (bfd_coff_mkobject_hook (abfd, (PTR) & internal_f, NULL) == NULL)
-    return false;
+    return FALSE;
 
   coff_data (abfd)->pe = 1;
-#ifdef THUMBPEMAGIC  
+#ifdef THUMBPEMAGIC
   if (vars.magic == THUMBPEMAGIC)
     /* Stop some linker warnings about thumb code not supporting interworking.  */
     coff_data (abfd)->flags |= F_INTERWORK | F_INTERWORK_SET;
 #endif
-    
+
   /* Switch from file contents to memory contents.  */
   bfd_cache_close (abfd);
 
@@ -1043,12 +1022,10 @@ pe_ILF_build_a_bfd (bfd *           abfd,
   /* Now create a symbol describing the imported value.  */
   switch (import_type)
     {
-      bfd_byte * ptr;
-      
     case IMPORT_CODE:
       pe_ILF_make_a_symbol (& vars, "", symbol_name, text,
                            BSF_NOT_AT_END | BSF_FUNCTION);
-      
+
       /* Create an import symbol for the DLL, without the
        .dll suffix.  */
       ptr = strrchr (source_dll, '.');
@@ -1062,7 +1039,7 @@ pe_ILF_build_a_bfd (bfd *           abfd,
     case IMPORT_DATA:
       /* Nothing to do here.  */
       break;
-      
+
     default:
       /* XXX code not yet written.  */
       abort ();
@@ -1071,22 +1048,22 @@ pe_ILF_build_a_bfd (bfd *           abfd,
   /* Point the bfd at the symbol table.  */
   obj_symbols (abfd) = vars.sym_cache;
   bfd_get_symcount (abfd) = vars.sym_index;
-  
+
   obj_raw_syments (abfd) = vars.native_syms;
   obj_raw_syment_count (abfd) = vars.sym_index;
 
   obj_coff_external_syms (abfd) = (PTR) vars.esym_table;
-  obj_coff_keep_syms (abfd) = true;
-  
+  obj_coff_keep_syms (abfd) = TRUE;
+
   obj_convert (abfd) = vars.sym_table;
   obj_conv_table_size (abfd) = vars.sym_index;
-  
+
   obj_coff_strings (abfd) = vars.string_table;
-  obj_coff_keep_strings (abfd) = true;
+  obj_coff_keep_strings (abfd) = TRUE;
 
   abfd->flags |= HAS_SYMS;
 
-  return true;
+  return TRUE;
 }
 
 /* We have detected a Image Library Format archive element.
@@ -1099,28 +1076,28 @@ pe_ILF_object_p (bfd * abfd)
   bfd_byte *      symbol_name;
   bfd_byte *      source_dll;
   unsigned int    machine;
-  unsigned long   size;
+  bfd_size_type   size;
   unsigned int    ordinal;
   unsigned int    types;
-  unsigned short  magic;
-  
+  unsigned int    magic;
+
   /* Upon entry the first four buyes of the ILF header have
       already been read.  Now read the rest of the header.  */
-  if (bfd_read (buffer, 1, 16, abfd) != 16)
+  if (bfd_bread (buffer, (bfd_size_type) 16, abfd) != 16)
     return NULL;
 
   ptr = buffer;
-  
+
   /*  We do not bother to check the version number.
-      version = bfd_h_get_16 (abfd, ptr);  */
+      version = H_GET_16 (abfd, ptr);  */
   ptr += 2;
 
-  machine = bfd_h_get_16 (abfd, ptr);
+  machine = H_GET_16 (abfd, ptr);
   ptr += 2;
 
   /* Check that the machine type is recognised.  */
   magic = 0;
-  
+
   switch (machine)
     {
     case IMAGE_FILE_MACHINE_UNKNOWN:
@@ -1128,23 +1105,23 @@ pe_ILF_object_p (bfd * abfd)
     case IMAGE_FILE_MACHINE_ALPHA64:
     case IMAGE_FILE_MACHINE_IA64:
       break;
-      
+
     case IMAGE_FILE_MACHINE_I386:
 #ifdef I386MAGIC
       magic = I386MAGIC;
 #endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_M68K:
 #ifdef MC68AGIC
       magic = MC68MAGIC;
 #endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_R3000:
     case IMAGE_FILE_MACHINE_R4000:
     case IMAGE_FILE_MACHINE_R10000:
-      
+
     case IMAGE_FILE_MACHINE_MIPS16:
     case IMAGE_FILE_MACHINE_MIPSFPU:
     case IMAGE_FILE_MACHINE_MIPSFPU16:
@@ -1152,40 +1129,40 @@ pe_ILF_object_p (bfd * abfd)
       magic = MIPS_ARCH_MAGIC_WINCE;
 #endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_SH3:
     case IMAGE_FILE_MACHINE_SH4:
 #ifdef SH_ARCH_MAGIC_WINCE
       magic = SH_ARCH_MAGIC_WINCE;
 #endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_ARM:
 #ifdef ARMPEMAGIC
       magic = ARMPEMAGIC;
-#endif      
+#endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_THUMB:
 #ifdef THUMBPEMAGIC
       {
        extern const bfd_target TARGET_LITTLE_SYM;
-       
+
        if (abfd->xvec == & TARGET_LITTLE_SYM)
          magic = THUMBPEMAGIC;
       }
-#endif      
+#endif
       break;
-      
+
     case IMAGE_FILE_MACHINE_POWERPC:
       /* We no longer support PowerPC.  */
     default:
       _bfd_error_handler
        (
 _("%s: Unrecognised machine type (0x%x) in Import Library Format archive"),
-         bfd_get_filename (abfd), machine);
+         bfd_archive_filename (abfd), machine);
       bfd_set_error (bfd_error_malformed_archive);
-       
+
       return NULL;
       break;
     }
@@ -1195,128 +1172,136 @@ _("%s: Unrecognised machine type (0x%x) in Import Library Format archive"),
       _bfd_error_handler
        (
 _("%s: Recognised but unhandled machine type (0x%x) in Import Library Format archive"),
-        bfd_get_filename (abfd), machine);
+        bfd_archive_filename (abfd), machine);
       bfd_set_error (bfd_error_wrong_format);
-       
+
       return NULL;
-    }  
+    }
 
   /* We do not bother to check the date.
-     date = bfd_h_get_32 (abfd, ptr);  */
+     date = H_GET_32 (abfd, ptr);  */
   ptr += 4;
-  
-  size = bfd_h_get_32 (abfd, ptr);
+
+  size = H_GET_32 (abfd, ptr);
   ptr += 4;
 
   if (size == 0)
     {
       _bfd_error_handler
        (_("%s: size field is zero in Import Library Format header"),
-        bfd_get_filename (abfd));
+        bfd_archive_filename (abfd));
       bfd_set_error (bfd_error_malformed_archive);
-       
+
       return NULL;
     }
 
-  ordinal = bfd_h_get_16 (abfd, ptr);
+  ordinal = H_GET_16 (abfd, ptr);
   ptr += 2;
 
-  types = bfd_h_get_16 (abfd, ptr);
+  types = H_GET_16 (abfd, ptr);
   /* ptr += 2; */
 
   /* Now read in the two strings that follow.  */
   ptr = bfd_alloc (abfd, size);
   if (ptr == NULL)
     return NULL;
-  
-  if (bfd_read (ptr, 1, size, abfd) != size)
-    return NULL;
+
+  if (bfd_bread (ptr, size, abfd) != size)
+    {
+      bfd_release (abfd, ptr);
+      return NULL;
+    }
 
   symbol_name = ptr;
   source_dll  = ptr + strlen (ptr) + 1;
-  
+
   /* Verify that the strings are null terminated.  */
   if (ptr[size - 1] != 0 || ((unsigned long) (source_dll - ptr) >= size))
     {
       _bfd_error_handler
        (_("%s: string not null terminated in ILF object file."),
-        bfd_get_filename (abfd));
+        bfd_archive_filename (abfd));
       bfd_set_error (bfd_error_malformed_archive);
-       
+      bfd_release (abfd, ptr);
       return NULL;
     }
-  
+
   /* Now construct the bfd.  */
   if (! pe_ILF_build_a_bfd (abfd, magic, symbol_name,
                            source_dll, ordinal, types))
-    return NULL;
-  
+    {
+      bfd_release (abfd, ptr);
+      return NULL;
+    }
+
   return abfd->xvec;
 }
 
 static const bfd_target *
 pe_bfd_object_p (bfd * abfd)
 {
-  /* We need to handle a PE image correctly.  In PE images created by
-     the GNU linker, the offset to the COFF header is always the size.
-     However, this is not the case in images generated by other PE
-     linkers.  The PE format stores a four byte offset to the PE
-     signature just before the COFF header at location 0x3c of the file.
-     We pick up that offset, verify that the PE signature is there, and
-     then set ourselves up to read in the COFF header.  */
   bfd_byte buffer[4];
+  struct external_PEI_DOS_hdr dos_hdr;
+  struct external_PEI_IMAGE_hdr image_hdr;
   file_ptr offset;
-  unsigned long signature;
 
   /* Detect if this a Microsoft Import Library Format element.  */
-  if (bfd_seek (abfd, 0x00, SEEK_SET) != 0
-      || bfd_read (buffer, 1, 4, abfd) != 4)
+  if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0
+      || bfd_bread (buffer, (bfd_size_type) 4, abfd) != 4)
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
-  
-  signature = bfd_h_get_32 (abfd, buffer);
-  
-  if (signature == 0xffff0000)
+
+  if (H_GET_32 (abfd, buffer) == 0xffff0000)
     return pe_ILF_object_p (abfd);
-  
-  if (bfd_seek (abfd, 0x3c, SEEK_SET) != 0
-      || bfd_read (buffer, 1, 4, abfd) != 4)
+
+  if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0
+      || bfd_bread (&dos_hdr, (bfd_size_type) sizeof (dos_hdr), abfd)
+        != sizeof (dos_hdr))
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
 
-  offset = bfd_h_get_32 (abfd, buffer);
+  /* There are really two magic numbers involved; the magic number
+     that says this is a NT executable (PEI) and the magic number that
+     determines the architecture.  The former is DOSMAGIC, stored in
+     the e_magic field.  The latter is stored in the f_magic field.
+     If the NT magic number isn't valid, the architecture magic number
+     could be mimicked by some other field (specifically, the number
+     of relocs in section 3).  Since this routine can only be called
+     correctly for a PEI file, check the e_magic number here, and, if
+     it doesn't match, clobber the f_magic number so that we don't get
+     a false match.  */
+  if (H_GET_16 (abfd, dos_hdr.e_magic) != DOSMAGIC)
+    {
+      bfd_set_error (bfd_error_wrong_format);
+      return NULL;
+    }
 
+  offset = H_GET_32 (abfd, dos_hdr.e_lfanew);
   if (bfd_seek (abfd, offset, SEEK_SET) != 0
-      || bfd_read (buffer, 1, 4, abfd) != 4)
+      || (bfd_bread (&image_hdr, (bfd_size_type) sizeof (image_hdr), abfd)
+         != sizeof (image_hdr)))
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
 
-  signature = bfd_h_get_32 (abfd, buffer);
-
-  if (signature != 0x4550)
+  if (H_GET_32 (abfd, image_hdr.nt_signature) != 0x4550)
     {
       bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
-  
+
   /* Here is the hack.  coff_object_p wants to read filhsz bytes to
-     pick up the COFF header.  We adjust so that that will work.  20
-     is the size of the i386 COFF filehdr.  */
-  if (bfd_seek (abfd,
-               (bfd_tell (abfd)
-                - bfd_coff_filhsz (abfd)
-                + 20),
-               SEEK_SET)
-      != 0)
+     pick up the COFF header for PE, see "struct external_PEI_filehdr"
+     in include/coff/pe.h.  We adjust so that that will work. */
+  if (bfd_seek (abfd, (file_ptr) (offset - sizeof (dos_hdr)), SEEK_SET) != 0)
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
This page took 0.03948 seconds and 4 git commands to generate.