Update to ISO-C90 and fix formatting
[deliverable/binutils-gdb.git] / bfd / coffgen.c
index 33a85cf2d534404189c7cdee2dc04a699d567935..310e6c9e9645cb330b81002991e6d858e14da1a0 100644 (file)
@@ -1,22 +1,24 @@
 /* Support for the generic parts of COFF, for BFD.
-   Copyright 1990, 1991, 1992, 1993, 1994 Free Software Foundation, Inc.
+   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
+   2000, 2001, 2002, 2003, 2004, 2005
+   Free Software Foundation, Inc.
    Written by Cygnus Support.
 
-This file is part of BFD, the Binary File Descriptor library.
+   This file is part of BFD, the Binary File Descriptor library.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   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.  */
+   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., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 /* Most of this hacked by  Steve Chamberlain, sac@cygnus.com.
    Split out of coffcode.h by Ian Taylor, ian@cygnus.com.  */
@@ -41,68 +43,85 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include "coff/internal.h"
 #include "libcoff.h"
 
-static boolean coff_write_symbol PARAMS ((bfd *, asymbol *,
-                                         combined_entry_type *,
-                                         unsigned int *));
-static boolean coff_write_alien_symbol PARAMS ((bfd *, asymbol *,
-                                               unsigned int *));
-static boolean coff_write_native_symbol PARAMS ((bfd *, coff_symbol_type *,
-                                                unsigned int *));
-
-static asection bfd_debug_section = { "*DEBUG*" };
-
 /* Take a section header read from a coff file (in HOST byte order),
    and make a BFD "section" out of it.  This is used by ECOFF.  */
-static          boolean
-make_a_section_from_file (abfd, hdr, target_index)
-     bfd            *abfd;
-     struct internal_scnhdr  *hdr;
-     unsigned int target_index;
+
+static bfd_boolean
+make_a_section_from_file (bfd *abfd,
+                         struct internal_scnhdr *hdr,
+                         unsigned int target_index)
 {
-  asection       *return_section;
+  asection *return_section;
   char *name;
-    
-  /* Assorted wastage to null-terminate the name, thanks AT&T! */
-  name = bfd_alloc(abfd, sizeof (hdr->s_name)+1);
-  if (name == NULL) {
-      bfd_set_error (bfd_error_no_memory);
-      return false;
-    }
-  strncpy(name, (char *) &hdr->s_name[0], sizeof (hdr->s_name));
-  name[sizeof (hdr->s_name)] = 0;
+  bfd_boolean result = TRUE;
+  flagword flags;
 
-  return_section = bfd_make_section(abfd, name);
-  if (return_section == NULL)
-    return_section = bfd_coff_make_section_hook (abfd, name);
+  name = NULL;
 
-  /* Handle several sections of the same name.  For example, if an executable
-     has two .bss sections, GDB better be able to find both of them
-     (PR 3562).  */
-  if (return_section == NULL)
-    return_section = bfd_make_section_anyway (abfd, name);
+  /* Handle long section names as in PE.  */
+  if (bfd_coff_long_section_names (abfd)
+      && hdr->s_name[0] == '/')
+    {
+      char buf[SCNNMLEN];
+      long strindex;
+      char *p;
+      const char *strings;
+
+      memcpy (buf, hdr->s_name + 1, SCNNMLEN - 1);
+      buf[SCNNMLEN - 1] = '\0';
+      strindex = strtol (buf, &p, 10);
+      if (*p == '\0' && strindex >= 0)
+       {
+         strings = _bfd_coff_read_string_table (abfd);
+         if (strings == NULL)
+           return FALSE;
+         /* FIXME: For extra safety, we should make sure that
+             strindex does not run us past the end, but right now we
+             don't know the length of the string table.  */
+         strings += strindex;
+         name = bfd_alloc (abfd, (bfd_size_type) strlen (strings) + 1);
+         if (name == NULL)
+           return FALSE;
+         strcpy (name, strings);
+       }
+    }
 
-  if (return_section == NULL)
-    return false;
+  if (name == NULL)
+    {
+      /* Assorted wastage to null-terminate the name, thanks AT&T! */
+      name = bfd_alloc (abfd, (bfd_size_type) sizeof (hdr->s_name) + 1);
+      if (name == NULL)
+       return FALSE;
+      strncpy (name, (char *) &hdr->s_name[0], sizeof (hdr->s_name));
+      name[sizeof (hdr->s_name)] = 0;
+    }
 
-  /* s_paddr is presumed to be = to s_vaddr */
+  return_section = bfd_make_section_anyway (abfd, name);
+  if (return_section == NULL)
+    return FALSE;
 
   return_section->vma = hdr->s_vaddr;
-  return_section->_raw_size = hdr->s_size;
+  return_section->lma = hdr->s_paddr;
+  return_section->size = hdr->s_size;
   return_section->filepos = hdr->s_scnptr;
-  return_section->rel_filepos =  hdr->s_relptr;
+  return_section->rel_filepos = hdr->s_relptr;
   return_section->reloc_count = hdr->s_nreloc;
 
   bfd_coff_set_alignment_hook (abfd, return_section, hdr);
 
-  return_section->line_filepos =  hdr->s_lnnoptr;
+  return_section->line_filepos = hdr->s_lnnoptr;
 
   return_section->lineno_count = hdr->s_nlnno;
   return_section->userdata = NULL;
-  return_section->next = (asection *) NULL;
-  return_section->flags = bfd_coff_styp_to_sec_flags_hook (abfd, hdr);
-
+  return_section->next = NULL;
   return_section->target_index = target_index;
 
+  if (! bfd_coff_styp_to_sec_flags_hook (abfd, hdr, name, return_section,
+                                        & flags))
+    result = FALSE;
+
+  return_section->flags = flags;
+
   /* At least on i386-coff, the line number count for a shared library
      section must be ignored.  */
   if ((return_section->flags & SEC_COFF_SHARED_LIBRARY) != 0)
@@ -110,61 +129,30 @@ make_a_section_from_file (abfd, hdr, target_index)
 
   if (hdr->s_nreloc != 0)
     return_section->flags |= SEC_RELOC;
-  /* FIXME: should this check 'hdr->s_size > 0' */
+  /* FIXME: should this check 'hdr->s_size > 0' */
   if (hdr->s_scnptr != 0)
     return_section->flags |= SEC_HAS_CONTENTS;
-  return true;
+
+  return result;
 }
 
 /* Read in a COFF object and make it into a BFD.  This is used by
    ECOFF as well.  */
 
 static const bfd_target *
-coff_real_object_p (abfd, nscns, internal_f, internal_a)
-     bfd            *abfd;
-     unsigned        nscns;
-     struct internal_filehdr *internal_f;
-     struct internal_aouthdr *internal_a;
+coff_real_object_p (bfd *abfd,
+                   unsigned nscns,
+                   struct internal_filehdr *internal_f,
+                   struct internal_aouthdr *internal_a)
 {
-  PTR tdata;
-  size_t          readsize;    /* length of file_info */
+  flagword oflags = abfd->flags;
+  bfd_vma ostart = bfd_get_start_address (abfd);
+  void * tdata;
+  void * tdata_save;
+  bfd_size_type readsize;      /* Length of file_info.  */
   unsigned int scnhsz;
   char *external_sections;
 
-  /* Build a play area */
-  tdata = bfd_coff_mkobject_hook (abfd, (PTR) internal_f, (PTR) internal_a);
-  if (tdata == NULL)
-    return 0;
-
-  scnhsz = bfd_coff_scnhsz (abfd);
-  readsize = nscns * scnhsz;
-  external_sections = (char *)bfd_alloc(abfd, readsize);
-  if (!external_sections)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      goto fail;
-    }
-
-  if (bfd_read((PTR)external_sections, 1, readsize, abfd) != readsize) {
-    goto fail;
-  }
-
-  /* Now copy data as required; construct all asections etc */
-  if (nscns != 0) {
-    unsigned int    i;
-    for (i = 0; i < nscns; i++) {
-      struct internal_scnhdr tmp;
-      bfd_coff_swap_scnhdr_in(abfd, (PTR) (external_sections + i * scnhsz),
-                             (PTR) &tmp);
-      make_a_section_from_file(abfd,&tmp, i+1);
-    }
-  }
-
-/*  make_abs_section(abfd);*/
-  
-  if (bfd_coff_set_arch_mach_hook (abfd, (PTR) internal_f) == false)
-    goto fail;
-
   if (!(internal_f->f_flags & F_RELFLG))
     abfd->flags |= HAS_RELOC;
   if ((internal_f->f_flags & F_EXEC))
@@ -178,7 +166,7 @@ coff_real_object_p (abfd, nscns, internal_f, internal_a)
   if ((internal_f->f_flags & F_EXEC) != 0)
     abfd->flags |= D_PAGED;
 
-  bfd_get_symcount(abfd) = internal_f->f_nsyms;
+  bfd_get_symcount (abfd) = internal_f->f_nsyms;
   if (internal_f->f_nsyms)
     abfd->flags |= HAS_SYMS;
 
@@ -187,341 +175,537 @@ coff_real_object_p (abfd, nscns, internal_f, internal_a)
   else
     bfd_get_start_address (abfd) = 0;
 
+  /* Set up the tdata area.  ECOFF uses its own routine, and overrides
+     abfd->flags.  */
+  tdata_save = abfd->tdata.any;
+  tdata = bfd_coff_mkobject_hook (abfd, (void *) internal_f, (void *) internal_a);
+  if (tdata == NULL)
+    goto fail2;
+
+  scnhsz = bfd_coff_scnhsz (abfd);
+  readsize = (bfd_size_type) nscns * scnhsz;
+  external_sections = bfd_alloc (abfd, readsize);
+  if (!external_sections)
+    goto fail;
+
+  if (bfd_bread ((void *) external_sections, readsize, abfd) != readsize)
+    goto fail;
+
+  /* Set the arch/mach *before* swapping in sections; section header swapping
+     may depend on arch/mach info.  */
+  if (! bfd_coff_set_arch_mach_hook (abfd, (void *) internal_f))
+    goto fail;
+
+  /* Now copy data as required; construct all asections etc.  */
+  if (nscns != 0)
+    {
+      unsigned int i;
+      for (i = 0; i < nscns; i++)
+       {
+         struct internal_scnhdr tmp;
+         bfd_coff_swap_scnhdr_in (abfd,
+                                  (void *) (external_sections + i * scnhsz),
+                                  (void *) & tmp);
+         if (! make_a_section_from_file (abfd, &tmp, i + 1))
+           goto fail;
+       }
+    }
+
   return abfd->xvec;
+
  fail:
-  bfd_release(abfd, tdata);
-  return (const bfd_target *)NULL;
+  bfd_release (abfd, tdata);
+ fail2:
+  abfd->tdata.any = tdata_save;
+  abfd->flags = oflags;
+  bfd_get_start_address (abfd) = ostart;
+  return (const bfd_target *) NULL;
 }
 
 /* Turn a COFF file into a BFD, but fail with bfd_error_wrong_format if it is
    not a COFF file.  This is also used by ECOFF.  */
 
 const bfd_target *
-coff_object_p (abfd)
-     bfd            *abfd;
+coff_object_p (bfd *abfd)
 {
-  unsigned int filhsz;
-  unsigned int aoutsz;
-  int   nscns;
-  PTR filehdr;
+  bfd_size_type filhsz;
+  bfd_size_type aoutsz;
+  unsigned int nscns;
+  void * filehdr;
   struct internal_filehdr internal_f;
   struct internal_aouthdr internal_a;
 
-  /* figure out how much to read */
+  /* Figure out how much to read.  */
   filhsz = bfd_coff_filhsz (abfd);
   aoutsz = bfd_coff_aoutsz (abfd);
 
   filehdr = bfd_alloc (abfd, filhsz);
   if (filehdr == NULL)
-    return 0;
-  if (bfd_read(filehdr, 1, filhsz, abfd) != filhsz)
+    return NULL;
+  if (bfd_bread (filehdr, filhsz, abfd) != filhsz)
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
-      return 0;
+      bfd_release (abfd, filehdr);
+      return NULL;
     }
-  bfd_coff_swap_filehdr_in(abfd, filehdr, &internal_f);
+  bfd_coff_swap_filehdr_in (abfd, filehdr, &internal_f);
   bfd_release (abfd, filehdr);
 
-  if (bfd_coff_bad_format_hook (abfd, &internal_f) == false) {
-    bfd_set_error (bfd_error_wrong_format);
-    return 0;
-  }
-  nscns =internal_f.f_nscns;
+  /* The XCOFF format has two sizes for the f_opthdr.  SMALL_AOUTSZ
+     (less than aoutsz) used in object files and AOUTSZ (equal to
+     aoutsz) in executables.  The bfd_coff_swap_aouthdr_in function
+     expects this header to be aoutsz bytes in length, so we use that
+     value in the call to bfd_alloc below.  But we must be careful to
+     only read in f_opthdr bytes in the call to bfd_bread.  We should
+     also attempt to catch corrupt or non-COFF binaries with a strange
+     value for f_opthdr.  */
+  if (! bfd_coff_bad_format_hook (abfd, &internal_f)
+      || internal_f.f_opthdr > aoutsz)
+    {
+      bfd_set_error (bfd_error_wrong_format);
+      return NULL;
+    }
+  nscns = internal_f.f_nscns;
 
-  if (internal_f.f_opthdr) {
-    PTR opthdr;
+  if (internal_f.f_opthdr)
+    {
+      void * opthdr;
 
-    opthdr = bfd_alloc (abfd, aoutsz);
-    if (opthdr == NULL)
-      return 0;;
-    if (bfd_read(opthdr, 1,aoutsz, abfd) != aoutsz) {
-      return 0;
+      opthdr = bfd_alloc (abfd, aoutsz);
+      if (opthdr == NULL)
+       return NULL;
+      if (bfd_bread (opthdr, (bfd_size_type) internal_f.f_opthdr, abfd)
+         != internal_f.f_opthdr)
+       {
+         bfd_release (abfd, opthdr);
+         return NULL;
+       }
+      bfd_coff_swap_aouthdr_in (abfd, opthdr, (void *) &internal_a);
+      bfd_release (abfd, opthdr);
     }
-    bfd_coff_swap_aouthdr_in(abfd, opthdr, (PTR)&internal_a);
-  }
-
-  /* Seek past the opt hdr stuff */
-  if (bfd_seek(abfd, (file_ptr) (internal_f.f_opthdr + filhsz), SEEK_SET)
-      != 0)
-    return NULL;
 
-  return coff_real_object_p(abfd, nscns, &internal_f,
-                           (internal_f.f_opthdr != 0
-                            ? &internal_a
-                            : (struct internal_aouthdr *) NULL));
+  return coff_real_object_p (abfd, nscns, &internal_f,
+                            (internal_f.f_opthdr != 0
+                             ? &internal_a
+                             : (struct internal_aouthdr *) NULL));
 }
 
 /* Get the BFD section from a COFF symbol section number.  */
 
-struct sec *
-coff_section_from_bfd_index (abfd, index)
-     bfd            *abfd;
-     int             index;
+asection *
+coff_section_from_bfd_index (bfd *abfd, int index)
 {
-  struct sec *answer = abfd->sections;
+  struct bfd_section *answer = abfd->sections;
 
-  if (index == N_ABS) 
-  {
-    return &bfd_abs_section;
-  }
+  if (index == N_ABS)
+    return bfd_abs_section_ptr;
   if (index == N_UNDEF)
-  {
-    return &bfd_und_section;
-  }
-  if(index == N_DEBUG)
-  {
-    return &bfd_debug_section;
-    
-  }
-  
-  while (answer) {
+    return bfd_und_section_ptr;
+  if (index == N_DEBUG)
+    return bfd_abs_section_ptr;
+
+  while (answer)
+    {
       if (answer->target_index == index)
-       return answer;
+       return answer;
       answer = answer->next;
     }
 
   /* We should not reach this point, but the SCO 3.2v4 /lib/libc_s.a
      has a bad symbol table in biglitpow.o.  */
-  return &bfd_und_section;
+  return bfd_und_section_ptr;
 }
 
 /* Get the upper bound of a COFF symbol table.  */
 
 long
-coff_get_symtab_upper_bound(abfd)
-bfd            *abfd;
+coff_get_symtab_upper_bound (bfd *abfd)
 {
-  if (!bfd_coff_slurp_symbol_table(abfd))
+  if (!bfd_coff_slurp_symbol_table (abfd))
     return -1;
 
-  return (bfd_get_symcount(abfd) + 1) * (sizeof(coff_symbol_type *));
+  return (bfd_get_symcount (abfd) + 1) * (sizeof (coff_symbol_type *));
 }
 
-
 /* Canonicalize a COFF symbol table.  */
 
 long
-coff_get_symtab (abfd, alocation)
-     bfd            *abfd;
-     asymbol       **alocation;
+coff_canonicalize_symtab (bfd *abfd, asymbol **alocation)
+{
+  unsigned int counter;
+  coff_symbol_type *symbase;
+  coff_symbol_type **location = (coff_symbol_type **) alocation;
+
+  if (!bfd_coff_slurp_symbol_table (abfd))
+    return -1;
+
+  symbase = obj_symbols (abfd);
+  counter = bfd_get_symcount (abfd);
+  while (counter-- > 0)
+    *location++ = symbase++;
+
+  *location = NULL;
+
+  return bfd_get_symcount (abfd);
+}
+
+/* Get the name of a symbol.  The caller must pass in a buffer of size
+   >= SYMNMLEN + 1.  */
+
+const char *
+_bfd_coff_internal_syment_name (bfd *abfd,
+                               const struct internal_syment *sym,
+                               char *buf)
 {
-    unsigned int    counter = 0;
-    coff_symbol_type *symbase;
-    coff_symbol_type **location = (coff_symbol_type **) (alocation);
-    if (!bfd_coff_slurp_symbol_table(abfd))
-     return -1;
-
-    symbase = obj_symbols(abfd);
-    while (counter <  bfd_get_symcount(abfd))
+  /* FIXME: It's not clear this will work correctly if sizeof
+     (_n_zeroes) != 4.  */
+  if (sym->_n._n_n._n_zeroes != 0
+      || sym->_n._n_n._n_offset == 0)
+    {
+      memcpy (buf, sym->_n._n_name, SYMNMLEN);
+      buf[SYMNMLEN] = '\0';
+      return buf;
+    }
+  else
     {
-       /* This nasty code looks at the symbol to decide whether or
-          not it is descibes a constructor/destructor entry point. It
-          is structured this way to (hopefully) speed non matches */
-#if 0  
-       if (0 && symbase->symbol.name[9] == '$') 
+      const char *strings;
+
+      BFD_ASSERT (sym->_n._n_n._n_offset >= STRING_SIZE_SIZE);
+      strings = obj_coff_strings (abfd);
+      if (strings == NULL)
        {
-           bfd_constructor_entry(abfd, 
-                                (asymbol **)location,
-                                 symbase->symbol.name[10] == 'I' ?
-                                 "CTOR" : "DTOR");
+         strings = _bfd_coff_read_string_table (abfd);
+         if (strings == NULL)
+           return NULL;
        }
-#endif
-       *(location++) = symbase++;
-       counter++;
+      return strings + sym->_n._n_n._n_offset;
+    }
+}
+
+/* Read in and swap the relocs.  This returns a buffer holding the
+   relocs for section SEC in file ABFD.  If CACHE is TRUE and
+   INTERNAL_RELOCS is NULL, the relocs read in will be saved in case
+   the function is called again.  If EXTERNAL_RELOCS is not NULL, it
+   is a buffer large enough to hold the unswapped relocs.  If
+   INTERNAL_RELOCS is not NULL, it is a buffer large enough to hold
+   the swapped relocs.  If REQUIRE_INTERNAL is TRUE, then the return
+   value must be INTERNAL_RELOCS.  The function returns NULL on error.  */
+
+struct internal_reloc *
+_bfd_coff_read_internal_relocs (bfd *abfd,
+                               asection *sec,
+                               bfd_boolean cache,
+                               bfd_byte *external_relocs,
+                               bfd_boolean require_internal,
+                               struct internal_reloc *internal_relocs)
+{
+  bfd_size_type relsz;
+  bfd_byte *free_external = NULL;
+  struct internal_reloc *free_internal = NULL;
+  bfd_byte *erel;
+  bfd_byte *erel_end;
+  struct internal_reloc *irel;
+  bfd_size_type amt;
+
+  if (coff_section_data (abfd, sec) != NULL
+      && coff_section_data (abfd, sec)->relocs != NULL)
+    {
+      if (! require_internal)
+       return coff_section_data (abfd, sec)->relocs;
+      memcpy (internal_relocs, coff_section_data (abfd, sec)->relocs,
+             sec->reloc_count * sizeof (struct internal_reloc));
+      return internal_relocs;
+    }
+
+  relsz = bfd_coff_relsz (abfd);
+
+  amt = sec->reloc_count * relsz;
+  if (external_relocs == NULL)
+    {
+      free_external = bfd_malloc (amt);
+      if (free_external == NULL && sec->reloc_count > 0)
+       goto error_return;
+      external_relocs = free_external;
+    }
+
+  if (bfd_seek (abfd, sec->rel_filepos, SEEK_SET) != 0
+      || bfd_bread (external_relocs, amt, abfd) != amt)
+    goto error_return;
+
+  if (internal_relocs == NULL)
+    {
+      amt = sec->reloc_count;
+      amt *= sizeof (struct internal_reloc);
+      free_internal = bfd_malloc (amt);
+      if (free_internal == NULL && sec->reloc_count > 0)
+       goto error_return;
+      internal_relocs = free_internal;
+    }
+
+  /* Swap in the relocs.  */
+  erel = external_relocs;
+  erel_end = erel + relsz * sec->reloc_count;
+  irel = internal_relocs;
+  for (; erel < erel_end; erel += relsz, irel++)
+    bfd_coff_swap_reloc_in (abfd, (void *) erel, (void *) irel);
+
+  if (free_external != NULL)
+    {
+      free (free_external);
+      free_external = NULL;
+    }
+
+  if (cache && free_internal != NULL)
+    {
+      if (coff_section_data (abfd, sec) == NULL)
+       {
+         amt = sizeof (struct coff_section_tdata);
+         sec->used_by_bfd = bfd_zalloc (abfd, amt);
+         if (sec->used_by_bfd == NULL)
+           goto error_return;
+         coff_section_data (abfd, sec)->contents = NULL;
+       }
+      coff_section_data (abfd, sec)->relocs = free_internal;
     }
-    *location++ = 0;
-    return bfd_get_symcount(abfd);
+
+  return internal_relocs;
+
+ error_return:
+  if (free_external != NULL)
+    free (free_external);
+  if (free_internal != NULL)
+    free (free_internal);
+  return NULL;
 }
 
 /* Set lineno_count for the output sections of a COFF file.  */
 
 int
-coff_count_linenumbers (abfd)
-     bfd            *abfd;
+coff_count_linenumbers (bfd *abfd)
 {
-  unsigned int    limit = bfd_get_symcount(abfd);
-  unsigned int    i;
+  unsigned int limit = bfd_get_symcount (abfd);
+  unsigned int i;
   int total = 0;
-  asymbol       **p;
- {
-   asection       *s = abfd->sections->output_section;
-   while (s) {
-     BFD_ASSERT(s->lineno_count == 0);
-     s = s->next;
-   }
- }
-
-
-  for (p = abfd->outsymbols, i = 0; i < limit; i++, p++) {
-    asymbol        *q_maybe = *p;
-    if (bfd_asymbol_flavour(q_maybe) == bfd_target_coff_flavour) {
-      coff_symbol_type *q = coffsymbol(q_maybe);
-      if (q->lineno) {
-       /*
-         This symbol has a linenumber, increment the owning
-         section's linenumber count
-         */
-       alent          *l = q->lineno;
-       q->symbol.section->output_section->lineno_count++;
-       total ++;
-       l++;
-       while (l->line_number) {
-         total ++;
-         q->symbol.section->output_section->lineno_count++;
-         l++;
+  asymbol **p;
+  asection *s;
+
+  if (limit == 0)
+    {
+      /* This may be from the backend linker, in which case the
+         lineno_count in the sections is correct.  */
+      for (s = abfd->sections; s != NULL; s = s->next)
+       total += s->lineno_count;
+      return total;
+    }
+
+  for (s = abfd->sections; s != NULL; s = s->next)
+    BFD_ASSERT (s->lineno_count == 0);
+
+  for (p = abfd->outsymbols, i = 0; i < limit; i++, p++)
+    {
+      asymbol *q_maybe = *p;
+
+      if (bfd_family_coff (bfd_asymbol_bfd (q_maybe)))
+       {
+         coff_symbol_type *q = coffsymbol (q_maybe);
+
+         /* The AIX 4.1 compiler can sometimes generate line numbers
+             attached to debugging symbols.  We try to simply ignore
+             those here.  */
+         if (q->lineno != NULL
+             && q->symbol.section->owner != NULL)
+           {
+             /* This symbol has line numbers.  Increment the owning
+                section's linenumber count.  */
+             alent *l = q->lineno;
+
+             do
+               {
+                 asection * sec = q->symbol.section->output_section;
+
+                 /* Do not try to update fields in read-only sections.  */
+                 if (! bfd_is_const_section (sec))
+                   sec->lineno_count ++;
+
+                 ++total;
+                 ++l;
+               }
+             while (l->line_number != 0);
+           }
        }
-      }
     }
-  }
+
   return total;
 }
 
 /* Takes a bfd and a symbol, returns a pointer to the coff specific
    area of the symbol if there is one.  */
 
-/*ARGSUSED*/
 coff_symbol_type *
-coff_symbol_from (ignore_abfd, symbol)
-     bfd            *ignore_abfd;
-     asymbol        *symbol;
+coff_symbol_from (bfd *ignore_abfd ATTRIBUTE_UNUSED,
+                 asymbol *symbol)
 {
-  if (bfd_asymbol_flavour(symbol) != bfd_target_coff_flavour)
-    return (coff_symbol_type *)NULL;
+  if (!bfd_family_coff (bfd_asymbol_bfd (symbol)))
+    return (coff_symbol_type *) NULL;
 
-  if (bfd_asymbol_bfd(symbol)->tdata.coff_obj_data == (coff_data_type*)NULL)
-    return (coff_symbol_type *)NULL;
+  if (bfd_asymbol_bfd (symbol)->tdata.coff_obj_data == (coff_data_type *) NULL)
+    return (coff_symbol_type *) NULL;
 
-  return  (coff_symbol_type *) symbol;
+  return (coff_symbol_type *) symbol;
 }
 
 static void
-fixup_symbol_value (coff_symbol_ptr, syment)
-     coff_symbol_type *coff_symbol_ptr;
-     struct internal_syment *syment;
+fixup_symbol_value (bfd *abfd,
+                   coff_symbol_type *coff_symbol_ptr,
+                   struct internal_syment *syment)
 {
-
-  /* Normalize the symbol flags */
-  if (bfd_is_com_section (coff_symbol_ptr->symbol.section)) {
-    /* a common symbol is undefined with a value */
-    syment->n_scnum = N_UNDEF;
-    syment->n_value = coff_symbol_ptr->symbol.value;
-  }
-  else if (coff_symbol_ptr->symbol.flags & BSF_DEBUGGING) {
-    syment->n_value = coff_symbol_ptr->symbol.value;
-  }
-  else if (coff_symbol_ptr->symbol.section == & bfd_und_section) {
-    syment->n_scnum = N_UNDEF;
-    syment->n_value = 0;
-  }
-  else {
-    if (coff_symbol_ptr->symbol.section) {
-      syment->n_scnum   =
-       coff_symbol_ptr->symbol.section->output_section->target_index;
-
-      syment->n_value =
-       coff_symbol_ptr->symbol.value +
-       coff_symbol_ptr->symbol.section->output_offset +
-        coff_symbol_ptr->symbol.section->output_section->vma;
+  /* Normalize the symbol flags.  */
+  if (bfd_is_com_section (coff_symbol_ptr->symbol.section))
+    {
+      /* A common symbol is undefined with a value.  */
+      syment->n_scnum = N_UNDEF;
+      syment->n_value = coff_symbol_ptr->symbol.value;
     }
-    else {
-       BFD_ASSERT(0);
-      /* This can happen, but I don't know why yet (steve@cygnus.com) */
-      syment->n_scnum = N_ABS;
+  else if ((coff_symbol_ptr->symbol.flags & BSF_DEBUGGING) != 0
+          && (coff_symbol_ptr->symbol.flags & BSF_DEBUGGING_RELOC) == 0)
+    {
       syment->n_value = coff_symbol_ptr->symbol.value;
     }
-  }
+  else if (bfd_is_und_section (coff_symbol_ptr->symbol.section))
+    {
+      syment->n_scnum = N_UNDEF;
+      syment->n_value = 0;
+    }
+  /* FIXME: Do we need to handle the absolute section here?  */
+  else
+    {
+      if (coff_symbol_ptr->symbol.section)
+       {
+         syment->n_scnum =
+           coff_symbol_ptr->symbol.section->output_section->target_index;
+
+         syment->n_value = (coff_symbol_ptr->symbol.value
+                            + coff_symbol_ptr->symbol.section->output_offset);
+         if (! obj_pe (abfd))
+            {
+              syment->n_value += (syment->n_sclass == C_STATLAB)
+                ? coff_symbol_ptr->symbol.section->output_section->lma
+                : coff_symbol_ptr->symbol.section->output_section->vma;
+            }
+       }
+      else
+       {
+         BFD_ASSERT (0);
+         /* This can happen, but I don't know why yet (steve@cygnus.com) */
+         syment->n_scnum = N_ABS;
+         syment->n_value = coff_symbol_ptr->symbol.value;
+       }
+    }
 }
 
-/* run through all the symbols in the symbol table and work out what
-   their indexes into the symbol table will be when output
+/* Run through all the symbols in the symbol table and work out what
+   their indexes into the symbol table will be when output.
 
- Coff requires that each C_FILE symbol points to the next one in the
- chain, and that the last one points to the first external symbol. We
- do that here too.
  Coff requires that each C_FILE symbol points to the next one in the
  chain, and that the last one points to the first external symbol. We
+   do that here too.  */
 
-*/
-boolean
-coff_renumber_symbols (bfd_ptr)
-     bfd *bfd_ptr;
+bfd_boolean
+coff_renumber_symbols (bfd *bfd_ptr, int *first_undef)
 {
-  unsigned int symbol_count = bfd_get_symcount(bfd_ptr);
+  unsigned int symbol_count = bfd_get_symcount (bfd_ptr);
   asymbol **symbol_ptr_ptr = bfd_ptr->outsymbols;
   unsigned int native_index = 0;
-  struct internal_syment *last_file = (struct internal_syment *)NULL;
+  struct internal_syment *last_file = NULL;
   unsigned int symbol_index;
 
   /* COFF demands that undefined symbols come after all other symbols.
-     Since we don't need to impose this extra knowledge on all our client
-     programs, deal with that here.  Sort the symbol table; just move the
-     undefined symbols to the end, leaving the rest alone.  */
+     Since we don't need to impose this extra knowledge on all our
+     client programs, deal with that here.  Sort the symbol table;
+     just move the undefined symbols to the end, leaving the rest
+     alone.  The O'Reilly book says that defined global symbols come
+     at the end before the undefined symbols, so we do that here as
+     well.  */
   /* @@ Do we have some condition we could test for, so we don't always
      have to do this?  I don't think relocatability is quite right, but
      I'm not certain.  [raeburn:19920508.1711EST]  */
   {
     asymbol **newsyms;
-    int i;
+    unsigned int i;
+    bfd_size_type amt;
 
-    newsyms = (asymbol **) bfd_alloc_by_size_t (bfd_ptr,
-                                               sizeof (asymbol *)
-                                               * (symbol_count + 1));
+    amt = sizeof (asymbol *) * ((bfd_size_type) symbol_count + 1);
+    newsyms = bfd_alloc (bfd_ptr, amt);
     if (!newsyms)
-      {
-       bfd_set_error (bfd_error_no_memory);
-       return false;
-      }
+      return FALSE;
     bfd_ptr->outsymbols = newsyms;
     for (i = 0; i < symbol_count; i++)
-      if (symbol_ptr_ptr[i]->section != &bfd_und_section)
+      if ((symbol_ptr_ptr[i]->flags & BSF_NOT_AT_END) != 0
+         || (!bfd_is_und_section (symbol_ptr_ptr[i]->section)
+             && !bfd_is_com_section (symbol_ptr_ptr[i]->section)
+             && ((symbol_ptr_ptr[i]->flags & BSF_FUNCTION) != 0
+                 || ((symbol_ptr_ptr[i]->flags & (BSF_GLOBAL | BSF_WEAK))
+                     == 0))))
        *newsyms++ = symbol_ptr_ptr[i];
+
+    for (i = 0; i < symbol_count; i++)
+      if ((symbol_ptr_ptr[i]->flags & BSF_NOT_AT_END) == 0
+         && !bfd_is_und_section (symbol_ptr_ptr[i]->section)
+         && (bfd_is_com_section (symbol_ptr_ptr[i]->section)
+             || ((symbol_ptr_ptr[i]->flags & BSF_FUNCTION) == 0
+                 && ((symbol_ptr_ptr[i]->flags & (BSF_GLOBAL | BSF_WEAK))
+                     != 0))))
+       *newsyms++ = symbol_ptr_ptr[i];
+
+    *first_undef = newsyms - bfd_ptr->outsymbols;
+
     for (i = 0; i < symbol_count; i++)
-      if (symbol_ptr_ptr[i]->section == &bfd_und_section)
+      if ((symbol_ptr_ptr[i]->flags & BSF_NOT_AT_END) == 0
+         && bfd_is_und_section (symbol_ptr_ptr[i]->section))
        *newsyms++ = symbol_ptr_ptr[i];
     *newsyms = (asymbol *) NULL;
     symbol_ptr_ptr = bfd_ptr->outsymbols;
   }
 
   for (symbol_index = 0; symbol_index < symbol_count; symbol_index++)
-      {
-       coff_symbol_type *coff_symbol_ptr = coff_symbol_from(bfd_ptr, symbol_ptr_ptr[symbol_index]);
-       if (coff_symbol_ptr && coff_symbol_ptr->native) {
+    {
+      coff_symbol_type *coff_symbol_ptr = coff_symbol_from (bfd_ptr, symbol_ptr_ptr[symbol_index]);
+      symbol_ptr_ptr[symbol_index]->udata.i = symbol_index;
+      if (coff_symbol_ptr && coff_symbol_ptr->native)
+       {
          combined_entry_type *s = coff_symbol_ptr->native;
          int i;
 
          if (s->u.syment.n_sclass == C_FILE)
-             {
-               if (last_file != (struct internal_syment *)NULL) {
-                 last_file->n_value = native_index;
-               }
-               last_file = &(s->u.syment);
-             }
-         else {
-
+           {
+             if (last_file != NULL)
+               last_file->n_value = native_index;
+             last_file = &(s->u.syment);
+           }
+         else
            /* Modify the symbol values according to their section and
-              type */
+              type.  */
+           fixup_symbol_value (bfd_ptr, coff_symbol_ptr, &(s->u.syment));
 
-           fixup_symbol_value(coff_symbol_ptr, &(s->u.syment));
-         }
-         for (i = 0; i < s->u.syment.n_numaux + 1; i++) {
-           s[i].offset = native_index ++;
-         }
-       }
-       else {
-         native_index++;
+         for (i = 0; i < s->u.syment.n_numaux + 1; i++)
+           s[i].offset = native_index++;
        }
-      }
+      else
+       native_index++;
+    }
+
   obj_conv_table_size (bfd_ptr) = native_index;
-  return true;
+
+  return TRUE;
 }
 
-/*
- Run thorough the symbol table again, and fix it so that all pointers to
- entries are changed to the entries' index in the output symbol table.
+/* Run thorough the symbol table again, and fix it so that all
+   pointers to entries are changed to the entries' index in the output
+   symbol table.  */
 
-*/
 void
-coff_mangle_symbols (bfd_ptr)
-     bfd *bfd_ptr;
+coff_mangle_symbols (bfd *bfd_ptr)
 {
   unsigned int symbol_count = bfd_get_symcount (bfd_ptr);
   asymbol **symbol_ptr_ptr = bfd_ptr->outsymbols;
@@ -530,7 +714,7 @@ coff_mangle_symbols (bfd_ptr)
   for (symbol_index = 0; symbol_index < symbol_count; symbol_index++)
     {
       coff_symbol_type *coff_symbol_ptr =
-       coff_symbol_from (bfd_ptr, symbol_ptr_ptr[symbol_index]);
+      coff_symbol_from (bfd_ptr, symbol_ptr_ptr[symbol_index]);
 
       if (coff_symbol_ptr && coff_symbol_ptr->native)
        {
@@ -541,10 +725,23 @@ coff_mangle_symbols (bfd_ptr)
            {
              /* FIXME: We should use a union here.  */
              s->u.syment.n_value =
-               ((combined_entry_type *) s->u.syment.n_value)->offset;
+               (bfd_vma)((combined_entry_type *)
+                         ((unsigned long) s->u.syment.n_value))->offset;
              s->fix_value = 0;
            }
-         for (i = 0; i < s->u.syment.n_numaux ; i++)
+         if (s->fix_line)
+           {
+             /* The value is the offset into the line number entries
+                 for the symbol's section.  On output, the symbol's
+                 section should be N_DEBUG.  */
+             s->u.syment.n_value =
+               (coff_symbol_ptr->symbol.section->output_section->line_filepos
+                + s->u.syment.n_value * bfd_coff_linesz (bfd_ptr));
+             coff_symbol_ptr->symbol.section =
+               coff_section_from_bfd_index (bfd_ptr, N_DEBUG);
+             BFD_ASSERT (coff_symbol_ptr->symbol.flags & BSF_DEBUGGING);
+           }
+         for (i = 0; i < s->u.syment.n_numaux; i++)
            {
              combined_entry_type *a = s + i + 1;
              if (a->fix_tag)
@@ -570,91 +767,113 @@ coff_mangle_symbols (bfd_ptr)
     }
 }
 
-static bfd_size_type string_size;
-static bfd_size_type debug_string_size;
-static asection *debug_string_section;
-
 static void
-coff_fix_symbol_name (abfd, symbol, native)
-     bfd *abfd;
-     asymbol *symbol;
-     combined_entry_type *native;
+coff_fix_symbol_name (bfd *abfd,
+                     asymbol *symbol,
+                     combined_entry_type *native,
+                     bfd_size_type *string_size_p,
+                     asection **debug_string_section_p,
+                     bfd_size_type *debug_string_size_p)
 {
-  unsigned int    name_length;
+  unsigned int name_length;
   union internal_auxent *auxent;
-  char *  name = ( char *)(symbol->name);
+  char *name = (char *) (symbol->name);
 
-  if (name == (char *) NULL) {
-    /* coff symbols always have names, so we'll make one up */
-    symbol->name = "strange";
-    name = (char *)symbol->name;
-  }
-  name_length = strlen(name);
+  if (name == NULL)
+    {
+      /* COFF symbols always have names, so we'll make one up.  */
+      symbol->name = "strange";
+      name = (char *) symbol->name;
+    }
+  name_length = strlen (name);
 
-  if (native->u.syment.n_sclass == C_FILE) {
-    strncpy(native->u.syment._n._n_name, ".file", SYMNMLEN);
-    auxent = &(native+1)->u.auxent;
+  if (native->u.syment.n_sclass == C_FILE
+      && native->u.syment.n_numaux > 0)
+    {
+      unsigned int filnmlen;
 
-    if (bfd_coff_long_filenames (abfd)) {
-      if (name_length <= FILNMLEN) {
-       strncpy(auxent->x_file.x_fname, name, FILNMLEN);
-      }
-      else {
-       auxent->x_file.x_n.x_offset = string_size + 4;
-       auxent->x_file.x_n.x_zeroes = 0;
-       string_size += name_length + 1;
-      }
-    }
-    else {
-      strncpy(auxent->x_file.x_fname, name, FILNMLEN);
-      if (name_length > FILNMLEN) {
-       name[FILNMLEN] = '\0';
-      }
-    }
-  }
-  else
-    {                          /* NOT A C_FILE SYMBOL */
-      if (name_length <= SYMNMLEN)
+      if (bfd_coff_force_symnames_in_strings (abfd))
+       {
+          native->u.syment._n._n_n._n_offset =
+             (*string_size_p + STRING_SIZE_SIZE);
+         native->u.syment._n._n_n._n_zeroes = 0;
+         *string_size_p += 6;  /* strlen(".file") + 1 */
+       }
+      else
+       strncpy (native->u.syment._n._n_name, ".file", SYMNMLEN);
+
+      auxent = &(native + 1)->u.auxent;
+
+      filnmlen = bfd_coff_filnmlen (abfd);
+
+      if (bfd_coff_long_filenames (abfd))
+       {
+         if (name_length <= filnmlen)
+           strncpy (auxent->x_file.x_fname, name, filnmlen);
+         else
+           {
+             auxent->x_file.x_n.x_offset = *string_size_p + STRING_SIZE_SIZE;
+             auxent->x_file.x_n.x_zeroes = 0;
+             *string_size_p += name_length + 1;
+           }
+       }
+      else
        {
-         /* This name will fit into the symbol neatly */
-         strncpy(native->u.syment._n._n_name, symbol->name, SYMNMLEN);
+         strncpy (auxent->x_file.x_fname, name, filnmlen);
+         if (name_length > filnmlen)
+           name[filnmlen] = '\0';
        }
-      else if (! bfd_coff_symname_in_debug (abfd, &native->u.syment))
+    }
+  else
+    {
+      if (name_length <= SYMNMLEN && !bfd_coff_force_symnames_in_strings (abfd))
+       /* This name will fit into the symbol neatly.  */
+       strncpy (native->u.syment._n._n_name, symbol->name, SYMNMLEN);
+
+      else if (!bfd_coff_symname_in_debug (abfd, &native->u.syment))
        {
-         native->u.syment._n._n_n._n_offset =  string_size + 4;
+         native->u.syment._n._n_n._n_offset = (*string_size_p
+                                               + STRING_SIZE_SIZE);
          native->u.syment._n._n_n._n_zeroes = 0;
-         string_size += name_length + 1;
+         *string_size_p += name_length + 1;
        }
       else
        {
-         long filepos;
-         bfd_byte buf[2];
+         file_ptr filepos;
+         bfd_byte buf[4];
+         int prefix_len = bfd_coff_debug_string_prefix_length (abfd);
 
          /* This name should be written into the .debug section.  For
             some reason each name is preceded by a two byte length
             and also followed by a null byte.  FIXME: We assume that
             the .debug section has already been created, and that it
             is large enough.  */
-         if (debug_string_section == (asection *) NULL)
-           debug_string_section = bfd_get_section_by_name (abfd, ".debug");
+         if (*debug_string_section_p == (asection *) NULL)
+           *debug_string_section_p = bfd_get_section_by_name (abfd, ".debug");
          filepos = bfd_tell (abfd);
-         bfd_put_16 (abfd, name_length + 1, buf);
-         if (! bfd_set_section_contents (abfd,
-                                         debug_string_section,
-                                         (PTR) buf,
-                                         (file_ptr) debug_string_size,
-                                         (bfd_size_type) 2)
-             || ! bfd_set_section_contents (abfd,
-                                            debug_string_section,
-                                            (PTR) symbol->name,
-                                            (file_ptr) debug_string_size + 2,
-                                            (bfd_size_type) name_length + 1))
+         if (prefix_len == 4)
+           bfd_put_32 (abfd, (bfd_vma) (name_length + 1), buf);
+         else
+           bfd_put_16 (abfd, (bfd_vma) (name_length + 1), buf);
+
+         if (!bfd_set_section_contents (abfd,
+                                        *debug_string_section_p,
+                                        (void *) buf,
+                                        (file_ptr) *debug_string_size_p,
+                                        (bfd_size_type) prefix_len)
+             || !bfd_set_section_contents (abfd,
+                                           *debug_string_section_p,
+                                           (void *) symbol->name,
+                                           (file_ptr) (*debug_string_size_p
+                                                       + prefix_len),
+                                           (bfd_size_type) name_length + 1))
            abort ();
          if (bfd_seek (abfd, filepos, SEEK_SET) != 0)
            abort ();
-         native->u.syment._n._n_n._n_offset = debug_string_size + 2;
+         native->u.syment._n._n_n._n_offset =
+             *debug_string_size_p + prefix_len;
          native->u.syment._n._n_n._n_zeroes = 0;
-         debug_string_size += name_length + 3;
+         *debug_string_size_p += name_length + 1 + prefix_len;
        }
     }
 }
@@ -663,58 +882,52 @@ coff_fix_symbol_name (abfd, symbol, native)
    the relocs we can get the index for a symbol.  This method is a
    hack.  FIXME.  */
 
-#define set_index(symbol, idx) ((symbol)->udata = (PTR) (idx))
+#define set_index(symbol, idx) ((symbol)->udata.i = (idx))
 
 /* Write a symbol out to a COFF file.  */
 
-static boolean
-coff_write_symbol (abfd, symbol, native, written)
-     bfd *abfd;
-     asymbol *symbol;
-     combined_entry_type *native;
-     unsigned int *written;
+static bfd_boolean
+coff_write_symbol (bfd *abfd,
+                  asymbol *symbol,
+                  combined_entry_type *native,
+                  bfd_vma *written,
+                  bfd_size_type *string_size_p,
+                  asection **debug_string_section_p,
+                  bfd_size_type *debug_string_size_p)
 {
   unsigned int numaux = native->u.syment.n_numaux;
   int type = native->u.syment.n_type;
-  int class =  native->u.syment.n_sclass;
-  PTR buf;
+  int class = native->u.syment.n_sclass;
+  void * buf;
   bfd_size_type symesz;
 
-  /* @@ bfd_debug_section isn't accessible outside this file, but we
-     know that C_FILE symbols belong there.  So move them.  */
   if (native->u.syment.n_sclass == C_FILE)
-    symbol->section = &bfd_debug_section;
+    symbol->flags |= BSF_DEBUGGING;
 
-  if (symbol->section == &bfd_abs_section) 
-    {
-      native->u.syment.n_scnum = N_ABS;
-    }
-  else if (symbol->section == &bfd_debug_section) 
-    {
-      native->u.syment.n_scnum = N_DEBUG;
-    }
-  else if (symbol->section == &bfd_und_section)   
-    {
-      native->u.syment.n_scnum = N_UNDEF;
-    }
-  else 
-    {
-      native->u.syment.n_scnum =
-       symbol->section->output_section->target_index;
-    }
-  
-  coff_fix_symbol_name (abfd, symbol, native);
+  if (symbol->flags & BSF_DEBUGGING
+      && bfd_is_abs_section (symbol->section))
+    native->u.syment.n_scnum = N_DEBUG;
+
+  else if (bfd_is_abs_section (symbol->section))
+    native->u.syment.n_scnum = N_ABS;
+
+  else if (bfd_is_und_section (symbol->section))
+    native->u.syment.n_scnum = N_UNDEF;
+
+  else
+    native->u.syment.n_scnum =
+      symbol->section->output_section->target_index;
+
+  coff_fix_symbol_name (abfd, symbol, native, string_size_p,
+                       debug_string_section_p, debug_string_size_p);
 
   symesz = bfd_coff_symesz (abfd);
   buf = bfd_alloc (abfd, symesz);
   if (!buf)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return false;
-    }
+    return FALSE;
   bfd_coff_swap_sym_out (abfd, &native->u.syment, buf);
-  if (bfd_write (buf, 1, symesz, abfd) != symesz)
-    return false;
+  if (bfd_bwrite (buf, symesz, abfd) != symesz)
+    return FALSE;
   bfd_release (abfd, buf);
 
   if (native->u.syment.n_numaux > 0)
@@ -725,21 +938,16 @@ coff_write_symbol (abfd, symbol, native, written)
       auxesz = bfd_coff_auxesz (abfd);
       buf = bfd_alloc (abfd, auxesz);
       if (!buf)
-       {
-         bfd_set_error (bfd_error_no_memory);
-         return false;
-       }
-      for (j = 0; j < native->u.syment.n_numaux;  j++)
+       return FALSE;
+      for (j = 0; j < native->u.syment.n_numaux; j++)
        {
          bfd_coff_swap_aux_out (abfd,
                                 &((native + j + 1)->u.auxent),
-                                type,
-                                class,
-                                j,
+                                type, class, (int) j,
                                 native->u.syment.n_numaux,
                                 buf);
-         if (bfd_write (buf, 1, auxesz, abfd)!= auxesz)
-           return false;
+         if (bfd_bwrite (buf, auxesz, abfd) != auxesz)
+           return FALSE;
        }
       bfd_release (abfd, buf);
     }
@@ -748,26 +956,28 @@ coff_write_symbol (abfd, symbol, native, written)
   set_index (symbol, *written);
 
   *written += numaux + 1;
-  return true;
+  return TRUE;
 }
 
 /* Write out a symbol to a COFF file that does not come from a COFF
    file originally.  This symbol may have been created by the linker,
    or we may be linking a non COFF file to a COFF file.  */
 
-static boolean
-coff_write_alien_symbol (abfd, symbol, written)
-     bfd *abfd;
-     asymbol *symbol;
-     unsigned int *written;
+static bfd_boolean
+coff_write_alien_symbol (bfd *abfd,
+                        asymbol *symbol,
+                        bfd_vma *written,
+                        bfd_size_type *string_size_p,
+                        asection **debug_string_section_p,
+                        bfd_size_type *debug_string_size_p)
 {
   combined_entry_type *native;
   combined_entry_type dummy;
 
   native = &dummy;
-  native->u.syment.n_type =  T_NULL;
-  native->u.syment.n_flags =  0;
-  if (symbol->section == &bfd_und_section) 
+  native->u.syment.n_type = T_NULL;
+  native->u.syment.n_flags = 0;
+  if (bfd_is_und_section (symbol->section))
     {
       native->u.syment.n_scnum = N_UNDEF;
       native->u.syment.n_value = symbol->value;
@@ -779,20 +989,24 @@ coff_write_alien_symbol (abfd, symbol, written)
     }
   else if (symbol->flags & BSF_DEBUGGING)
     {
-      /* Remove the symbol name so that it does not take up any space.
-        COFF won't know what to do with it anyhow.  */
+      /* There isn't much point to writing out a debugging symbol
+         unless we are prepared to convert it into COFF debugging
+         format.  So, we just ignore them.  We must clobber the symbol
+         name to keep it from being put in the string table.  */
       symbol->name = "";
+      return TRUE;
     }
   else
     {
       native->u.syment.n_scnum =
        symbol->section->output_section->target_index;
       native->u.syment.n_value = (symbol->value
-                                 + symbol->section->output_section->vma
                                  + symbol->section->output_offset);
+      if (! obj_pe (abfd))
+       native->u.syment.n_value += symbol->section->output_section->vma;
 
-      /* Copy the any flags from the the file header into the symbol.
-        FIXME: Why?  */
+      /* Copy the any flags from the file header into the symbol.
+         FIXME: Why?  */
       {
        coff_symbol_type *c = coff_symbol_from (abfd, symbol);
        if (c != (coff_symbol_type *) NULL)
@@ -803,20 +1017,25 @@ coff_write_alien_symbol (abfd, symbol, written)
   native->u.syment.n_type = 0;
   if (symbol->flags & BSF_LOCAL)
     native->u.syment.n_sclass = C_STAT;
+  else if (symbol->flags & BSF_WEAK)
+    native->u.syment.n_sclass = obj_pe (abfd) ? C_NT_WEAK : C_WEAKEXT;
   else
     native->u.syment.n_sclass = C_EXT;
   native->u.syment.n_numaux = 0;
 
-  return coff_write_symbol (abfd, symbol, native, written);
+  return coff_write_symbol (abfd, symbol, native, written, string_size_p,
+                           debug_string_section_p, debug_string_size_p);
 }
 
 /* Write a native symbol to a COFF file.  */
 
-static boolean
-coff_write_native_symbol (abfd, symbol, written)
-     bfd *abfd;
-     coff_symbol_type *symbol;
-     unsigned int *written;
+static bfd_boolean
+coff_write_native_symbol (bfd *abfd,
+                         coff_symbol_type *symbol,
+                         bfd_vma *written,
+                         bfd_size_type *string_size_p,
+                         asection **debug_string_section_p,
+                         bfd_size_type *debug_string_size_p)
 {
   combined_entry_type *native = symbol->native;
   alent *lineno = symbol->lineno;
@@ -824,13 +1043,14 @@ coff_write_native_symbol (abfd, symbol, written)
   /* If this symbol has an associated line number, we must store the
      symbol index in the line number field.  We also tag the auxent to
      point to the right place in the lineno table.  */
-  if (lineno && !symbol->done_lineno)
+  if (lineno && !symbol->done_lineno && symbol->symbol.section->owner != NULL)
     {
       unsigned int count = 0;
+
       lineno[count].u.offset = *written;
       if (native->u.syment.n_numaux)
        {
-         union internal_auxent  *a = &((native+1)->u.auxent);
+         union internal_auxent *a = &((native + 1)->u.auxent);
 
          a->x_sym.x_fcnary.x_fcn.x_lnnoptr =
            symbol->symbol.section->output_section->moving_line_filepos;
@@ -840,59 +1060,63 @@ coff_write_native_symbol (abfd, symbol, written)
       count++;
       while (lineno[count].line_number != 0)
        {
-#if 0
-         /* 13 april 92. sac 
-            I've been told this, but still need proof:
-            > The second bug is also in `bfd/coffcode.h'.  This bug
-            > causes the linker to screw up the pc-relocations for
-            > all the line numbers in COFF code.  This bug isn't only
-            > specific to A29K implementations, but affects all
-            > systems using COFF format binaries.  Note that in COFF
-            > object files, the line number core offsets output by
-            > the assembler are relative to the start of each
-            > procedure, not to the start of the .text section.  This
-            > patch relocates the line numbers relative to the
-            > `native->u.syment.n_value' instead of the section
-            > virtual address.
-            > modular!olson@cs.arizona.edu (Jon Olson)
-            */
-         lineno[count].u.offset += native->u.syment.n_value;
-#else
          lineno[count].u.offset +=
            (symbol->symbol.section->output_section->vma
             + symbol->symbol.section->output_offset);
-#endif
          count++;
        }
-      symbol->done_lineno = true;
-    
-      symbol->symbol.section->output_section->moving_line_filepos +=
-       count * bfd_coff_linesz (abfd);
+      symbol->done_lineno = TRUE;
+
+      if (! bfd_is_const_section (symbol->symbol.section->output_section))
+       symbol->symbol.section->output_section->moving_line_filepos +=
+         count * bfd_coff_linesz (abfd);
     }
 
-  return coff_write_symbol (abfd, &(symbol->symbol), native, written);
+  return coff_write_symbol (abfd, &(symbol->symbol), native, written,
+                           string_size_p, debug_string_section_p,
+                           debug_string_size_p);
 }
 
 /* Write out the COFF symbols.  */
 
-boolean
-coff_write_symbols (abfd)
-     bfd *abfd;
+bfd_boolean
+coff_write_symbols (bfd *abfd)
 {
+  bfd_size_type string_size;
+  asection *debug_string_section;
+  bfd_size_type debug_string_size;
   unsigned int i;
-  unsigned int limit = bfd_get_symcount(abfd);
-  unsigned int written = 0;
+  unsigned int limit = bfd_get_symcount (abfd);
+  bfd_vma written = 0;
   asymbol **p;
 
   string_size = 0;
+  debug_string_section = NULL;
   debug_string_size = 0;
 
-  /* Seek to the right place */
-  if (bfd_seek (abfd, obj_sym_filepos(abfd), SEEK_SET) != 0)
-    return false;
+  /* If this target supports long section names, they must be put into
+     the string table.  This is supported by PE.  This code must
+     handle section names just as they are handled in
+     coff_write_object_contents.  */
+  if (bfd_coff_long_section_names (abfd))
+    {
+      asection *o;
+
+      for (o = abfd->sections; o != NULL; o = o->next)
+       {
+         size_t len;
+
+         len = strlen (o->name);
+         if (len > SCNNMLEN)
+           string_size += len + 1;
+       }
+    }
 
-  /* Output all the symbols we have */
+  /* Seek to the right place.  */
+  if (bfd_seek (abfd, obj_sym_filepos (abfd), SEEK_SET) != 0)
+    return FALSE;
 
+  /* Output all the symbols we have.  */
   written = 0;
   for (p = abfd->outsymbols, i = 0; i < limit; i++, p++)
     {
@@ -900,587 +1124,748 @@ coff_write_symbols (abfd)
       coff_symbol_type *c_symbol = coff_symbol_from (abfd, symbol);
 
       if (c_symbol == (coff_symbol_type *) NULL
-         || c_symbol->native == (combined_entry_type *)NULL)
+         || c_symbol->native == (combined_entry_type *) NULL)
        {
-         if (! coff_write_alien_symbol (abfd, symbol, &written))
-           return false;
+         if (!coff_write_alien_symbol (abfd, symbol, &written, &string_size,
+                                       &debug_string_section,
+                                       &debug_string_size))
+           return FALSE;
        }
       else
        {
-         if (! coff_write_native_symbol (abfd, c_symbol, &written))
-           return false;
+         if (!coff_write_native_symbol (abfd, c_symbol, &written,
+                                        &string_size, &debug_string_section,
+                                        &debug_string_size))
+           return FALSE;
        }
     }
 
-  bfd_get_symcount (abfd) = written;
-
-  /* Now write out strings */
+  obj_raw_syment_count (abfd) = written;
 
+  /* Now write out strings.  */
   if (string_size != 0)
-   {
-     unsigned int size = string_size + 4;
-     bfd_byte buffer[4];
-
-     bfd_h_put_32 (abfd, size, buffer);
-     if (bfd_write ((PTR) buffer, 1, sizeof (buffer), abfd) != sizeof (buffer))
-       return false;
-     for (p = abfd->outsymbols, i = 0;
-         i < limit;
-         i++, p++)
-       {
-        asymbol *q = *p;
-        size_t name_length = strlen (q->name);
-        coff_symbol_type *c_symbol = coff_symbol_from (abfd, q);
-        size_t maxlen;
-
-        /* Figure out whether the symbol name should go in the string
-           table.  Symbol names that are short enough are stored
-           directly in the syment structure.  File names permit a
-           different, longer, length in the syment structure.  On
-           XCOFF, some symbol names are stored in the .debug section
-           rather than in the string table.  */
-
-        if (c_symbol == NULL
-            || c_symbol->native == NULL)
-          {
-            /* This is not a COFF symbol, so it certainly is not a
-               file name, nor does it go in the .debug section.  */
-            maxlen = SYMNMLEN;
-          }
-        else if (bfd_coff_symname_in_debug (abfd,
-                                            &c_symbol->native->u.syment))
-          {
-            /* This symbol name is in the XCOFF .debug section.
-               Don't write it into the string table.  */
-            maxlen = name_length;
-          }
-        else if (c_symbol->native->u.syment.n_sclass == C_FILE)
-          maxlen = FILNMLEN;
-        else
-          maxlen = SYMNMLEN;
-
-        if (name_length > maxlen)
-          {
-            if (bfd_write ((PTR) (q->name), 1, name_length + 1, abfd)
-                != name_length + 1)
-              return false;
-          }
-       }
-   }
-  else
     {
-      /* We would normally not write anything here, but we'll write
-        out 4 so that any stupid coff reader which tries to read the
-        string table even when there isn't one won't croak.  */
-      unsigned int size = 4;
-      bfd_byte buffer[4];
-
-      bfd_h_put_32 (abfd, size, buffer);
-      if (bfd_write ((PTR) buffer, 1, 4, abfd) != 4)
-       return false;
-    }
+      unsigned int size = string_size + STRING_SIZE_SIZE;
+      bfd_byte buffer[STRING_SIZE_SIZE];
 
-  /* Make sure the .debug section was created to be the correct size.
-     We should create it ourselves on the fly, but we don't because
-     BFD won't let us write to any section until we know how large all
-     the sections are.  We could still do it by making another pass
-     over the symbols.  FIXME.  */
-  BFD_ASSERT (debug_string_size == 0
-             || (debug_string_section != (asection *) NULL
-                 && (BFD_ALIGN (debug_string_size,
-                                1 << debug_string_section->alignment_power)
-                     == bfd_section_size (abfd, debug_string_section))));
+#if STRING_SIZE_SIZE == 4
+      H_PUT_32 (abfd, size, buffer);
+#else
+ #error Change H_PUT_32
+#endif
+      if (bfd_bwrite ((void *) buffer, (bfd_size_type) sizeof (buffer), abfd)
+         != sizeof (buffer))
+       return FALSE;
 
-  return true;
-}
+      /* Handle long section names.  This code must handle section
+        names just as they are handled in coff_write_object_contents.  */
+      if (bfd_coff_long_section_names (abfd))
+       {
+         asection *o;
 
-boolean
-coff_write_linenumbers (abfd)
-     bfd            *abfd;
-{
-  asection       *s;
-  bfd_size_type linesz;
-  PTR buff;
+         for (o = abfd->sections; o != NULL; o = o->next)
+           {
+             size_t len;
 
-  linesz = bfd_coff_linesz (abfd);
-  buff = bfd_alloc (abfd, linesz);
-  if (!buff)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return false;
-    }
-  for (s = abfd->sections; s != (asection *) NULL; s = s->next) {
-    if (s->lineno_count) {
-      asymbol       **q = abfd->outsymbols;
-      if (bfd_seek(abfd, s->line_filepos, SEEK_SET) != 0)
-       return false;
-      /* Find all the linenumbers in this section */
-      while (*q) {
-       asymbol        *p = *q;
-       if (p->section->output_section == s) {
-         alent          *l =
-          BFD_SEND(bfd_asymbol_bfd(p), _get_lineno, (bfd_asymbol_bfd(p), p));
-         if (l) {
-           /* Found a linenumber entry, output */
-           struct internal_lineno  out;
-           memset( (PTR)&out, 0, sizeof(out));
-           out.l_lnno = 0;
-           out.l_addr.l_symndx = l->u.offset;
-           bfd_coff_swap_lineno_out(abfd, &out, buff);
-           if (bfd_write(buff, 1, linesz, abfd) != linesz)
-             return false;
-           l++;
-           while (l->line_number) {
-             out.l_lnno = l->line_number;
-             out.l_addr.l_symndx = l->u.offset;
-             bfd_coff_swap_lineno_out(abfd, &out, buff);
-             if (bfd_write(buff, 1, linesz, abfd) != linesz)
-               return false;
-             l++;
+             len = strlen (o->name);
+             if (len > SCNNMLEN)
+               {
+                 if (bfd_bwrite (o->name, (bfd_size_type) (len + 1), abfd)
+                     != len + 1)
+                   return FALSE;
+               }
+           }
+       }
+
+      for (p = abfd->outsymbols, i = 0;
+          i < limit;
+          i++, p++)
+       {
+         asymbol *q = *p;
+         size_t name_length = strlen (q->name);
+         coff_symbol_type *c_symbol = coff_symbol_from (abfd, q);
+         size_t maxlen;
+
+         /* Figure out whether the symbol name should go in the string
+            table.  Symbol names that are short enough are stored
+            directly in the syment structure.  File names permit a
+            different, longer, length in the syment structure.  On
+            XCOFF, some symbol names are stored in the .debug section
+            rather than in the string table.  */
+
+         if (c_symbol == NULL
+             || c_symbol->native == NULL)
+           /* This is not a COFF symbol, so it certainly is not a
+              file name, nor does it go in the .debug section.  */
+           maxlen = bfd_coff_force_symnames_in_strings (abfd) ? 0 : SYMNMLEN;
+
+         else if (bfd_coff_symname_in_debug (abfd,
+                                             &c_symbol->native->u.syment))
+           /* This symbol name is in the XCOFF .debug section.
+              Don't write it into the string table.  */
+           maxlen = name_length;
+
+         else if (c_symbol->native->u.syment.n_sclass == C_FILE
+                  && c_symbol->native->u.syment.n_numaux > 0)
+           {
+             if (bfd_coff_force_symnames_in_strings (abfd))
+               {
+                 if (bfd_bwrite (".file", (bfd_size_type) 6, abfd) != 6)
+                   return FALSE;
+               }
+             maxlen = bfd_coff_filnmlen (abfd);
+           }
+         else
+           maxlen = bfd_coff_force_symnames_in_strings (abfd) ? 0 : SYMNMLEN;
+
+         if (name_length > maxlen)
+           {
+             if (bfd_bwrite ((void *) (q->name), (bfd_size_type) name_length + 1,
+                            abfd) != name_length + 1)
+               return FALSE;
            }
-         }
        }
-       q++;
-      }
     }
-  }
-  bfd_release (abfd, buff);
-  return true;
-}
+  else
+    {
+      /* We would normally not write anything here, but we'll write
+         out 4 so that any stupid coff reader which tries to read the
+         string table even when there isn't one won't croak.  */
+      unsigned int size = STRING_SIZE_SIZE;
+      bfd_byte buffer[STRING_SIZE_SIZE];
 
-/*ARGSUSED*/
-alent   *
-coff_get_lineno (ignore_abfd, symbol)
-     bfd            *ignore_abfd;
-     asymbol        *symbol;
-{
-  return coffsymbol(symbol)->lineno;
+#if STRING_SIZE_SIZE == 4
+      H_PUT_32 (abfd, size, buffer);
+#else
+ #error Change H_PUT_32
+#endif
+      if (bfd_bwrite ((void *) buffer, (bfd_size_type) STRING_SIZE_SIZE, abfd)
+         != STRING_SIZE_SIZE)
+       return FALSE;
+    }
+
+  /* Make sure the .debug section was created to be the correct size.
+     We should create it ourselves on the fly, but we don't because
+     BFD won't let us write to any section until we know how large all
+     the sections are.  We could still do it by making another pass
+     over the symbols.  FIXME.  */
+  BFD_ASSERT (debug_string_size == 0
+             || (debug_string_section != (asection *) NULL
+                 && (BFD_ALIGN (debug_string_size,
+                                1 << debug_string_section->alignment_power)
+                     == debug_string_section->size)));
+
+  return TRUE;
 }
 
-asymbol *
-coff_section_symbol (abfd, name)
-     bfd *abfd;
-     char *name;
+bfd_boolean
+coff_write_linenumbers (bfd *abfd)
 {
-  asection *sec = bfd_make_section_old_way (abfd, name);
-  asymbol *sym;
-  combined_entry_type *csym;
-
-  sym = sec->symbol;
-  csym = coff_symbol_from (abfd, sym)->native;
-  /* Make sure back-end COFF stuff is there.  */
-  if (csym == 0)
+  asection *s;
+  bfd_size_type linesz;
+  void * buff;
+
+  linesz = bfd_coff_linesz (abfd);
+  buff = bfd_alloc (abfd, linesz);
+  if (!buff)
+    return FALSE;
+  for (s = abfd->sections; s != (asection *) NULL; s = s->next)
     {
-      struct foo {
-       coff_symbol_type sym;
-       /* @@FIXME This shouldn't use a fixed size!!  */
-       combined_entry_type e[10];
-      };
-      struct foo *f;
-      f = (struct foo *) bfd_alloc_by_size_t (abfd, sizeof (*f));
-      if (!f)
+      if (s->lineno_count)
        {
-         bfd_set_error (bfd_error_no_error);
-         return NULL;
+         asymbol **q = abfd->outsymbols;
+         if (bfd_seek (abfd, s->line_filepos, SEEK_SET) != 0)
+           return FALSE;
+         /* Find all the linenumbers in this section.  */
+         while (*q)
+           {
+             asymbol *p = *q;
+             if (p->section->output_section == s)
+               {
+                 alent *l =
+                 BFD_SEND (bfd_asymbol_bfd (p), _get_lineno,
+                           (bfd_asymbol_bfd (p), p));
+                 if (l)
+                   {
+                     /* Found a linenumber entry, output.  */
+                     struct internal_lineno out;
+                     memset ((void *) & out, 0, sizeof (out));
+                     out.l_lnno = 0;
+                     out.l_addr.l_symndx = l->u.offset;
+                     bfd_coff_swap_lineno_out (abfd, &out, buff);
+                     if (bfd_bwrite (buff, (bfd_size_type) linesz, abfd)
+                         != linesz)
+                       return FALSE;
+                     l++;
+                     while (l->line_number)
+                       {
+                         out.l_lnno = l->line_number;
+                         out.l_addr.l_symndx = l->u.offset;
+                         bfd_coff_swap_lineno_out (abfd, &out, buff);
+                         if (bfd_bwrite (buff, (bfd_size_type) linesz, abfd)
+                             != linesz)
+                           return FALSE;
+                         l++;
+                       }
+                   }
+               }
+             q++;
+           }
        }
-      memset ((char *) f, 0, sizeof (*f));
-      coff_symbol_from (abfd, sym)->native = csym = f->e;
-    }
-  csym[0].u.syment.n_sclass = C_STAT;
-  csym[0].u.syment.n_numaux = 1;
-/*  SF_SET_STATICS (sym);      @@ ??? */
-  csym[1].u.auxent.x_scn.x_scnlen = sec->_raw_size;
-  csym[1].u.auxent.x_scn.x_nreloc = sec->reloc_count;
-  csym[1].u.auxent.x_scn.x_nlinno = sec->lineno_count;
-
-  if (sec->output_section == NULL)
-    {
-      sec->output_section = sec;
-      sec->output_offset = 0;
     }
+  bfd_release (abfd, buff);
+  return TRUE;
+}
 
-  return sym;
+alent *
+coff_get_lineno (bfd *ignore_abfd ATTRIBUTE_UNUSED, asymbol *symbol)
+{
+  return coffsymbol (symbol)->lineno;
 }
 
 /* This function transforms the offsets into the symbol table into
    pointers to syments.  */
 
 static void
-coff_pointerize_aux (abfd, table_base, type, class, auxent)
-     bfd *abfd;
-     combined_entry_type *table_base;
-     int type;
-     int class;
-     combined_entry_type *auxent;
+coff_pointerize_aux (bfd *abfd,
+                    combined_entry_type *table_base,
+                    combined_entry_type *symbol,
+                    unsigned int indaux,
+                    combined_entry_type *auxent)
 {
-  /* Don't bother if this is a file or a section */
-  if (class == C_STAT && type == T_NULL) return;
-  if (class == C_FILE) return;
+  unsigned int type = symbol->u.syment.n_type;
+  unsigned int class = symbol->u.syment.n_sclass;
+
+  if (coff_backend_info (abfd)->_bfd_coff_pointerize_aux_hook)
+    {
+      if ((*coff_backend_info (abfd)->_bfd_coff_pointerize_aux_hook)
+         (abfd, table_base, symbol, indaux, auxent))
+       return;
+    }
 
-  /* Otherwise patch up */
-#define N_TMASK coff_data (abfd)->local_n_tmask
+  /* Don't bother if this is a file or a section.  */
+  if (class == C_STAT && type == T_NULL)
+    return;
+  if (class == C_FILE)
+    return;
+
+  /* Otherwise patch up.  */
+#define N_TMASK coff_data  (abfd)->local_n_tmask
 #define N_BTSHFT coff_data (abfd)->local_n_btshft
-  if (ISFCN(type) || ISTAG(class) || class == C_BLOCK) {
-      auxent->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.p = table_base +
-       auxent->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.l;
+  
+  if ((ISFCN (type) || ISTAG (class) || class == C_BLOCK || class == C_FCN)
+      && auxent->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.l > 0)
+    {
+      auxent->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.p =
+       table_base + auxent->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.l;
       auxent->fix_end = 1;
     }
   /* A negative tagndx is meaningless, but the SCO 3.2v4 cc can
      generate one, so we must be careful to ignore it.  */
-  if (auxent->u.auxent.x_sym.x_tagndx.l > 0) {
+  if (auxent->u.auxent.x_sym.x_tagndx.l > 0)
+    {
       auxent->u.auxent.x_sym.x_tagndx.p =
-       table_base +  auxent->u.auxent.x_sym.x_tagndx.l;
+       table_base + auxent->u.auxent.x_sym.x_tagndx.l;
       auxent->fix_tag = 1;
     }
 }
 
-static char *
-build_string_table (abfd)
-     bfd *abfd;
-{
-  char string_table_size_buffer[4];
-  unsigned int string_table_size;
-  char *string_table;
-
-  /* At this point we should be "seek"'d to the end of the
-     symbols === the symbol table size.  */
-  if (bfd_read((char *) string_table_size_buffer,
-              sizeof(string_table_size_buffer),
-              1, abfd) != sizeof(string_table_size))
-    return (NULL);
-
-  string_table_size = bfd_h_get_32(abfd, (bfd_byte *) string_table_size_buffer);
-
-  if ((string_table = (PTR) bfd_alloc(abfd, string_table_size -= 4)) == NULL) {
-    bfd_set_error (bfd_error_no_memory);
-    return (NULL);
-  }                            /* on mallocation error */
-  if (bfd_read(string_table, string_table_size, 1, abfd) != string_table_size)
-    return (NULL);
-  return string_table;
-}
-
 /* Allocate space for the ".debug" section, and read it.
    We did not read the debug section until now, because
-   we didn't want to go to the trouble until someone needed it. */
+   we didn't want to go to the trouble until someone needed it.  */
 
 static char *
-build_debug_section (abfd)
-     bfd *abfd;
+build_debug_section (bfd *abfd)
 {
   char *debug_section;
-  long position;
+  file_ptr position;
+  bfd_size_type sec_size;
 
   asection *sect = bfd_get_section_by_name (abfd, ".debug");
 
-  if (!sect) {
-     bfd_set_error (bfd_error_no_debug_section);
-     return NULL;
-  }
+  if (!sect)
+    {
+      bfd_set_error (bfd_error_no_debug_section);
+      return NULL;
+    }
 
-  debug_section = (PTR) bfd_alloc (abfd,
-                                  bfd_get_section_size_before_reloc (sect));
-  if (debug_section == NULL) {
-    bfd_set_error (bfd_error_no_memory);
+  sec_size = sect->size;
+  debug_section = bfd_alloc (abfd, sec_size);
+  if (debug_section == NULL)
     return NULL;
-  }
 
-  /* Seek to the beginning of the `.debug' section and read it. 
+  /* Seek to the beginning of the `.debug' section and read it.
      Save the current position first; it is needed by our caller.
      Then read debug section and reset the file pointer.  */
 
   position = bfd_tell (abfd);
   if (bfd_seek (abfd, sect->filepos, SEEK_SET) != 0
-      || (bfd_read (debug_section, 
-                   bfd_get_section_size_before_reloc (sect), 1, abfd)
-         != bfd_get_section_size_before_reloc(sect))
+      || bfd_bread (debug_section, sec_size, abfd) != sec_size
       || bfd_seek (abfd, position, SEEK_SET) != 0)
     return NULL;
   return debug_section;
 }
 
-
 /* Return a pointer to a malloc'd copy of 'name'.  'name' may not be
- \0-terminated, but will not exceed 'maxlen' characters.  The copy *will*
- be \0-terminated.  */
+   \0-terminated, but will not exceed 'maxlen' characters.  The copy *will*
+   be \0-terminated.  */
+
 static char *
-copy_name (abfd, name, maxlen)
-     bfd *abfd;
-     char *name;
-     int maxlen;
+copy_name (bfd *abfd, char *name, size_t maxlen)
 {
-  int  len;
+  size_t len;
   char *newname;
 
-  for (len = 0; len < maxlen; ++len) {
-    if (name[len] == '\0') {
+  for (len = 0; len < maxlen; ++len)
+    if (name[len] == '\0')
       break;
-    }
-  }
 
-  if ((newname = (PTR) bfd_alloc(abfd, len+1)) == NULL) {
-    bfd_set_error (bfd_error_no_memory);
-    return (NULL);
-  }
-  strncpy(newname, name, len);
+  if ((newname = bfd_alloc (abfd, (bfd_size_type) len + 1)) == NULL)
+    return NULL;
+
+  strncpy (newname, name, len);
   newname[len] = '\0';
   return newname;
 }
 
-/* Read a symbol table into freshly bfd_allocated memory, swap it, and
-   knit the symbol names into a normalized form.  By normalized here I
-   mean that all symbols have an n_offset pointer that points to a null-
-   terminated string.  */
+/* Read in the external symbols.  */
 
-combined_entry_type *
-coff_get_normalized_symtab (abfd)
-     bfd            *abfd;
+bfd_boolean
+_bfd_coff_get_external_symbols (bfd *abfd)
 {
-  combined_entry_type          *internal;
-  combined_entry_type          *internal_ptr;
-  combined_entry_type          *symbol_ptr;
-  combined_entry_type         *internal_end;
   bfd_size_type symesz;
-  PTR raw;
-  char *raw_src;
-  char *raw_end;
-  char           *string_table = NULL;
-  char          *debug_section = NULL;
-  unsigned long   size;
+  bfd_size_type size;
+  void * syms;
+
+  if (obj_coff_external_syms (abfd) != NULL)
+    return TRUE;
+
+  symesz = bfd_coff_symesz (abfd);
+
+  size = obj_raw_syment_count (abfd) * symesz;
+
+  syms = bfd_malloc (size);
+  if (syms == NULL && size != 0)
+    return FALSE;
 
-  unsigned int raw_size;
-  if (obj_raw_syments(abfd) != (combined_entry_type *)NULL) {
-      return obj_raw_syments(abfd);
+  if (bfd_seek (abfd, obj_sym_filepos (abfd), SEEK_SET) != 0
+      || bfd_bread (syms, size, abfd) != size)
+    {
+      if (syms != NULL)
+       free (syms);
+      return FALSE;
     }
-  if ((size = bfd_get_symcount(abfd) * sizeof(combined_entry_type)) == 0) {
+
+  obj_coff_external_syms (abfd) = syms;
+
+  return TRUE;
+}
+
+/* Read in the external strings.  The strings are not loaded until
+   they are needed.  This is because we have no simple way of
+   detecting a missing string table in an archive.  */
+
+const char *
+_bfd_coff_read_string_table (bfd *abfd)
+{
+  char extstrsize[STRING_SIZE_SIZE];
+  bfd_size_type strsize;
+  char *strings;
+  file_ptr pos;
+
+  if (obj_coff_strings (abfd) != NULL)
+    return obj_coff_strings (abfd);
+
+  if (obj_sym_filepos (abfd) == 0)
+    {
       bfd_set_error (bfd_error_no_symbols);
-      return (NULL);
+      return NULL;
+    }
+
+  pos = obj_sym_filepos (abfd);
+  pos += obj_raw_syment_count (abfd) * bfd_coff_symesz (abfd);
+  if (bfd_seek (abfd, pos, SEEK_SET) != 0)
+    return NULL;
+
+  if (bfd_bread (extstrsize, (bfd_size_type) sizeof extstrsize, abfd)
+      != sizeof extstrsize)
+    {
+      if (bfd_get_error () != bfd_error_file_truncated)
+       return NULL;
+
+      /* There is no string table.  */
+      strsize = STRING_SIZE_SIZE;
+    }
+  else
+    {
+#if STRING_SIZE_SIZE == 4
+      strsize = H_GET_32 (abfd, extstrsize);
+#else
+ #error Change H_GET_32
+#endif
     }
 
-  internal = (combined_entry_type *)bfd_alloc(abfd, size);
-  if (!internal)
+  if (strsize < STRING_SIZE_SIZE)
     {
-      bfd_set_error (bfd_error_no_memory);
+      (*_bfd_error_handler)
+       (_("%B: bad string table size %lu"), abfd, (unsigned long) strsize);
+      bfd_set_error (bfd_error_bad_value);
       return NULL;
     }
-  internal_end = internal + bfd_get_symcount(abfd);
 
-  symesz = bfd_coff_symesz (abfd);
-  raw_size =      bfd_get_symcount(abfd) * symesz;
-  raw = bfd_alloc(abfd,raw_size);
-  if (!raw)
+  strings = bfd_malloc (strsize);
+  if (strings == NULL)
+    return NULL;
+
+  if (bfd_bread (strings + STRING_SIZE_SIZE, strsize - STRING_SIZE_SIZE, abfd)
+      != strsize - STRING_SIZE_SIZE)
     {
-      bfd_set_error (bfd_error_no_memory);
+      free (strings);
       return NULL;
     }
 
-  if (bfd_seek(abfd, obj_sym_filepos(abfd), SEEK_SET) == -1
-      || bfd_read(raw, raw_size, 1, abfd) != raw_size)
-      return (NULL);
-  /* mark the end of the symbols */
-  raw_end = (char *) raw + bfd_get_symcount(abfd) * symesz;
-  /*
-    FIXME SOMEDAY.  A string table size of zero is very weird, but
-    probably possible.  If one shows up, it will probably kill us.
-    */
-
-  /* Swap all the raw entries */
-  for (raw_src = (char *) raw, internal_ptr = internal;
+  obj_coff_strings (abfd) = strings;
+
+  return strings;
+}
+
+/* Free up the external symbols and strings read from a COFF file.  */
+
+bfd_boolean
+_bfd_coff_free_symbols (bfd *abfd)
+{
+  if (obj_coff_external_syms (abfd) != NULL
+      && ! obj_coff_keep_syms (abfd))
+    {
+      free (obj_coff_external_syms (abfd));
+      obj_coff_external_syms (abfd) = NULL;
+    }
+  if (obj_coff_strings (abfd) != NULL
+      && ! obj_coff_keep_strings (abfd))
+    {
+      free (obj_coff_strings (abfd));
+      obj_coff_strings (abfd) = NULL;
+    }
+  return TRUE;
+}
+
+/* Read a symbol table into freshly bfd_allocated memory, swap it, and
+   knit the symbol names into a normalized form.  By normalized here I
+   mean that all symbols have an n_offset pointer that points to a null-
+   terminated string.  */
+
+combined_entry_type *
+coff_get_normalized_symtab (bfd *abfd)
+{
+  combined_entry_type *internal;
+  combined_entry_type *internal_ptr;
+  combined_entry_type *symbol_ptr;
+  combined_entry_type *internal_end;
+  size_t symesz;
+  char *raw_src;
+  char *raw_end;
+  const char *string_table = NULL;
+  char *debug_section = NULL;
+  bfd_size_type size;
+
+  if (obj_raw_syments (abfd) != NULL)
+    return obj_raw_syments (abfd);
+
+  size = obj_raw_syment_count (abfd) * sizeof (combined_entry_type);
+  internal = bfd_zalloc (abfd, size);
+  if (internal == NULL && size != 0)
+    return NULL;
+  internal_end = internal + obj_raw_syment_count (abfd);
+
+  if (! _bfd_coff_get_external_symbols (abfd))
+    return NULL;
+
+  raw_src = (char *) obj_coff_external_syms (abfd);
+
+  /* Mark the end of the symbols.  */
+  symesz = bfd_coff_symesz (abfd);
+  raw_end = (char *) raw_src + obj_raw_syment_count (abfd) * symesz;
+
+  /* FIXME SOMEDAY.  A string table size of zero is very weird, but
+     probably possible.  If one shows up, it will probably kill us.  */
+
+  /* Swap all the raw entries.  */
+  for (internal_ptr = internal;
        raw_src < raw_end;
-       raw_src += symesz, internal_ptr++) {
+       raw_src += symesz, internal_ptr++)
+    {
 
       unsigned int i;
-      bfd_coff_swap_sym_in(abfd, (PTR)raw_src, (PTR)&internal_ptr->u.syment);
-      internal_ptr->fix_value = 0;
-      internal_ptr->fix_tag = 0;
-      internal_ptr->fix_end = 0;
-      internal_ptr->fix_scnlen = 0;
+      bfd_coff_swap_sym_in (abfd, (void *) raw_src,
+                           (void *) & internal_ptr->u.syment);
       symbol_ptr = internal_ptr;
 
       for (i = 0;
           i < symbol_ptr->u.syment.n_numaux;
-          i++) 
-      {
-       internal_ptr++;
-       raw_src += symesz;
-      
-       internal_ptr->fix_value = 0;
-       internal_ptr->fix_tag = 0;
-       internal_ptr->fix_end = 0;
-       internal_ptr->fix_scnlen = 0;
-       bfd_coff_swap_aux_in(abfd, (PTR) raw_src,
-                            symbol_ptr->u.syment.n_type,
-                            symbol_ptr->u.syment.n_sclass,
-                            i, symbol_ptr->u.syment.n_numaux,
-                            &(internal_ptr->u.auxent));
-       /* Remember that bal entries arn't pointerized */
-       if (i != 1 || symbol_ptr->u.syment.n_sclass != C_LEAFPROC)
+          i++)
        {
-         
-       coff_pointerize_aux(abfd,
-                           internal,
-                           symbol_ptr->u.syment.n_type,
-                           symbol_ptr->u.syment.n_sclass,
-                           internal_ptr);
-      }
-       
-      }
+         internal_ptr++;
+         raw_src += symesz;
+         bfd_coff_swap_aux_in (abfd, (void *) raw_src,
+                               symbol_ptr->u.syment.n_type,
+                               symbol_ptr->u.syment.n_sclass,
+                               (int) i, symbol_ptr->u.syment.n_numaux,
+                               &(internal_ptr->u.auxent));
+         coff_pointerize_aux (abfd, internal, symbol_ptr, i,
+                              internal_ptr);
+       }
     }
 
-  /* Free all the raw stuff */
-  bfd_release(abfd, raw);
+  /* Free the raw symbols, but not the strings (if we have them).  */
+  obj_coff_keep_strings (abfd) = TRUE;
+  if (! _bfd_coff_free_symbols (abfd))
+    return NULL;
 
   for (internal_ptr = internal; internal_ptr < internal_end;
-       internal_ptr ++)
-  {
-    if (internal_ptr->u.syment.n_sclass == C_FILE) {
-       /* make a file symbol point to the name in the auxent, since
-          the text ".file" is redundant */
-       if ((internal_ptr+1)->u.auxent.x_file.x_n.x_zeroes == 0) {
-           /* the filename is a long one, point into the string table */
-           if (string_table == NULL) {
-               string_table = build_string_table(abfd);
-             }
-
-           internal_ptr->u.syment._n._n_n._n_offset =
-            (long) (string_table - 4 +
-                   (internal_ptr+1)->u.auxent.x_file.x_n.x_offset);
-         }
-       else {
-           /* ordinary short filename, put into memory anyway */
-           internal_ptr->u.syment._n._n_n._n_offset = (long)
-            copy_name(abfd, (internal_ptr+1)->u.auxent.x_file.x_fname,
-                      FILNMLEN);
-         }
-      }
-    else {
-       if (internal_ptr->u.syment._n._n_n._n_zeroes != 0) {
-           /* This is a "short" name.  Make it long.  */
-           unsigned long   i = 0;
-           char           *newstring = NULL;
-
-           /* find the length of this string without walking into memory
-              that isn't ours.  */
-           for (i = 0; i < 8; ++i) {
-               if (internal_ptr->u.syment._n._n_name[i] == '\0') {
-                   break;
-                 }             /* if end of string */
-             }                 /* possible lengths of this string. */
-
-           if ((newstring = (PTR) bfd_alloc(abfd, ++i)) == NULL) {
-               bfd_set_error (bfd_error_no_memory);
-               return (NULL);
-             }                 /* on error */
-           memset(newstring, 0, i);
-           strncpy(newstring, internal_ptr->u.syment._n._n_name, i-1);
-           internal_ptr->u.syment._n._n_n._n_offset =  (long int) newstring;
-           internal_ptr->u.syment._n._n_n._n_zeroes = 0;
-         }
-       else if (internal_ptr->u.syment._n._n_n._n_offset == 0)
-         internal_ptr->u.syment._n._n_n._n_offset = (long int) "";
-       else if (!bfd_coff_symname_in_debug(abfd, &internal_ptr->u.syment)) {
-           /* Long name already.  Point symbol at the string in the table.  */
-           if (string_table == NULL) {
-               string_table = build_string_table(abfd);
-             }
-           internal_ptr->u.syment._n._n_n._n_offset = (long int)
-            (string_table - 4 + internal_ptr->u.syment._n._n_n._n_offset);
-         }
-       else {
-           /* Long name in debug section.  Very similar.  */
-           if (debug_section == NULL) {
-               debug_section = build_debug_section(abfd);
-             }
-           internal_ptr->u.syment._n._n_n._n_offset = (long int)
-            (debug_section + internal_ptr->u.syment._n._n_n._n_offset);
-         }
-      }
-    internal_ptr += internal_ptr->u.syment.n_numaux;
-  }
+       internal_ptr++)
+    {
+      if (internal_ptr->u.syment.n_sclass == C_FILE
+         && internal_ptr->u.syment.n_numaux > 0)
+       {
+         /* Make a file symbol point to the name in the auxent, since
+            the text ".file" is redundant.  */
+         if ((internal_ptr + 1)->u.auxent.x_file.x_n.x_zeroes == 0)
+           {
+             /* The filename is a long one, point into the string table.  */
+             if (string_table == NULL)
+               {
+                 string_table = _bfd_coff_read_string_table (abfd);
+                 if (string_table == NULL)
+                   return NULL;
+               }
+
+             internal_ptr->u.syment._n._n_n._n_offset =
+               ((long)
+                (string_table
+                 + (internal_ptr + 1)->u.auxent.x_file.x_n.x_offset));
+           }
+         else
+           {
+             /* Ordinary short filename, put into memory anyway.  The
+                 Microsoft PE tools sometimes store a filename in
+                 multiple AUX entries.  */
+             if (internal_ptr->u.syment.n_numaux > 1
+                 && coff_data (abfd)->pe)
+               internal_ptr->u.syment._n._n_n._n_offset =
+                 ((long)
+                  copy_name (abfd,
+                             (internal_ptr + 1)->u.auxent.x_file.x_fname,
+                             internal_ptr->u.syment.n_numaux * symesz));
+             else
+               internal_ptr->u.syment._n._n_n._n_offset =
+                 ((long)
+                  copy_name (abfd,
+                             (internal_ptr + 1)->u.auxent.x_file.x_fname,
+                             (size_t) bfd_coff_filnmlen (abfd)));
+           }
+       }
+      else
+       {
+         if (internal_ptr->u.syment._n._n_n._n_zeroes != 0)
+           {
+             /* This is a "short" name.  Make it long.  */
+             size_t i;
+             char *newstring;
+
+             /* Find the length of this string without walking into memory
+                that isn't ours.  */
+             for (i = 0; i < 8; ++i)
+               if (internal_ptr->u.syment._n._n_name[i] == '\0')
+                 break;
+
+             newstring = bfd_zalloc (abfd, (bfd_size_type) (i + 1));
+             if (newstring == NULL)
+               return NULL;
+             strncpy (newstring, internal_ptr->u.syment._n._n_name, i);
+             internal_ptr->u.syment._n._n_n._n_offset = (long int) newstring;
+             internal_ptr->u.syment._n._n_n._n_zeroes = 0;
+           }
+         else if (internal_ptr->u.syment._n._n_n._n_offset == 0)
+           internal_ptr->u.syment._n._n_n._n_offset = (long int) "";
+         else if (!bfd_coff_symname_in_debug (abfd, &internal_ptr->u.syment))
+           {
+             /* Long name already.  Point symbol at the string in the
+                 table.  */
+             if (string_table == NULL)
+               {
+                 string_table = _bfd_coff_read_string_table (abfd);
+                 if (string_table == NULL)
+                   return NULL;
+               }
+             internal_ptr->u.syment._n._n_n._n_offset =
+               ((long int)
+                (string_table
+                 + internal_ptr->u.syment._n._n_n._n_offset));
+           }
+         else
+           {
+             /* Long name in debug section.  Very similar.  */
+             if (debug_section == NULL)
+               debug_section = build_debug_section (abfd);
+             internal_ptr->u.syment._n._n_n._n_offset = (long int)
+               (debug_section + internal_ptr->u.syment._n._n_n._n_offset);
+           }
+       }
+      internal_ptr += internal_ptr->u.syment.n_numaux;
+    }
 
-  obj_raw_syments(abfd) = internal;
-  obj_raw_syment_count(abfd) = internal_ptr - internal;
+  obj_raw_syments (abfd) = internal;
+  BFD_ASSERT (obj_raw_syment_count (abfd)
+             == (unsigned int) (internal_ptr - internal));
 
-  return (internal);
-}                              /* coff_get_normalized_symtab() */
+  return internal;
+}
 
 long
-coff_get_reloc_upper_bound (abfd, asect)
-     bfd            *abfd;
-     sec_ptr         asect;
+coff_get_reloc_upper_bound (bfd *abfd, sec_ptr asect)
 {
-  if (bfd_get_format(abfd) != bfd_object) {
-    bfd_set_error (bfd_error_invalid_operation);
-    return -1;
-  }
-  return (asect->reloc_count + 1) * sizeof(arelent *);
+  if (bfd_get_format (abfd) != bfd_object)
+    {
+      bfd_set_error (bfd_error_invalid_operation);
+      return -1;
+    }
+  return (asect->reloc_count + 1) * sizeof (arelent *);
 }
 
 asymbol *
-coff_make_empty_symbol (abfd)
-     bfd            *abfd;
+coff_make_empty_symbol (bfd *abfd)
 {
-  coff_symbol_type *new = (coff_symbol_type *) bfd_alloc(abfd, sizeof(coff_symbol_type));
-  if (new == NULL) {
-    bfd_set_error (bfd_error_no_memory);
-    return (NULL);
-  }                            /* on error */
-  memset (new, 0, sizeof *new);
+  bfd_size_type amt = sizeof (coff_symbol_type);
+  coff_symbol_type *new = bfd_zalloc (abfd, amt);
+
+  if (new == NULL)
+    return NULL;
   new->symbol.section = 0;
   new->native = 0;
-  new->lineno = (alent *) NULL;
-  new->done_lineno = false;
+  new->lineno = NULL;
+  new->done_lineno = FALSE;
   new->symbol.the_bfd = abfd;
-  return &new->symbol;
+
+  return & new->symbol;
 }
 
 /* Make a debugging symbol.  */
 
 asymbol *
-coff_bfd_make_debug_symbol (abfd, ptr, sz)
-     bfd *abfd;
-     PTR ptr;
-     unsigned long sz;
+coff_bfd_make_debug_symbol (bfd *abfd,
+                           void * ptr ATTRIBUTE_UNUSED,
+                           unsigned long sz ATTRIBUTE_UNUSED)
 {
-  coff_symbol_type *new = (coff_symbol_type *) bfd_alloc(abfd, sizeof(coff_symbol_type));
-  if (new == NULL) {
-    bfd_set_error (bfd_error_no_memory);
-    return (NULL);
-  }                            /* on error */
-  /* @@ This shouldn't be using a constant multiplier.  */
-  new->native = (combined_entry_type *) bfd_zalloc (abfd, sizeof (combined_entry_type) * 10);
+  bfd_size_type amt = sizeof (coff_symbol_type);
+  coff_symbol_type *new = bfd_alloc (abfd, amt);
+
+  if (new == NULL)
+    return NULL;
+  /* @@ The 10 is a guess at a plausible maximum number of aux entries
+     (but shouldn't be a constant).  */
+  amt = sizeof (combined_entry_type) * 10;
+  new->native = bfd_zalloc (abfd, amt);
   if (!new->native)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return (NULL);
-    }                          /* on error */
-  new->symbol.section = &bfd_debug_section;
-  new->lineno = (alent *) NULL;
-  new->done_lineno = false;
+    return NULL;
+  new->symbol.section = bfd_abs_section_ptr;
+  new->symbol.flags = BSF_DEBUGGING;
+  new->lineno = NULL;
+  new->done_lineno = FALSE;
   new->symbol.the_bfd = abfd;
-  return &new->symbol;
+  
+  return & new->symbol;
 }
 
-/*ARGSUSED*/
 void
-coff_get_symbol_info (abfd, symbol, ret)
-     bfd *abfd;
-     asymbol *symbol;
-     symbol_info *ret;
+coff_get_symbol_info (bfd *abfd, asymbol *symbol, symbol_info *ret)
 {
   bfd_symbol_info (symbol, ret);
+
+  if (coffsymbol (symbol)->native != NULL
+      && coffsymbol (symbol)->native->fix_value)
+    ret->value = coffsymbol (symbol)->native->u.syment.n_value -
+      (unsigned long) obj_raw_syments (abfd);
+}
+
+/* Return the COFF syment for a symbol.  */
+
+bfd_boolean
+bfd_coff_get_syment (bfd *abfd,
+                    asymbol *symbol,
+                    struct internal_syment *psyment)
+{
+  coff_symbol_type *csym;
+
+  csym = coff_symbol_from (abfd, symbol);
+  if (csym == NULL || csym->native == NULL)
+    {
+      bfd_set_error (bfd_error_invalid_operation);
+      return FALSE;
+    }
+
+  *psyment = csym->native->u.syment;
+
+  if (csym->native->fix_value)
+    psyment->n_value = psyment->n_value -
+      (unsigned long) obj_raw_syments (abfd);
+
+  /* FIXME: We should handle fix_line here.  */
+
+  return TRUE;
+}
+
+/* Return the COFF auxent for a symbol.  */
+
+bfd_boolean
+bfd_coff_get_auxent (bfd *abfd,
+                    asymbol *symbol,
+                    int indx,
+                    union internal_auxent *pauxent)
+{
+  coff_symbol_type *csym;
+  combined_entry_type *ent;
+
+  csym = coff_symbol_from (abfd, symbol);
+
+  if (csym == NULL
+      || csym->native == NULL
+      || indx >= csym->native->u.syment.n_numaux)
+    {
+      bfd_set_error (bfd_error_invalid_operation);
+      return FALSE;
+    }
+
+  ent = csym->native + indx + 1;
+
+  *pauxent = ent->u.auxent;
+
+  if (ent->fix_tag)
+    pauxent->x_sym.x_tagndx.l =
+      ((combined_entry_type *) pauxent->x_sym.x_tagndx.p
+       - obj_raw_syments (abfd));
+
+  if (ent->fix_end)
+    pauxent->x_sym.x_fcnary.x_fcn.x_endndx.l =
+      ((combined_entry_type *) pauxent->x_sym.x_fcnary.x_fcn.x_endndx.p
+       - obj_raw_syments (abfd));
+
+  if (ent->fix_scnlen)
+    pauxent->x_csect.x_scnlen.l =
+      ((combined_entry_type *) pauxent->x_csect.x_scnlen.p
+       - obj_raw_syments (abfd));
+
+  return TRUE;
 }
 
 /* Print out information about COFF symbol.  */
 
 void
-coff_print_symbol (abfd, filep, symbol, how)
-     bfd *abfd;
-     PTR filep;
-     asymbol *symbol;
-     bfd_print_symbol_type how;
+coff_print_symbol (bfd *abfd,
+                  void * filep,
+                  asymbol *symbol,
+                  bfd_print_symbol_type how)
 {
-  FILE *file = (FILE *) filep;
+  FILE * file = (FILE *) filep;
 
   switch (how)
     {
@@ -1490,31 +1875,36 @@ coff_print_symbol (abfd, filep, symbol, how)
 
     case bfd_print_symbol_more:
       fprintf (file, "coff %s %s",
-              coffsymbol(symbol)->native ? "n" : "g",
-              coffsymbol(symbol)->lineno ? "l" : " ");
+              coffsymbol (symbol)->native ? "n" : "g",
+              coffsymbol (symbol)->lineno ? "l" : " ");
       break;
 
     case bfd_print_symbol_all:
-      if (coffsymbol(symbol)->native) 
+      if (coffsymbol (symbol)->native)
        {
+         bfd_vma val;
          unsigned int aux;
          combined_entry_type *combined = coffsymbol (symbol)->native;
          combined_entry_type *root = obj_raw_syments (abfd);
-         struct lineno_cache_entry *l = coffsymbol(symbol)->lineno;
-       
-         fprintf (file,"[%3d]", combined - root);
+         struct lineno_cache_entry *l = coffsymbol (symbol)->lineno;
+
+         fprintf (file, "[%3ld]", (long) (combined - root));
+
+         if (! combined->fix_value)
+           val = (bfd_vma) combined->u.syment.n_value;
+         else
+           val = combined->u.syment.n_value - (unsigned long) root;
 
-         fprintf (file,
-                  "(sc %2d)(fl 0x%02x)(ty %3x)(sc %3d) (nx %d) 0x%08lx %s",
+         fprintf (file, "(sec %2d)(fl 0x%02x)(ty %3x)(scl %3d) (nx %d) 0x",
                   combined->u.syment.n_scnum,
                   combined->u.syment.n_flags,
                   combined->u.syment.n_type,
                   combined->u.syment.n_sclass,
-                  combined->u.syment.n_numaux,
-                  (unsigned long) combined->u.syment.n_value,
-                  symbol->name);
+                  combined->u.syment.n_numaux);
+         fprintf_vma (file, val);
+         fprintf (file, " %s", symbol->name);
 
-         for (aux = 0; aux < combined->u.syment.n_numaux; aux++) 
+         for (aux = 0; aux < combined->u.syment.n_numaux; aux++)
            {
              combined_entry_type *auxp = combined + aux + 1;
              long tagndx;
@@ -1525,169 +1915,398 @@ coff_print_symbol (abfd, filep, symbol, how)
                tagndx = auxp->u.auxent.x_sym.x_tagndx.l;
 
              fprintf (file, "\n");
+
+             if (bfd_coff_print_aux (abfd, file, root, combined, auxp, aux))
+               continue;
+
              switch (combined->u.syment.n_sclass)
                {
                case C_FILE:
                  fprintf (file, "File ");
                  break;
-               default:
 
+               case C_STAT:
+                 if (combined->u.syment.n_type == T_NULL)
+                   /* Probably a section symbol ?  */
+                   {
+                     fprintf (file, "AUX scnlen 0x%lx nreloc %d nlnno %d",
+                              (long) auxp->u.auxent.x_scn.x_scnlen,
+                              auxp->u.auxent.x_scn.x_nreloc,
+                              auxp->u.auxent.x_scn.x_nlinno);
+                     if (auxp->u.auxent.x_scn.x_checksum != 0
+                         || auxp->u.auxent.x_scn.x_associated != 0
+                         || auxp->u.auxent.x_scn.x_comdat != 0)
+                       fprintf (file, " checksum 0x%lx assoc %d comdat %d",
+                                auxp->u.auxent.x_scn.x_checksum,
+                                auxp->u.auxent.x_scn.x_associated,
+                                auxp->u.auxent.x_scn.x_comdat);
+                     break;
+                   }
+                   /* Otherwise fall through.  */
+               case C_EXT:
+                 if (ISFCN (combined->u.syment.n_type))
+                   {
+                     long next, llnos;
+
+                     if (auxp->fix_end)
+                       next = (auxp->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.p
+                              - root);
+                     else
+                       next = auxp->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.l;
+                     llnos = auxp->u.auxent.x_sym.x_fcnary.x_fcn.x_lnnoptr;
+                     fprintf (file,
+                              "AUX tagndx %ld ttlsiz 0x%lx lnnos %ld next %ld",
+                              tagndx, auxp->u.auxent.x_sym.x_misc.x_fsize,
+                              llnos, next);
+                     break;
+                   }
+                 /* Otherwise fall through.  */
+               default:
                  fprintf (file, "AUX lnno %d size 0x%x tagndx %ld",
                           auxp->u.auxent.x_sym.x_misc.x_lnsz.x_lnno,
                           auxp->u.auxent.x_sym.x_misc.x_lnsz.x_size,
                           tagndx);
+                 if (auxp->fix_end)
+                   fprintf (file, " endndx %ld",
+                            ((long)
+                             (auxp->u.auxent.x_sym.x_fcnary.x_fcn.x_endndx.p
+                              - root)));
                  break;
                }
            }
-       
+
          if (l)
            {
              fprintf (file, "\n%s :", l->u.sym->name);
              l++;
-             while (l->line_number) 
+             while (l->line_number)
                {
-                 fprintf (file, "\n%4d : 0x%lx",
-                          l->line_number,
-                          ((unsigned long)
-                           (l->u.offset + symbol->section->vma)));
+                 fprintf (file, "\n%4d : ", l->line_number);
+                 fprintf_vma (file, l->u.offset + symbol->section->vma);
                  l++;
                }
            }
-       } 
+       }
       else
        {
-         bfd_print_symbol_vandf ((PTR) file, symbol);
+         bfd_print_symbol_vandf (abfd, (void *) file, symbol);
          fprintf (file, " %-5s %s %s %s",
                   symbol->section->name,
-                  coffsymbol(symbol)->native ? "n" : "g",
-                  coffsymbol(symbol)->lineno ? "l" : " ",
+                  coffsymbol (symbol)->native ? "n" : "g",
+                  coffsymbol (symbol)->lineno ? "l" : " ",
                   symbol->name);
        }
     }
 }
 
-/* Provided a BFD, a section and an offset into the section, calculate
-   and return the name of the source file and the line nearest to the
-   wanted location.  */
-
-/*ARGSUSED*/
-boolean
-coff_find_nearest_line (abfd, section, ignore_symbols, offset, filename_ptr,
-                       functionname_ptr, line_ptr)
-     bfd            *abfd;
-     asection       *section;
-     asymbol       **ignore_symbols;
-     bfd_vma         offset;
-     CONST char      **filename_ptr;
-     CONST char       **functionname_ptr;
-     unsigned int   *line_ptr;
+/* Return whether a symbol name implies a local symbol.  In COFF,
+   local symbols generally start with ``.L''.  Most targets use this
+   function for the is_local_label_name entry point, but some may
+   override it.  */
+
+bfd_boolean
+_bfd_coff_is_local_label_name (bfd *abfd ATTRIBUTE_UNUSED,
+                              const char *name)
 {
-  static bfd     *cache_abfd;
-  static asection *cache_section;
-  static bfd_vma  cache_offset;
-  static unsigned int cache_i;
-  static CONST char *cache_function;
-  static unsigned int    line_base = 0;
-
-  unsigned int    i = 0;
-  coff_data_type *cof = coff_data(abfd);
-  /* Run through the raw syments if available */
-  combined_entry_type *p;
-  alent          *l;
+  return name[0] == '.' && name[1] == 'L';
+}
 
+/* Provided a BFD, a section and an offset (in bytes, not octets) into the
+   section, calculate and return the name of the source file and the line
+   nearest to the wanted location.  */
+
+bfd_boolean
+coff_find_nearest_line (bfd *abfd,
+                       asection *section,
+                       asymbol **symbols,
+                       bfd_vma offset,
+                       const char **filename_ptr,
+                       const char **functionname_ptr,
+                       unsigned int *line_ptr)
+{
+  bfd_boolean found;
+  unsigned int i;
+  unsigned int line_base;
+  coff_data_type *cof = coff_data (abfd);
+  /* Run through the raw syments if available.  */
+  combined_entry_type *p;
+  combined_entry_type *pend;
+  alent *l;
+  struct coff_section_tdata *sec_data;
+  bfd_size_type amt;
+
+  /* Before looking through the symbol table, try to use a .stab
+     section to find the information.  */
+  if (! _bfd_stab_section_find_nearest_line (abfd, symbols, section, offset,
+                                            &found, filename_ptr,
+                                            functionname_ptr, line_ptr,
+                                            &coff_data(abfd)->line_info))
+    return FALSE;
+
+  if (found)
+    return TRUE;
+
+  /* Also try examining DWARF2 debugging information.  */
+  if (_bfd_dwarf2_find_nearest_line (abfd, section, symbols, offset,
+                                    filename_ptr, functionname_ptr,
+                                    line_ptr, 0,
+                                    &coff_data(abfd)->dwarf2_find_line_info))
+    return TRUE;
 
   *filename_ptr = 0;
   *functionname_ptr = 0;
   *line_ptr = 0;
 
-  /* Don't try and find line numbers in a non coff file */
-  if (abfd->xvec->flavour != bfd_target_coff_flavour)
-    return false;
+  /* Don't try and find line numbers in a non coff file */
+  if (!bfd_family_coff (abfd))
+    return FALSE;
 
   if (cof == NULL)
-    return false;
+    return FALSE;
 
+  /* Find the first C_FILE symbol.  */
   p = cof->raw_syments;
+  if (!p)
+    return FALSE;
 
-  for (i = 0; i < cof->raw_syment_count; i++) {
-    if (p->u.syment.n_sclass == C_FILE) {
-      /* File name has been moved into symbol */
-      *filename_ptr = (char *) p->u.syment._n._n_n._n_offset;
-      break;
+  pend = p + cof->raw_syment_count;
+  while (p < pend)
+    {
+      if (p->u.syment.n_sclass == C_FILE)
+       break;
+      p += 1 + p->u.syment.n_numaux;
     }
-    p += 1 +  p->u.syment.n_numaux;
-  }
-  /* Now wander though the raw linenumbers of the section */
-  /*
-    If this is the same BFD as we were previously called with and this is
-    the same section, and the offset we want is further down then we can
-    prime the lookup loop
-    */
-  if (abfd == cache_abfd &&
-      section == cache_section &&
-      offset >= cache_offset) {
-    i = cache_i;
-    *functionname_ptr = cache_function;
-  }
-  else {
-    i = 0;
-  }
-  l = &section->lineno[i];
 
-  for (; i < section->lineno_count; i++) {
-    if (l->line_number == 0) {
-      /* Get the symbol this line number points at */
-      coff_symbol_type *coff = (coff_symbol_type *) (l->u.sym);
-      if (coff->symbol.value > offset)
-       break;
-      *functionname_ptr = coff->symbol.name;
-      if (coff->native) {
-       combined_entry_type  *s = coff->native;
-       s = s + 1 + s->u.syment.n_numaux;
-       /*
-         S should now point to the .bf of the function
-         */
-       if (s->u.syment.n_numaux) {
-         /*
-           The linenumber is stored in the auxent
-           */
-         union internal_auxent   *a = &((s + 1)->u.auxent);
-         line_base = a->x_sym.x_misc.x_lnsz.x_lnno;
-         *line_ptr = line_base;
+  if (p < pend)
+    {
+      bfd_vma sec_vma;
+      bfd_vma maxdiff;
+
+      /* Look through the C_FILE symbols to find the best one.  */
+      sec_vma = bfd_get_section_vma (abfd, section);
+      *filename_ptr = (char *) p->u.syment._n._n_n._n_offset;
+      maxdiff = (bfd_vma) 0 - (bfd_vma) 1;
+      while (1)
+       {
+         combined_entry_type *p2;
+
+         for (p2 = p + 1 + p->u.syment.n_numaux;
+              p2 < pend;
+              p2 += 1 + p2->u.syment.n_numaux)
+           {
+             if (p2->u.syment.n_scnum > 0
+                 && (section
+                     == coff_section_from_bfd_index (abfd,
+                                                     p2->u.syment.n_scnum)))
+               break;
+             if (p2->u.syment.n_sclass == C_FILE)
+               {
+                 p2 = pend;
+                 break;
+               }
+           }
+
+         /* We use <= MAXDIFF here so that if we get a zero length
+             file, we actually use the next file entry.  */
+         if (p2 < pend
+             && offset + sec_vma >= (bfd_vma) p2->u.syment.n_value
+             && offset + sec_vma - (bfd_vma) p2->u.syment.n_value <= maxdiff)
+           {
+             *filename_ptr = (char *) p->u.syment._n._n_n._n_offset;
+             maxdiff = offset + sec_vma - p2->u.syment.n_value;
+           }
+
+         /* Avoid endless loops on erroneous files by ensuring that
+            we always move forward in the file.  */
+         if (p >= cof->raw_syments + p->u.syment.n_value)
+           break;
+
+         p = cof->raw_syments + p->u.syment.n_value;
+         if (p > pend || p->u.syment.n_sclass != C_FILE)
+           break;
        }
-      }
     }
-    else {
-      if (l->u.offset > offset)
-       break;
-      *line_ptr = l->line_number + line_base - 1;
+
+  /* Now wander though the raw linenumbers of the section.  */
+  /* If we have been called on this section before, and th. e offset we
+     want is further down then we can prime the lookup loop.  */
+  sec_data = coff_section_data (abfd, section);
+  if (sec_data != NULL
+      && sec_data->i > 0
+      && offset >= sec_data->offset)
+    {
+      i = sec_data->i;
+      *functionname_ptr = sec_data->function;
+      line_base = sec_data->line_base;
+    }
+  else
+    {
+      i = 0;
+      line_base = 0;
+    }
+
+  if (section->lineno != NULL)
+    {
+      bfd_vma last_value = 0;
+
+      l = &section->lineno[i];
+
+      for (; i < section->lineno_count; i++)
+       {
+         if (l->line_number == 0)
+           {
+             /* Get the symbol this line number points at.  */
+             coff_symbol_type *coff = (coff_symbol_type *) (l->u.sym);
+             if (coff->symbol.value > offset)
+               break;
+             *functionname_ptr = coff->symbol.name;
+             last_value = coff->symbol.value;
+             if (coff->native)
+               {
+                 combined_entry_type *s = coff->native;
+                 s = s + 1 + s->u.syment.n_numaux;
+
+                 /* In XCOFF a debugging symbol can follow the
+                    function symbol.  */
+                 if (s->u.syment.n_scnum == N_DEBUG)
+                   s = s + 1 + s->u.syment.n_numaux;
+
+                 /* S should now point to the .bf of the function.  */
+                 if (s->u.syment.n_numaux)
+                   {
+                     /* The linenumber is stored in the auxent.  */
+                     union internal_auxent *a = &((s + 1)->u.auxent);
+                     line_base = a->x_sym.x_misc.x_lnsz.x_lnno;
+                     *line_ptr = line_base;
+                   }
+               }
+           }
+         else
+           {
+             if (l->u.offset > offset)
+               break;
+             *line_ptr = l->line_number + line_base - 1;
+           }
+         l++;
+       }
+
+      /* If we fell off the end of the loop, then assume that this
+        symbol has no line number info.  Otherwise, symbols with no
+        line number info get reported with the line number of the
+        last line of the last symbol which does have line number
+        info.  We use 0x100 as a slop to account for cases where the
+        last line has executable code.  */
+      if (i >= section->lineno_count
+         && last_value != 0
+         && offset - last_value > 0x100)
+       {
+         *functionname_ptr = NULL;
+         *line_ptr = 0;
+       }
     }
-    l++;
-  }
 
-  cache_abfd = abfd;
-  cache_section = section;
-  cache_offset = offset;
-  cache_i = i;
-  cache_function = *functionname_ptr;
+  /* Cache the results for the next call.  */
+  if (sec_data == NULL && section->owner == abfd)
+    {
+      amt = sizeof (struct coff_section_tdata);
+      section->used_by_bfd = bfd_zalloc (abfd, amt);
+      sec_data = (struct coff_section_tdata *) section->used_by_bfd;
+    }
+  if (sec_data != NULL)
+    {
+      sec_data->offset = offset;
+      sec_data->i = i;
+      sec_data->function = *functionname_ptr;
+      sec_data->line_base = line_base;
+    }
 
-  return true;
+  return TRUE;
 }
 
 int
-coff_sizeof_headers (abfd, reloc)
-     bfd *abfd;
-     boolean reloc;
+coff_sizeof_headers (bfd *abfd, bfd_boolean reloc)
+{
+  size_t size;
+
+  if (! reloc)
+    size = bfd_coff_filhsz (abfd) + bfd_coff_aoutsz (abfd);
+  else
+    size = bfd_coff_filhsz (abfd);
+
+  size += abfd->section_count * bfd_coff_scnhsz (abfd);
+  return size;
+}
+
+/* Change the class of a coff symbol held by BFD.  */
+
+bfd_boolean
+bfd_coff_set_symbol_class (bfd *         abfd,
+                          asymbol *     symbol,
+                          unsigned int  class)
 {
-    size_t size;
+  coff_symbol_type * csym;
 
-    if (reloc == false) {
-       size = bfd_coff_filhsz (abfd) + bfd_coff_aoutsz (abfd);
+  csym = coff_symbol_from (abfd, symbol);
+  if (csym == NULL)
+    {
+      bfd_set_error (bfd_error_invalid_operation);
+      return FALSE;
     }
-    else {
-       size = bfd_coff_filhsz (abfd);
+  else if (csym->native == NULL)
+    {
+      /* This is an alien symbol which no native coff backend data.
+        We cheat here by creating a fake native entry for it and
+        then filling in the class.  This code is based on that in
+        coff_write_alien_symbol().  */
+
+      combined_entry_type * native;
+      bfd_size_type amt = sizeof (* native);
+
+      native = bfd_zalloc (abfd, amt);
+      if (native == NULL)
+       return FALSE;
+
+      native->u.syment.n_type   = T_NULL;
+      native->u.syment.n_sclass = class;
+
+      if (bfd_is_und_section (symbol->section))
+       {
+         native->u.syment.n_scnum = N_UNDEF;
+         native->u.syment.n_value = symbol->value;
+       }
+      else if (bfd_is_com_section (symbol->section))
+       {
+         native->u.syment.n_scnum = N_UNDEF;
+         native->u.syment.n_value = symbol->value;
+       }
+      else
+       {
+         native->u.syment.n_scnum =
+           symbol->section->output_section->target_index;
+         native->u.syment.n_value = (symbol->value
+                                     + symbol->section->output_offset);
+         if (! obj_pe (abfd))
+           native->u.syment.n_value += symbol->section->output_section->vma;
+
+         /* Copy the any flags from the file header into the symbol.
+            FIXME: Why?  */
+         native->u.syment.n_flags = bfd_asymbol_bfd (& csym->symbol)->flags;
+       }
+
+      csym->native = native;
     }
+  else
+    csym->native->u.syment.n_sclass = class;
 
-    size +=  abfd->section_count * bfd_coff_scnhsz (abfd);
-    return size;
+  return TRUE;
+}
+
+struct coff_comdat_info *
+bfd_coff_get_comdat_section (bfd *abfd, struct bfd_section *sec)
+{
+  if (bfd_get_flavour (abfd) == bfd_target_coff_flavour
+      && coff_section_data (abfd, sec) != NULL)
+    return coff_section_data (abfd, sec)->comdat;
+  else
+    return NULL;
 }
This page took 0.063982 seconds and 4 git commands to generate.