]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - bfd/som.c
Update year range in copyright notice of binutils files
[thirdparty/binutils-gdb.git] / bfd / som.c
index f36e3bc29285abf5b6061c54be5fac564f53780e..5897ae6c5b7e29d07f5f38454ab08cdd9830d643 100644 (file)
--- a/bfd/som.c
+++ b/bfd/som.c
@@ -1,7 +1,5 @@
 /* bfd back-end for HP PA-RISC SOM objects.
-   Copyright 1990, 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999,
-   2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007
-   Free Software Foundation, Inc.
+   Copyright (C) 1990-2021 Free Software Foundation, Inc.
 
    Contributed by the Center for Software Science at the
    University of Utah.
    Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA
    02110-1301, USA.  */
 
-#include "alloca-conf.h"
 #include "sysdep.h"
+#include "alloca-conf.h"
 #include "bfd.h"
-
-#if defined (HOST_HPPAHPUX) || defined (HOST_HPPABSD) || defined (HOST_HPPAOSF) || defined(HOST_HPPAMPEIX)
-
+#include "libiberty.h"
 #include "libbfd.h"
 #include "som.h"
 #include "safe-ctype.h"
-
-#include <sys/param.h>
-#include <signal.h>
-#include <machine/reg.h>
-#include <sys/file.h>
+#include "som/reloc.h"
+#include "aout/ar.h"
 
 static bfd_reloc_status_type hppa_som_reloc
   (bfd *, arelent *, asymbol *, void *, asection *, bfd *, char **);
@@ -44,7 +37,7 @@ static bfd_boolean som_mkobject (bfd *);
 static bfd_boolean som_is_space (asection *);
 static bfd_boolean som_is_subspace (asection *);
 static int compare_subspaces (const void *, const void *);
-static unsigned long som_compute_checksum (bfd *);
+static uint32_t som_compute_checksum (struct som_external_header *);
 static bfd_boolean som_build_and_write_symbol_table (bfd *);
 static unsigned int som_slurp_symbol_table (bfd *);
 
@@ -121,7 +114,7 @@ static unsigned int som_slurp_symbol_table (bfd *);
 
    Note one side effect of using a R_PREV_FIXUP is the relocation that
    is being repeated moves to the front of the queue.  */
-struct reloc_queue
+static struct reloc_queue
 {
   unsigned char *reloc;
   unsigned int size;
@@ -145,7 +138,7 @@ typedef enum
 
 struct section_to_type
 {
-  char *section;
+  const char *section;
   char type;
 };
 
@@ -328,7 +321,7 @@ static const struct fixup_format som_fixup_formats[256] =
   /* R_DATA_ONE_SYMBOL.  */
   {  0, "L4=Sb=" },            /* 0x25 */
   {  1, "L4=Sd=" },            /* 0x26 */
-  /* R_DATA_PLEBEL.  */
+  /* R_DATA_PLABEL.  */
   {  0, "L4=Sb=" },            /* 0x27 */
   {  1, "L4=Sd=" },            /* 0x28 */
   /* R_SPACE_REF.  */
@@ -412,8 +405,9 @@ static const struct fixup_format som_fixup_formats[256] =
   { 31, "L4=SD=" },            /* 0x6f */
   { 32, "L4=Sb=" },            /* 0x70 */
   { 33, "L4=Sd=" },            /* 0x71 */
+  /* R_DATA_GPREL.  */
+  {  0, "L4=Sd=" },            /* 0x72 */
   /* R_RESERVED.  */
-  {  0, "" },                  /* 0x72 */
   {  0, "" },                  /* 0x73 */
   {  0, "" },                  /* 0x74 */
   {  0, "" },                  /* 0x75 */
@@ -635,17 +629,6 @@ static const int comp3_opcodes[] =
 };
 
 /* These apparently are not in older versions of hpux reloc.h (hpux7).  */
-#ifndef R_DLT_REL
-#define R_DLT_REL 0x78
-#endif
-
-#ifndef R_AUX_UNWIND
-#define R_AUX_UNWIND 0xcf
-#endif
-
-#ifndef R_SEC_STMT
-#define R_SEC_STMT 0xd7
-#endif
 
 /* And these first appeared in hpux10.  */
 #ifndef R_SHORT_PCREL_MODE
@@ -653,34 +636,6 @@ static const int comp3_opcodes[] =
 #define R_SHORT_PCREL_MODE 0x3e
 #endif
 
-#ifndef R_LONG_PCREL_MODE
-#define R_LONG_PCREL_MODE 0x3f
-#endif
-
-#ifndef R_N0SEL
-#define R_N0SEL 0xd8
-#endif
-
-#ifndef R_N1SEL
-#define R_N1SEL 0xd9
-#endif
-
-#ifndef R_LINETAB
-#define R_LINETAB 0xda
-#endif
-
-#ifndef R_LINETAB_ESC
-#define R_LINETAB_ESC 0xdb
-#endif
-
-#ifndef R_LTP_OVERRIDE
-#define R_LTP_OVERRIDE 0xdc
-#endif
-
-#ifndef R_COMMENT
-#define R_COMMENT 0xdd
-#endif
-
 #define SOM_HOWTO(TYPE, NAME)  \
   HOWTO(TYPE, 0, 0, 32, FALSE, 0, 0, hppa_som_reloc, NAME, FALSE, 0, 0, FALSE)
 
@@ -800,7 +755,7 @@ static reloc_howto_type som_hppa_howto_table[] =
   SOM_HOWTO (R_DP_RELATIVE, "R_DP_RELATIVE"),
   SOM_HOWTO (R_DP_RELATIVE, "R_DP_RELATIVE"),
   SOM_HOWTO (R_DP_RELATIVE, "R_DP_RELATIVE"),
-  SOM_HOWTO (R_DP_RELATIVE, "R_DP_RELATIVE"),
+  SOM_HOWTO (R_DATA_GPREL, "R_DATA_GPREL"),
   SOM_HOWTO (R_RESERVED, "R_RESERVED"),
   SOM_HOWTO (R_RESERVED, "R_RESERVED"),
   SOM_HOWTO (R_RESERVED, "R_RESERVED"),
@@ -981,12 +936,12 @@ som_reloc_queue_insert (unsigned char *p,
    to the front of the queue.  */
 
 static void
-som_reloc_queue_fix (struct reloc_queue *queue, unsigned int index)
+som_reloc_queue_fix (struct reloc_queue *queue, unsigned int idx)
 {
-  if (index == 0)
+  if (idx == 0)
     return;
 
-  if (index == 1)
+  if (idx == 1)
     {
       unsigned char *tmp1 = queue[0].reloc;
       unsigned int tmp2 = queue[0].size;
@@ -998,7 +953,7 @@ som_reloc_queue_fix (struct reloc_queue *queue, unsigned int index)
       return;
     }
 
-  if (index == 2)
+  if (idx == 2)
     {
       unsigned char *tmp1 = queue[0].reloc;
       unsigned int tmp2 = queue[0].size;
@@ -1012,7 +967,7 @@ som_reloc_queue_fix (struct reloc_queue *queue, unsigned int index)
       return;
     }
 
-  if (index == 3)
+  if (idx == 3)
     {
       unsigned char *tmp1 = queue[0].reloc;
       unsigned int tmp2 = queue[0].size;
@@ -1470,7 +1425,7 @@ hppa_som_gen_reloc_type (bfd *abfd,
       /* The difference of two symbols needs *very* special handling.  */
       if (sym_diff)
        {
-         bfd_size_type amt = sizeof (int);
+         size_t amt = sizeof (int);
 
          final_types[0] = bfd_alloc (abfd, amt);
          final_types[1] = bfd_alloc (abfd, amt);
@@ -1546,13 +1501,15 @@ hppa_som_gen_reloc_type (bfd *abfd,
          || field == e_lpsel
          || field == e_rpsel)
        *final_type = R_DATA_PLABEL;
+      else if (field == e_fsel && format == 32)
+       *final_type = R_DATA_GPREL;
       break;
 
     case R_HPPA_COMPLEX:
       /* The difference of two symbols needs *very* special handling.  */
       if (sym_diff)
        {
-         bfd_size_type amt = sizeof (int);
+         size_t amt = sizeof (int);
 
          final_types[0] = bfd_alloc (abfd, amt);
          final_types[1] = bfd_alloc (abfd, amt);
@@ -1591,7 +1548,7 @@ hppa_som_gen_reloc_type (bfd *abfd,
        /* If we have short and long pcrel modes, then generate the proper
           mode selector, then the pcrel relocation.  Redundant selectors
           will be eliminated as the relocs are sized and emitted.  */
-       bfd_size_type amt = sizeof (int);
+       size_t amt = sizeof (int);
 
        final_types[0] = bfd_alloc (abfd, amt);
        if (!final_types[0])
@@ -1642,12 +1599,355 @@ som_bfd_reloc_name_lookup (bfd *abfd ATTRIBUTE_UNUSED,
   return NULL;
 }
 
+static void
+som_swap_clock_in (struct som_external_clock *src,
+                  struct som_clock *dst)
+{
+  dst->secs = bfd_getb32 (src->secs);
+  dst->nanosecs = bfd_getb32 (src->nanosecs);
+}
+
+static void
+som_swap_clock_out (struct som_clock *src,
+                   struct som_external_clock *dst)
+{
+  bfd_putb32 (src->secs, dst->secs);
+  bfd_putb32 (src->nanosecs, dst->nanosecs);
+}
+
+static void
+som_swap_header_in (struct som_external_header *src,
+                   struct som_header *dst)
+{
+  dst->system_id = bfd_getb16 (src->system_id);
+  dst->a_magic = bfd_getb16 (src->a_magic);
+  dst->version_id = bfd_getb32 (src->version_id);
+  som_swap_clock_in (&src->file_time, &dst->file_time);
+  dst->entry_space = bfd_getb32 (src->entry_space);
+  dst->entry_subspace = bfd_getb32 (src->entry_subspace);
+  dst->entry_offset = bfd_getb32 (src->entry_offset);
+  dst->aux_header_location = bfd_getb32 (src->aux_header_location);
+  dst->aux_header_size = bfd_getb32 (src->aux_header_size);
+  dst->som_length = bfd_getb32 (src->som_length);
+  dst->presumed_dp = bfd_getb32 (src->presumed_dp);
+  dst->space_location = bfd_getb32 (src->space_location);
+  dst->space_total = bfd_getb32 (src->space_total);
+  dst->subspace_location = bfd_getb32 (src->subspace_location);
+  dst->subspace_total = bfd_getb32 (src->subspace_total);
+  dst->loader_fixup_location = bfd_getb32 (src->loader_fixup_location);
+  dst->loader_fixup_total = bfd_getb32 (src->loader_fixup_total);
+  dst->space_strings_location = bfd_getb32 (src->space_strings_location);
+  dst->space_strings_size = bfd_getb32 (src->space_strings_size);
+  dst->init_array_location = bfd_getb32 (src->init_array_location);
+  dst->init_array_total = bfd_getb32 (src->init_array_total);
+  dst->compiler_location = bfd_getb32 (src->compiler_location);
+  dst->compiler_total = bfd_getb32 (src->compiler_total);
+  dst->symbol_location = bfd_getb32 (src->symbol_location);
+  dst->symbol_total = bfd_getb32 (src->symbol_total);
+  dst->fixup_request_location = bfd_getb32 (src->fixup_request_location);
+  dst->fixup_request_total = bfd_getb32 (src->fixup_request_total);
+  dst->symbol_strings_location = bfd_getb32 (src->symbol_strings_location);
+  dst->symbol_strings_size = bfd_getb32 (src->symbol_strings_size);
+  dst->unloadable_sp_location = bfd_getb32 (src->unloadable_sp_location);
+  dst->unloadable_sp_size = bfd_getb32 (src->unloadable_sp_size);
+  dst->checksum = bfd_getb32 (src->checksum);
+}
+
+static void
+som_swap_header_out (struct som_header *src,
+                   struct som_external_header *dst)
+{
+  bfd_putb16 (src->system_id, dst->system_id);
+  bfd_putb16 (src->a_magic, dst->a_magic);
+  bfd_putb32 (src->version_id, dst->version_id);
+  som_swap_clock_out (&src->file_time, &dst->file_time);
+  bfd_putb32 (src->entry_space, dst->entry_space);
+  bfd_putb32 (src->entry_subspace, dst->entry_subspace);
+  bfd_putb32 (src->entry_offset, dst->entry_offset);
+  bfd_putb32 (src->aux_header_location, dst->aux_header_location);
+  bfd_putb32 (src->aux_header_size, dst->aux_header_size);
+  bfd_putb32 (src->som_length, dst->som_length);
+  bfd_putb32 (src->presumed_dp, dst->presumed_dp);
+  bfd_putb32 (src->space_location, dst->space_location);
+  bfd_putb32 (src->space_total, dst->space_total);
+  bfd_putb32 (src->subspace_location, dst->subspace_location);
+  bfd_putb32 (src->subspace_total, dst->subspace_total);
+  bfd_putb32 (src->loader_fixup_location, dst->loader_fixup_location);
+  bfd_putb32 (src->loader_fixup_total, dst->loader_fixup_total);
+  bfd_putb32 (src->space_strings_location, dst->space_strings_location);
+  bfd_putb32 (src->space_strings_size, dst->space_strings_size);
+  bfd_putb32 (src->init_array_location, dst->init_array_location);
+  bfd_putb32 (src->init_array_total, dst->init_array_total);
+  bfd_putb32 (src->compiler_location, dst->compiler_location);
+  bfd_putb32 (src->compiler_total, dst->compiler_total);
+  bfd_putb32 (src->symbol_location, dst->symbol_location);
+  bfd_putb32 (src->symbol_total, dst->symbol_total);
+  bfd_putb32 (src->fixup_request_location, dst->fixup_request_location);
+  bfd_putb32 (src->fixup_request_total, dst->fixup_request_total);
+  bfd_putb32 (src->symbol_strings_location, dst->symbol_strings_location);
+  bfd_putb32 (src->symbol_strings_size, dst->symbol_strings_size);
+  bfd_putb32 (src->unloadable_sp_location, dst->unloadable_sp_location);
+  bfd_putb32 (src->unloadable_sp_size, dst->unloadable_sp_size);
+  bfd_putb32 (src->checksum, dst->checksum);
+}
+
+static void
+som_swap_space_dictionary_in (struct som_external_space_dictionary_record *src,
+                             struct som_space_dictionary_record *dst)
+{
+  unsigned int flags;
+
+  dst->name = bfd_getb32 (src->name);
+  flags = bfd_getb32 (src->flags);
+  dst->is_loadable = (flags & SOM_SPACE_IS_LOADABLE) != 0;
+  dst->is_defined = (flags & SOM_SPACE_IS_DEFINED) != 0;
+  dst->is_private = (flags & SOM_SPACE_IS_PRIVATE) != 0;
+  dst->has_intermediate_code = (flags & SOM_SPACE_HAS_INTERMEDIATE_CODE) != 0;
+  dst->is_tspecific = (flags & SOM_SPACE_IS_TSPECIFIC) != 0;
+  dst->reserved = 0;
+  dst->sort_key = (flags >> SOM_SPACE_SORT_KEY_SH) & SOM_SPACE_SORT_KEY_MASK;
+  dst->reserved2 = 0;
+  dst->space_number = bfd_getb32 (src->space_number);
+  dst->subspace_index = bfd_getb32 (src->subspace_index);
+  dst->subspace_quantity = bfd_getb32 (src->subspace_quantity);
+  dst->loader_fix_index = bfd_getb32 (src->loader_fix_index);
+  dst->loader_fix_quantity = bfd_getb32 (src->loader_fix_quantity);
+  dst->init_pointer_index = bfd_getb32 (src->init_pointer_index);
+  dst->init_pointer_quantity = bfd_getb32 (src->init_pointer_quantity);
+}
+
+static void
+som_swap_space_dictionary_out (struct som_space_dictionary_record *src,
+                              struct som_external_space_dictionary_record *dst)
+{
+  unsigned int flags;
+
+  bfd_putb32 (src->name, dst->name);
+
+  flags = 0;
+  if (src->is_loadable)
+    flags |= SOM_SPACE_IS_LOADABLE;
+  if (src->is_defined)
+    flags |= SOM_SPACE_IS_DEFINED;
+  if (src->is_private)
+    flags |= SOM_SPACE_IS_PRIVATE;
+  if (src->has_intermediate_code)
+    flags |= SOM_SPACE_HAS_INTERMEDIATE_CODE;
+  if (src->is_tspecific)
+    flags |= SOM_SPACE_IS_TSPECIFIC;
+  flags |= (src->sort_key & SOM_SPACE_SORT_KEY_MASK) << SOM_SPACE_SORT_KEY_SH;
+  bfd_putb32 (flags, dst->flags);
+  bfd_putb32 (src->space_number, dst->space_number);
+  bfd_putb32 (src->subspace_index, dst->subspace_index);
+  bfd_putb32 (src->subspace_quantity, dst->subspace_quantity);
+  bfd_putb32 (src->loader_fix_index, dst->loader_fix_index);
+  bfd_putb32 (src->loader_fix_quantity, dst->loader_fix_quantity);
+  bfd_putb32 (src->init_pointer_index, dst->init_pointer_index);
+  bfd_putb32 (src->init_pointer_quantity, dst->init_pointer_quantity);
+}
+
+static void
+som_swap_subspace_dictionary_in
+  (struct som_external_subspace_dictionary_record *src,
+   struct som_subspace_dictionary_record *dst)
+{
+  unsigned int flags;
+  dst->space_index = bfd_getb32 (src->space_index);
+  flags = bfd_getb32 (src->flags);
+  dst->access_control_bits = (flags >> SOM_SUBSPACE_ACCESS_CONTROL_BITS_SH)
+    & SOM_SUBSPACE_ACCESS_CONTROL_BITS_MASK;
+  dst->memory_resident = (flags & SOM_SUBSPACE_MEMORY_RESIDENT) != 0;
+  dst->dup_common = (flags & SOM_SUBSPACE_DUP_COMMON) != 0;
+  dst->is_common = (flags & SOM_SUBSPACE_IS_COMMON) != 0;
+  dst->is_loadable = (flags & SOM_SUBSPACE_IS_LOADABLE) != 0;
+  dst->quadrant = (flags >> SOM_SUBSPACE_QUADRANT_SH)
+    & SOM_SUBSPACE_QUADRANT_MASK;
+  dst->initially_frozen = (flags & SOM_SUBSPACE_INITIALLY_FROZEN) != 0;
+  dst->is_first = (flags & SOM_SUBSPACE_IS_FIRST) != 0;
+  dst->code_only = (flags & SOM_SUBSPACE_CODE_ONLY) != 0;
+  dst->sort_key = (flags >> SOM_SUBSPACE_SORT_KEY_SH)
+    & SOM_SUBSPACE_SORT_KEY_MASK;
+  dst->replicate_init = (flags & SOM_SUBSPACE_REPLICATE_INIT) != 0;
+  dst->continuation = (flags & SOM_SUBSPACE_CONTINUATION) != 0;
+  dst->is_tspecific = (flags & SOM_SUBSPACE_IS_TSPECIFIC) != 0;
+  dst->is_comdat = (flags & SOM_SUBSPACE_IS_COMDAT) != 0;
+  dst->reserved = 0;
+  dst->file_loc_init_value = bfd_getb32 (src->file_loc_init_value);
+  dst->initialization_length = bfd_getb32 (src->initialization_length);
+  dst->subspace_start = bfd_getb32 (src->subspace_start);
+  dst->subspace_length = bfd_getb32 (src->subspace_length);
+  dst->alignment = bfd_getb32 (src->alignment);
+  dst->name = bfd_getb32 (src->name);
+  dst->fixup_request_index = bfd_getb32 (src->fixup_request_index);
+  dst->fixup_request_quantity = bfd_getb32 (src->fixup_request_quantity);
+}
+
+static void
+som_swap_subspace_dictionary_record_out
+  (struct som_subspace_dictionary_record *src,
+   struct som_external_subspace_dictionary_record *dst)
+{
+  unsigned int flags;
+
+  bfd_putb32 (src->space_index, dst->space_index);
+  flags = (src->access_control_bits & SOM_SUBSPACE_ACCESS_CONTROL_BITS_MASK)
+    << SOM_SUBSPACE_ACCESS_CONTROL_BITS_SH;
+  if (src->memory_resident)
+    flags |= SOM_SUBSPACE_MEMORY_RESIDENT;
+  if (src->dup_common)
+    flags |= SOM_SUBSPACE_DUP_COMMON;
+  if (src->is_common)
+    flags |= SOM_SUBSPACE_IS_COMMON;
+  if (src->is_loadable)
+    flags |= SOM_SUBSPACE_IS_LOADABLE;
+  flags |= (src->quadrant & SOM_SUBSPACE_QUADRANT_MASK)
+    << SOM_SUBSPACE_QUADRANT_SH;
+  if (src->initially_frozen)
+    flags |= SOM_SUBSPACE_INITIALLY_FROZEN;
+  if (src->is_first)
+    flags |= SOM_SUBSPACE_IS_FIRST;
+  if (src->code_only)
+    flags |= SOM_SUBSPACE_CODE_ONLY;
+  flags |= (src->sort_key & SOM_SUBSPACE_SORT_KEY_MASK)
+    << SOM_SUBSPACE_SORT_KEY_SH;
+  if (src->replicate_init)
+    flags |= SOM_SUBSPACE_REPLICATE_INIT;
+  if (src->continuation)
+    flags |= SOM_SUBSPACE_CONTINUATION;
+  if (src->is_tspecific)
+    flags |= SOM_SUBSPACE_IS_TSPECIFIC;
+  if (src->is_comdat)
+    flags |= SOM_SUBSPACE_IS_COMDAT;
+  bfd_putb32 (flags, dst->flags);
+  bfd_putb32 (src->file_loc_init_value, dst->file_loc_init_value);
+  bfd_putb32 (src->initialization_length, dst->initialization_length);
+  bfd_putb32 (src->subspace_start, dst->subspace_start);
+  bfd_putb32 (src->subspace_length, dst->subspace_length);
+  bfd_putb32 (src->alignment, dst->alignment);
+  bfd_putb32 (src->name, dst->name);
+  bfd_putb32 (src->fixup_request_index, dst->fixup_request_index);
+  bfd_putb32 (src->fixup_request_quantity, dst->fixup_request_quantity);
+}
+
+static void
+som_swap_aux_id_in (struct som_external_aux_id *src,
+                   struct som_aux_id *dst)
+{
+  unsigned int flags = bfd_getb32 (src->flags);
+
+  dst->mandatory = (flags & SOM_AUX_ID_MANDATORY) != 0;
+  dst->copy = (flags & SOM_AUX_ID_COPY) != 0;
+  dst->append = (flags & SOM_AUX_ID_APPEND) != 0;
+  dst->ignore = (flags & SOM_AUX_ID_IGNORE) != 0;
+  dst->type = (flags >> SOM_AUX_ID_TYPE_SH) & SOM_AUX_ID_TYPE_MASK;
+  dst->length = bfd_getb32 (src->length);
+}
+
+static void
+som_swap_aux_id_out (struct som_aux_id *src,
+                   struct som_external_aux_id *dst)
+{
+  unsigned int flags = 0;
+
+  if (src->mandatory)
+    flags |= SOM_AUX_ID_MANDATORY;
+  if (src->copy)
+    flags |= SOM_AUX_ID_COPY;
+  if (src->append)
+    flags |= SOM_AUX_ID_APPEND;
+  if (src->ignore)
+    flags |= SOM_AUX_ID_IGNORE;
+  flags |= (src->type & SOM_AUX_ID_TYPE_MASK) << SOM_AUX_ID_TYPE_SH;
+  bfd_putb32 (flags, dst->flags);
+  bfd_putb32 (src->length, dst->length);
+}
+
+static void
+som_swap_string_auxhdr_out (struct som_string_auxhdr *src,
+                           struct som_external_string_auxhdr *dst)
+{
+  som_swap_aux_id_out (&src->header_id, &dst->header_id);
+  bfd_putb32 (src->string_length, dst->string_length);
+}
+
+static void
+som_swap_compilation_unit_out (struct som_compilation_unit *src,
+                              struct som_external_compilation_unit *dst)
+{
+  bfd_putb32 (src->name.strx, dst->name);
+  bfd_putb32 (src->language_name.strx, dst->language_name);
+  bfd_putb32 (src->product_id.strx, dst->product_id);
+  bfd_putb32 (src->version_id.strx, dst->version_id);
+  bfd_putb32 (src->flags, dst->flags);
+  som_swap_clock_out (&src->compile_time, &dst->compile_time);
+  som_swap_clock_out (&src->source_time, &dst->source_time);
+}
+
+static void
+som_swap_exec_auxhdr_in (struct som_external_exec_auxhdr *src,
+                        struct som_exec_auxhdr *dst)
+{
+  som_swap_aux_id_in (&src->som_auxhdr, &dst->som_auxhdr);
+  dst->exec_tsize = bfd_getb32 (src->exec_tsize);
+  dst->exec_tmem = bfd_getb32 (src->exec_tmem);
+  dst->exec_tfile = bfd_getb32 (src->exec_tfile);
+  dst->exec_dsize = bfd_getb32 (src->exec_dsize);
+  dst->exec_dmem = bfd_getb32 (src->exec_dmem);
+  dst->exec_dfile = bfd_getb32 (src->exec_dfile);
+  dst->exec_bsize = bfd_getb32 (src->exec_bsize);
+  dst->exec_entry = bfd_getb32 (src->exec_entry);
+  dst->exec_flags = bfd_getb32 (src->exec_flags);
+  dst->exec_bfill = bfd_getb32 (src->exec_bfill);
+}
+
+static void
+som_swap_exec_auxhdr_out (struct som_exec_auxhdr *src,
+                        struct som_external_exec_auxhdr *dst)
+{
+  som_swap_aux_id_out (&src->som_auxhdr, &dst->som_auxhdr);
+  bfd_putb32 (src->exec_tsize, dst->exec_tsize);
+  bfd_putb32 (src->exec_tmem, dst->exec_tmem);
+  bfd_putb32 (src->exec_tfile, dst->exec_tfile);
+  bfd_putb32 (src->exec_dsize, dst->exec_dsize);
+  bfd_putb32 (src->exec_dmem, dst->exec_dmem);
+  bfd_putb32 (src->exec_dfile, dst->exec_dfile);
+  bfd_putb32 (src->exec_bsize, dst->exec_bsize);
+  bfd_putb32 (src->exec_entry, dst->exec_entry);
+  bfd_putb32 (src->exec_flags, dst->exec_flags);
+  bfd_putb32 (src->exec_bfill, dst->exec_bfill);
+}
+
+static void
+som_swap_lst_header_in (struct som_external_lst_header *src,
+                       struct som_lst_header *dst)
+{
+  dst->system_id = bfd_getb16 (src->system_id);
+  dst->a_magic = bfd_getb16 (src->a_magic);
+  dst->version_id = bfd_getb32 (src->version_id);
+  som_swap_clock_in (&src->file_time, &dst->file_time);
+  dst->hash_loc = bfd_getb32 (src->hash_loc);
+  dst->hash_size = bfd_getb32 (src->hash_size);
+  dst->module_count = bfd_getb32 (src->module_count);
+  dst->module_limit = bfd_getb32 (src->module_limit);
+  dst->dir_loc = bfd_getb32 (src->dir_loc);
+  dst->export_loc = bfd_getb32 (src->export_loc);
+  dst->export_count = bfd_getb32 (src->export_count);
+  dst->import_loc = bfd_getb32 (src->import_loc);
+  dst->aux_loc = bfd_getb32 (src->aux_loc);
+  dst->aux_size = bfd_getb32 (src->aux_size);
+  dst->string_loc = bfd_getb32 (src->string_loc);
+  dst->string_size = bfd_getb32 (src->string_size);
+  dst->free_list = bfd_getb32 (src->free_list);
+  dst->file_end = bfd_getb32 (src->file_end);
+  dst->checksum = bfd_getb32 (src->checksum);
+}
+
 /* Perform some initialization for an object.  Save results of this
    initialization in the BFD.  */
 
-static const bfd_target *
+static bfd_cleanup
 som_object_setup (bfd *abfd,
-                 struct header *file_hdrp,
+                 struct som_header *file_hdrp,
                  struct som_exec_auxhdr *aux_hdrp,
                  unsigned long current_offset)
 {
@@ -1731,12 +2031,12 @@ som_object_setup (bfd *abfd,
          || (aux_hdrp->exec_entry & 0x3) != 0
          || ! found)
        {
-         bfd_get_start_address (abfd) = aux_hdrp->exec_flags;
+         abfd->start_address = aux_hdrp->exec_flags;
          obj_som_exec_data (abfd)->exec_flags = aux_hdrp->exec_entry;
        }
       else
        {
-         bfd_get_start_address (abfd) = aux_hdrp->exec_entry + current_offset;
+         abfd->start_address = aux_hdrp->exec_entry + current_offset;
          obj_som_exec_data (abfd)->exec_flags = aux_hdrp->exec_flags;
        }
     }
@@ -1744,7 +2044,7 @@ som_object_setup (bfd *abfd,
   obj_som_exec_data (abfd)->version_id = file_hdrp->version_id;
 
   bfd_default_set_arch_mach (abfd, bfd_arch_hppa, pa10);
-  bfd_get_symcount (abfd) = file_hdrp->symbol_total;
+  abfd->symcount = file_hdrp->symbol_total;
 
   /* Initialize the saved symbol table and string table to NULL.
      Save important offsets and sizes from the SOM header into
@@ -1760,7 +2060,7 @@ som_object_setup (bfd *abfd,
                                  + current_offset);
   obj_som_exec_data (abfd)->system_id = file_hdrp->system_id;
 
-  return abfd->xvec;
+  return _bfd_no_cleanup;
 }
 
 /* Convert all of the space and subspace info into BFD sections.  Each space
@@ -1771,32 +2071,39 @@ som_object_setup (bfd *abfd,
 
 static bfd_boolean
 setup_sections (bfd *abfd,
-               struct header *file_hdr,
+               struct som_header *file_hdr,
                unsigned long current_offset)
 {
-  char *space_strings;
+  char *space_strings = NULL;
   unsigned int space_index, i;
   unsigned int total_subspaces = 0;
   asection **subspace_sections = NULL;
   asection *section;
-  bfd_size_type amt;
+  size_t amt;
 
   /* First, read in space names.  */
   amt = file_hdr->space_strings_size;
-  space_strings = bfd_malloc (amt);
-  if (!space_strings && amt != 0)
-    goto error_return;
-
+  if (amt == (size_t) -1)
+    {
+      bfd_set_error (bfd_error_no_memory);
+      goto error_return;
+    }
   if (bfd_seek (abfd, current_offset + file_hdr->space_strings_location,
                SEEK_SET) != 0)
     goto error_return;
-  if (bfd_bread (space_strings, amt, abfd) != amt)
+  space_strings = (char *) _bfd_malloc_and_read (abfd, amt + 1, amt);
+  if (space_strings == NULL)
     goto error_return;
+  /* Make sure that the string table is NUL terminated.  */
+  space_strings[amt] = 0;
 
   /* Loop over all of the space dictionaries, building up sections.  */
   for (space_index = 0; space_index < file_hdr->space_total; space_index++)
     {
-      struct space_dictionary_record space;
+      struct som_space_dictionary_record space;
+      struct som_external_space_dictionary_record ext_space;
+      char *space_name;
+      struct som_external_subspace_dictionary_record ext_subspace;
       struct som_subspace_dictionary_record subspace, save_subspace;
       unsigned int subspace_index;
       asection *space_asect;
@@ -1806,22 +2113,27 @@ setup_sections (bfd *abfd,
       /* Read the space dictionary element.  */
       if (bfd_seek (abfd,
                    (current_offset + file_hdr->space_location
-                    + space_index * sizeof space),
+                    + space_index * sizeof (ext_space)),
                    SEEK_SET) != 0)
        goto error_return;
-      amt = sizeof space;
-      if (bfd_bread (&space, amt, abfd) != amt)
+      amt = sizeof ext_space;
+      if (bfd_bread (&ext_space, amt, abfd) != amt)
        goto error_return;
 
+      som_swap_space_dictionary_in (&ext_space, &space);
+
       /* Setup the space name string.  */
-      space.name.n_name = space.name.n_strx + space_strings;
+      if (space.name >= file_hdr->space_strings_size)
+       goto error_return;
+
+      space_name = space.name + space_strings;
 
       /* Make a section out of it.  */
-      amt = strlen (space.name.n_name) + 1;
+      amt = strlen (space_name) + 1;
       newname = bfd_alloc (abfd, amt);
       if (!newname)
        goto error_return;
-      strcpy (newname, space.name.n_name);
+      strcpy (newname, space_name);
 
       space_asect = bfd_make_section_anyway (abfd, newname);
       if (!space_asect)
@@ -1843,19 +2155,21 @@ setup_sections (bfd *abfd,
       /* Now, read in the first subspace for this space.  */
       if (bfd_seek (abfd,
                    (current_offset + file_hdr->subspace_location
-                    + space.subspace_index * sizeof subspace),
+                    + space.subspace_index * sizeof ext_subspace),
                    SEEK_SET) != 0)
        goto error_return;
-      amt = sizeof subspace;
-      if (bfd_bread (&subspace, amt, abfd) != amt)
+      amt = sizeof ext_subspace;
+      if (bfd_bread (&ext_subspace, amt, abfd) != amt)
        goto error_return;
       /* Seek back to the start of the subspaces for loop below.  */
       if (bfd_seek (abfd,
                    (current_offset + file_hdr->subspace_location
-                    + space.subspace_index * sizeof subspace),
+                    + space.subspace_index * sizeof ext_subspace),
                    SEEK_SET) != 0)
        goto error_return;
 
+      som_swap_subspace_dictionary_in (&ext_subspace, &subspace);
+
       /* Setup the start address and file loc from the first subspace
         record.  */
       space_asect->vma = subspace.subspace_start;
@@ -1873,20 +2187,26 @@ setup_sections (bfd *abfd,
           subspace_index++)
        {
          asection *subspace_asect;
+         char *subspace_name;
 
          /* Read in the next subspace.  */
-         amt = sizeof subspace;
-         if (bfd_bread (&subspace, amt, abfd) != amt)
+         amt = sizeof ext_subspace;
+         if (bfd_bread (&ext_subspace, amt, abfd) != amt)
            goto error_return;
 
+         som_swap_subspace_dictionary_in (&ext_subspace, &subspace);
+
          /* Setup the subspace name string.  */
-         subspace.name.n_name = subspace.name.n_strx + space_strings;
+         if (subspace.name >= file_hdr->space_strings_size)
+           goto error_return;
 
-         amt = strlen (subspace.name.n_name) + 1;
+         subspace_name = subspace.name + space_strings;
+
+         amt = strlen (subspace_name) + 1;
          newname = bfd_alloc (abfd, amt);
          if (!newname)
            goto error_return;
-         strcpy (newname, subspace.name.n_name);
+         strcpy (newname, subspace_name);
 
          /* Make a section out of this subspace.  */
          subspace_asect = bfd_make_section_anyway (abfd, newname);
@@ -2014,7 +2334,7 @@ setup_sections (bfd *abfd,
          else
            {
              /* The subspace_start field is not initialised in relocatable
-                only objects, so it cannot be used for length calculations.
+                only objects, so it cannot be used for length calculations.
                 Instead we use the space_size value which we have been
                 accumulating.  This isn't an accurate estimate since it
                 ignores alignment and ordering issues.  */
@@ -2024,8 +2344,11 @@ setup_sections (bfd *abfd,
     }
   /* Now that we've read in all the subspace records, we need to assign
      a target index to each subspace.  */
-  amt = total_subspaces;
-  amt *= sizeof (asection *);
+  if (_bfd_mul_overflow (total_subspaces, sizeof (asection *), &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      goto error_return;
+    }
   subspace_sections = bfd_malloc (amt);
   if (subspace_sections == NULL)
     goto error_return;
@@ -2046,44 +2369,42 @@ setup_sections (bfd *abfd,
   for (i = 0; i < total_subspaces; i++)
     subspace_sections[i]->target_index = i;
 
-  if (space_strings != NULL)
-    free (space_strings);
-
-  if (subspace_sections != NULL)
-    free (subspace_sections);
-
+  free (space_strings);
+  free (subspace_sections);
   return TRUE;
 
  error_return:
-  if (space_strings != NULL)
-    free (space_strings);
-
-  if (subspace_sections != NULL)
-    free (subspace_sections);
+  free (space_strings);
+  free (subspace_sections);
   return FALSE;
 }
 
+
 /* Read in a SOM object and make it into a BFD.  */
 
-static const bfd_target *
+static bfd_cleanup
 som_object_p (bfd *abfd)
 {
-  struct header file_hdr;
+  struct som_external_header ext_file_hdr;
+  struct som_header file_hdr;
   struct som_exec_auxhdr *aux_hdr_ptr = NULL;
   unsigned long current_offset = 0;
-  struct lst_header lst_header;
-  struct som_entry som_entry;
-  bfd_size_type amt;
-#define ENTRY_SIZE sizeof (struct som_entry)
-
-  amt = FILE_HDR_SIZE;
-  if (bfd_bread ((void *) &file_hdr, amt, abfd) != amt)
+  struct som_external_lst_header ext_lst_header;
+  struct som_external_som_entry ext_som_entry;
+  size_t amt;
+  unsigned int loc;
+#define ENTRY_SIZE sizeof (struct som_external_som_entry)
+
+  amt = sizeof (struct som_external_header);
+  if (bfd_bread (&ext_file_hdr, amt, abfd) != amt)
     {
       if (bfd_get_error () != bfd_error_system_call)
        bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
 
+  som_swap_header_in (&ext_file_hdr, &file_hdr);
+
   if (!_PA_RISC_ID (file_hdr.system_id))
     {
       bfd_set_error (bfd_error_wrong_format);
@@ -2096,18 +2417,13 @@ som_object_p (bfd *abfd)
     case EXEC_MAGIC:
     case SHARE_MAGIC:
     case DEMAND_MAGIC:
-#ifdef DL_MAGIC
     case DL_MAGIC:
-#endif
-#ifdef SHL_MAGIC
     case SHL_MAGIC:
-#endif
 #ifdef SHARED_MAGIC_CNX
     case SHARED_MAGIC_CNX:
 #endif
       break;
 
-#ifdef EXECLIBMAGIC
     case EXECLIBMAGIC:
       /* Read the lst header and determine where the SOM directory begins.  */
 
@@ -2118,8 +2434,8 @@ som_object_p (bfd *abfd)
          return NULL;
        }
 
-      amt = SLSTHDR;
-      if (bfd_bread ((void *) &lst_header, amt, abfd) != amt)
+      amt = sizeof (struct som_external_lst_header);
+      if (bfd_bread (&ext_lst_header, amt, abfd) != amt)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
@@ -2127,8 +2443,8 @@ som_object_p (bfd *abfd)
        }
 
       /* Position to and read the first directory entry.  */
-
-      if (bfd_seek (abfd, lst_header.dir_loc, SEEK_SET) != 0)
+      loc = bfd_getb32 (ext_lst_header.dir_loc);
+      if (bfd_seek (abfd, loc, SEEK_SET) != 0)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
@@ -2136,7 +2452,7 @@ som_object_p (bfd *abfd)
        }
 
       amt = ENTRY_SIZE;
-      if (bfd_bread ((void *) &som_entry, amt, abfd) != amt)
+      if (bfd_bread (&ext_som_entry, amt, abfd) != amt)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
@@ -2144,34 +2460,33 @@ som_object_p (bfd *abfd)
        }
 
       /* Now position to the first SOM.  */
-
-      if (bfd_seek (abfd, som_entry.location, SEEK_SET) != 0)
+      current_offset = bfd_getb32 (ext_som_entry.location);
+      if (bfd_seek (abfd, current_offset, SEEK_SET) != 0)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
          return NULL;
        }
 
-      current_offset = som_entry.location;
-
       /* And finally, re-read the som header.  */
-      amt = FILE_HDR_SIZE;
-      if (bfd_bread ((void *) &file_hdr, amt, abfd) != amt)
+      amt = sizeof (struct som_external_header);
+      if (bfd_bread (&ext_file_hdr, amt, abfd) != amt)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
          return NULL;
        }
 
+      som_swap_header_in (&ext_file_hdr, &file_hdr);
+
       break;
-#endif
 
     default:
       bfd_set_error (bfd_error_wrong_format);
       return NULL;
     }
 
-  if (file_hdr.version_id != VERSION_ID
+  if (file_hdr.version_id != OLD_VERSION_ID
       && file_hdr.version_id != NEW_VERSION_ID)
     {
       bfd_set_error (bfd_error_wrong_format);
@@ -2183,17 +2498,20 @@ som_object_p (bfd *abfd)
      a non-existant auxiliary header.  */
   if (file_hdr.aux_header_size != 0)
     {
-      aux_hdr_ptr = bfd_zalloc (abfd, 
+      struct som_external_exec_auxhdr ext_exec_auxhdr;
+
+      aux_hdr_ptr = bfd_zalloc (abfd,
                                (bfd_size_type) sizeof (*aux_hdr_ptr));
       if (aux_hdr_ptr == NULL)
        return NULL;
-      amt = AUX_HDR_SIZE;
-      if (bfd_bread ((void *) aux_hdr_ptr, amt, abfd) != amt)
+      amt = sizeof (struct som_external_exec_auxhdr);
+      if (bfd_bread (&ext_exec_auxhdr, amt, abfd) != amt)
        {
          if (bfd_get_error () != bfd_error_system_call)
            bfd_set_error (bfd_error_wrong_format);
          return NULL;
        }
+      som_swap_exec_auxhdr_in (&ext_exec_auxhdr, aux_hdr_ptr);
     }
 
   if (!setup_sections (abfd, &file_hdr, current_offset))
@@ -2226,9 +2544,9 @@ som_mkobject (bfd *abfd)
 static bfd_boolean
 som_prep_headers (bfd *abfd)
 {
-  struct header *file_hdr;
+  struct som_header *file_hdr;
   asection *section;
-  bfd_size_type amt = sizeof (struct header);
+  size_t amt = sizeof (struct som_header);
 
   /* Make and attach a file header to the BFD.  */
   file_hdr = bfd_zalloc (abfd, amt);
@@ -2281,7 +2599,7 @@ som_prep_headers (bfd *abfd)
       if (som_is_space (section))
        {
          /* Allocate space for the space dictionary.  */
-         amt = sizeof (struct space_dictionary_record);
+         amt = sizeof (struct som_space_dictionary_record);
          som_section_data (section)->space_dict = bfd_zalloc (abfd, amt);
          if (som_section_data (section)->space_dict == NULL)
            return FALSE;
@@ -2478,13 +2796,16 @@ compare_subspaces (const void *arg1, const void *arg2)
 
 /* Perform various work in preparation for emitting the fixup stream.  */
 
-static void
+static bfd_boolean
 som_prep_for_fixups (bfd *abfd, asymbol **syms, unsigned long num_syms)
 {
   unsigned long i;
   asection *section;
   asymbol **sorted_syms;
-  bfd_size_type amt;
+  size_t amt;
+
+  if (num_syms == 0)
+    return TRUE;
 
   /* Most SOM relocations involving a symbol have a length which is
      dependent on the index of the symbol.  So symbols which are
@@ -2556,9 +2877,14 @@ som_prep_for_fixups (bfd *abfd, asymbol **syms, unsigned long num_syms)
 
   /* Sort a copy of the symbol table, rather than the canonical
      output symbol table.  */
-  amt = num_syms;
-  amt *= sizeof (asymbol *);
+  if (_bfd_mul_overflow (num_syms, sizeof (asymbol *), &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return FALSE;
+    }
   sorted_syms = bfd_zalloc (abfd, amt);
+  if (sorted_syms == NULL)
+    return FALSE;
   memcpy (sorted_syms, syms, num_syms * sizeof (asymbol *));
   qsort (sorted_syms, num_syms, sizeof (asymbol *), compare_syms);
   obj_som_sorted_syms (abfd) = sorted_syms;
@@ -2574,6 +2900,7 @@ som_prep_for_fixups (bfd *abfd, asymbol **syms, unsigned long num_syms)
       else
        som_symbol_data (sorted_syms[i])->index = i;
     }
+  return TRUE;
 }
 
 static bfd_boolean
@@ -2590,7 +2917,7 @@ som_write_fixups (bfd *abfd,
   unsigned int subspace_reloc_size = 0;
   unsigned int num_spaces = obj_som_file_hdr (abfd)->space_total;
   asection *section = abfd->sections;
-  bfd_size_type amt;
+  size_t amt;
 
   memset (tmp_space, 0, SOM_TMP_BUFSIZE);
   p = tmp_space;
@@ -2774,6 +3101,24 @@ som_write_fixups (bfd *abfd,
                    abort ();
                  break;
 
+               case R_DATA_GPREL:
+                 /* Account for any addend.  */
+                 if (bfd_reloc->addend)
+                   p = som_reloc_addend (abfd, bfd_reloc->addend, p,
+                                         &subspace_reloc_size, reloc_queue);
+
+                 if (sym_num < 0x10000000)
+                   {
+                     bfd_put_8 (abfd, bfd_reloc->howto->type, p);
+                     bfd_put_8 (abfd, sym_num >> 16, p + 1);
+                     bfd_put_16 (abfd, (bfd_vma) sym_num, p + 2);
+                     p = try_prev_fixup (abfd, &subspace_reloc_size,
+                                         p, 4, reloc_queue);
+                   }
+                 else
+                   abort ();
+                 break;
+
                case R_DATA_ONE_SYMBOL:
                case R_DATA_PLABEL:
                case R_CODE_PLABEL:
@@ -2974,11 +3319,15 @@ som_write_space_strings (bfd *abfd,
   /* Chunk of memory that we can use as buffer space, then throw
      away.  */
   size_t tmp_space_size = SOM_TMP_BUFSIZE;
-  char *tmp_space = alloca (tmp_space_size);
+  char *tmp_space = bfd_malloc (tmp_space_size);
   char *p = tmp_space;
   unsigned int strings_size = 0;
   asection *section;
-  bfd_size_type amt;
+  size_t amt;
+  bfd_size_type res;
+
+  if (tmp_space == NULL)
+    return FALSE;
 
   /* Seek to the start of the space strings in preparation for writing
      them out.  */
@@ -3021,8 +3370,11 @@ som_write_space_strings (bfd *abfd,
                 returning from this function.)  The same technique is
                 used a few more times below when a buffer is
                 reallocated.  */
-             tmp_space_size = MAX (2 * tmp_space_size, 5 + length);
-             tmp_space = alloca (tmp_space_size);
+             if (2 * tmp_space_size < length + 5)
+               tmp_space_size = length + 5;
+             else
+               tmp_space_size = 2 * tmp_space_size;
+             tmp_space = xrealloc (tmp_space, tmp_space_size);
            }
 
          /* Reset to beginning of the (possibly new) buffer space.  */
@@ -3037,9 +3389,9 @@ som_write_space_strings (bfd *abfd,
 
       /* Record the index in the space/subspace records.  */
       if (som_is_space (section))
-       som_section_data (section)->space_dict->name.n_strx = strings_size;
+       som_section_data (section)->space_dict->name = strings_size;
       else
-       som_section_data (section)->subspace_dict->name.n_strx = strings_size;
+       som_section_data (section)->subspace_dict->name = strings_size;
 
       /* Next comes the string itself + a null terminator.  */
       strcpy (p, section->name);
@@ -3058,7 +3410,9 @@ som_write_space_strings (bfd *abfd,
   /* Done with the space/subspace strings.  Write out any information
      contained in a partial block.  */
   amt = p - tmp_space;
-  if (bfd_bwrite ((void *) &tmp_space[0], amt, abfd) != amt)
+  res = bfd_bwrite ((void *) &tmp_space[0], amt, abfd);
+  free (tmp_space);
+  if (res != amt)
     return FALSE;
   *string_sizep = strings_size;
   return TRUE;
@@ -3072,19 +3426,20 @@ som_write_symbol_strings (bfd *abfd,
                          asymbol **syms,
                          unsigned int num_syms,
                          unsigned int *string_sizep,
-                         COMPUNIT *compilation_unit)
+                         struct som_compilation_unit *compilation_unit)
 {
   unsigned int i;
-
   /* Chunk of memory that we can use as buffer space, then throw
      away.  */
   size_t tmp_space_size = SOM_TMP_BUFSIZE;
-  char *tmp_space = alloca (tmp_space_size);
+  char *tmp_space = bfd_malloc (tmp_space_size);
   char *p = tmp_space;
-
   unsigned int strings_size = 0;
-  char *comp[4];
-  bfd_size_type amt;
+  size_t amt;
+  bfd_size_type res;
+
+  if (tmp_space == NULL)
+    return FALSE;
 
   /* This gets a bit gruesome because of the compilation unit.  The
      strings within the compilation unit are part of the symbol
@@ -3092,13 +3447,6 @@ som_write_symbol_strings (bfd *abfd,
      write them and update the compilation unit header.  On input, the
      compilation unit header contains local copies of the strings.
      Move them aside.  */
-  if (compilation_unit)
-    {
-      comp[0] = compilation_unit->name.n_name;
-      comp[1] = compilation_unit->language_name.n_name;
-      comp[2] = compilation_unit->product_id.n_name;
-      comp[3] = compilation_unit->version_id.n_name;
-    }
 
   /* Seek to the start of the space strings in preparation for writing
      them out.  */
@@ -3109,7 +3457,28 @@ som_write_symbol_strings (bfd *abfd,
     {
       for (i = 0; i < 4; i++)
        {
-         size_t length = strlen (comp[i]);
+         struct som_name_pt *name;
+         size_t length;
+
+         switch (i)
+           {
+           case 0:
+             name = &compilation_unit->name;
+             break;
+           case 1:
+             name = &compilation_unit->language_name;
+             break;
+           case 2:
+             name = &compilation_unit->product_id;
+             break;
+           case 3:
+             name = &compilation_unit->version_id;
+             break;
+           default:
+             abort ();
+           }
+
+         length = strlen (name->name);
 
          /* If there is not enough room for the next entry, then dump
             the current buffer contents now and maybe allocate a
@@ -3125,8 +3494,11 @@ som_write_symbol_strings (bfd *abfd,
              if (5 + length > tmp_space_size)
                {
                  /* See alloca above for discussion of new size.  */
-                 tmp_space_size = MAX (2 * tmp_space_size, 5 + length);
-                 tmp_space = alloca (tmp_space_size);
+                 if (2 * tmp_space_size < 5 + length)
+                   tmp_space_size = 5 + length;
+                 else
+                   tmp_space_size = 2 * tmp_space_size;
+                 tmp_space = xrealloc (tmp_space, tmp_space_size);
                }
 
              /* Reset to beginning of the (possibly new) buffer
@@ -3143,26 +3515,9 @@ som_write_symbol_strings (bfd *abfd,
          p += 4;
 
          /* Next comes the string itself + a null terminator.  */
-         strcpy (p, comp[i]);
+         strcpy (p, name->name);
 
-         switch (i)
-           {
-           case 0:
-             obj_som_compilation_unit (abfd)->name.n_strx = strings_size;
-             break;
-           case 1:
-             obj_som_compilation_unit (abfd)->language_name.n_strx =
-               strings_size;
-             break;
-           case 2:
-             obj_som_compilation_unit (abfd)->product_id.n_strx =
-               strings_size;
-             break;
-           case 3:
-             obj_som_compilation_unit (abfd)->version_id.n_strx =
-               strings_size;
-             break;
-           }
+         name->strx = strings_size;
 
          p += length + 1;
          strings_size += length + 1;
@@ -3194,8 +3549,11 @@ som_write_symbol_strings (bfd *abfd,
          if (5 + length > tmp_space_size)
            {
              /* See alloca above for discussion of new size.  */
-             tmp_space_size = MAX (2 * tmp_space_size, 5 + length);
-             tmp_space = alloca (tmp_space_size);
+             if (2 * tmp_space_size < 5 + length)
+               tmp_space_size = 5 + length;
+             else
+               tmp_space_size = 2 * tmp_space_size;
+             tmp_space = xrealloc (tmp_space, tmp_space_size);
            }
 
          /* Reset to beginning of the (possibly new) buffer space.  */
@@ -3228,7 +3586,9 @@ som_write_symbol_strings (bfd *abfd,
 
   /* Scribble out any partial block.  */
   amt = p - tmp_space;
-  if (bfd_bwrite ((void *) &tmp_space[0], amt, abfd) != amt)
+  res = bfd_bwrite ((void *) &tmp_space[0], amt, abfd);
+  free (tmp_space);
+  if (res != amt)
     return FALSE;
 
   *string_sizep = strings_size;
@@ -3261,7 +3621,7 @@ som_begin_writing (bfd *abfd)
 
   /* Make room for the file header, it's contents are not complete
      yet, so it can not be written at this time.  */
-  current_offset += sizeof (struct header);
+  current_offset += sizeof (struct som_external_header);
 
   /* Any auxiliary headers will follow the file header.  Right now
      we support only the copyright and version headers.  */
@@ -3272,55 +3632,61 @@ som_begin_writing (bfd *abfd)
       /* Parts of the exec header will be filled in later, so
         delay writing the header itself.  Fill in the defaults,
         and write it later.  */
-      current_offset += sizeof (struct som_exec_auxhdr);
+      current_offset += sizeof (struct som_external_exec_auxhdr);
       obj_som_file_hdr (abfd)->aux_header_size
-       += sizeof (struct som_exec_auxhdr);
+       += sizeof (struct som_external_exec_auxhdr);
       exec_header = obj_som_exec_hdr (abfd);
       exec_header->som_auxhdr.type = EXEC_AUX_ID;
       exec_header->som_auxhdr.length = 40;
     }
   if (obj_som_version_hdr (abfd) != NULL)
     {
+      struct som_external_string_auxhdr ext_string_auxhdr;
       bfd_size_type len;
 
       if (bfd_seek (abfd, (file_ptr) current_offset, SEEK_SET) != 0)
        return FALSE;
 
       /* Write the aux_id structure and the string length.  */
-      len = sizeof (struct aux_id) + sizeof (unsigned int);
+      len = sizeof (struct som_external_string_auxhdr);
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
-      if (bfd_bwrite ((void *) obj_som_version_hdr (abfd), len, abfd) != len)
+      som_swap_string_auxhdr_out
+       (obj_som_version_hdr (abfd), &ext_string_auxhdr);
+      if (bfd_bwrite (&ext_string_auxhdr, len, abfd) != len)
        return FALSE;
 
       /* Write the version string.  */
-      len = obj_som_version_hdr (abfd)->header_id.length - sizeof (int);
+      len = obj_som_version_hdr (abfd)->header_id.length - 4;
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
-      if (bfd_bwrite ((void *) obj_som_version_hdr (abfd)->user_string, len, abfd)
+      if (bfd_bwrite ((void *) obj_som_version_hdr (abfd)->string, len, abfd)
          != len)
        return FALSE;
     }
 
   if (obj_som_copyright_hdr (abfd) != NULL)
     {
+      struct som_external_string_auxhdr ext_string_auxhdr;
       bfd_size_type len;
 
       if (bfd_seek (abfd, (file_ptr) current_offset, SEEK_SET) != 0)
        return FALSE;
 
       /* Write the aux_id structure and the string length.  */
-      len = sizeof (struct aux_id) + sizeof (unsigned int);
+      len = sizeof (struct som_external_string_auxhdr);
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
-      if (bfd_bwrite ((void *) obj_som_copyright_hdr (abfd), len, abfd) != len)
+      som_swap_string_auxhdr_out
+       (obj_som_copyright_hdr (abfd), &ext_string_auxhdr);
+      if (bfd_bwrite (&ext_string_auxhdr, len, abfd) != len)
        return FALSE;
 
       /* Write the copyright string.  */
-      len = obj_som_copyright_hdr (abfd)->header_id.length - sizeof (int);
+      len = obj_som_copyright_hdr (abfd)->header_id.length - 4;
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
-      if (bfd_bwrite ((void *) obj_som_copyright_hdr (abfd)->copyright, len, abfd)
+      if (bfd_bwrite ((void *) obj_som_copyright_hdr (abfd)->string, len, abfd)
          != len)
        return FALSE;
     }
@@ -3342,7 +3708,8 @@ som_begin_writing (bfd *abfd)
   num_spaces = som_count_spaces (abfd);
   obj_som_file_hdr (abfd)->space_location = current_offset;
   obj_som_file_hdr (abfd)->space_total = num_spaces;
-  current_offset += num_spaces * sizeof (struct space_dictionary_record);
+  current_offset +=
+    num_spaces * sizeof (struct som_external_space_dictionary_record);
 
   /* Next are the subspace records.  These are fixed length records.
 
@@ -3355,7 +3722,7 @@ som_begin_writing (bfd *abfd)
   obj_som_file_hdr (abfd)->subspace_location = current_offset;
   obj_som_file_hdr (abfd)->subspace_total = num_subspaces;
   current_offset
-    += num_subspaces * sizeof (struct som_subspace_dictionary_record);
+    += num_subspaces * sizeof (struct som_external_subspace_dictionary_record);
 
   /* Next is the string table for the space/subspace names.  We will
      build and write the string table on the fly.  At the same time
@@ -3384,7 +3751,7 @@ som_begin_writing (bfd *abfd)
   if (obj_som_compilation_unit (abfd))
     {
       obj_som_file_hdr (abfd)->compiler_total = 1;
-      current_offset += COMPUNITSZ;
+      current_offset += sizeof (struct som_external_compilation_unit);
     }
 
   /* Now compute the file positions for the loadable subspaces, taking
@@ -3598,7 +3965,8 @@ som_finish_writing (bfd *abfd)
   asection *section;
   unsigned long current_offset;
   unsigned int strings_size, total_reloc_size;
-  bfd_size_type amt;
+  size_t amt;
+  struct som_external_header ext_header;
 
   /* We must set up the version identifier here as objcopy/strip copy
      private BFD data too late for us to handle this in som_begin_writing.  */
@@ -3630,7 +3998,8 @@ som_finish_writing (bfd *abfd)
   num_syms = bfd_get_symcount (abfd);
   obj_som_file_hdr (abfd)->symbol_location = current_offset;
   obj_som_file_hdr (abfd)->symbol_total = num_syms;
-  current_offset += num_syms * sizeof (struct symbol_dictionary_record);
+  current_offset +=
+    num_syms * sizeof (struct som_external_symbol_dictionary_record);
 
   /* Next are the symbol strings.
      Align them to a word boundary.  */
@@ -3650,9 +4019,10 @@ som_finish_writing (bfd *abfd)
   current_offset += strings_size;
 
   /* Do prep work before handling fixups.  */
-  som_prep_for_fixups (abfd,
-                      bfd_get_outsymbols (abfd),
-                      bfd_get_symcount (abfd));
+  if (!som_prep_for_fixups (abfd,
+                           bfd_get_outsymbols (abfd),
+                           bfd_get_symcount (abfd)))
+    return FALSE;
 
   /* At the end of the file is the fixup stream which starts on a
      word boundary.  */
@@ -3699,6 +4069,7 @@ som_finish_writing (bfd *abfd)
           subsection != NULL;
           subsection = subsection->next)
        {
+         struct som_external_subspace_dictionary_record ext_subspace_dict;
 
          /* Skip any section which does not correspond to a space
             or subspace.  Or does not have SEC_ALLOC set (and therefore
@@ -3729,9 +4100,10 @@ som_finish_writing (bfd *abfd)
          som_section_data (subsection)->subspace_dict->space_index = i;
 
          /* Dump the current subspace header.  */
+         som_swap_subspace_dictionary_record_out
+           (som_section_data (subsection)->subspace_dict, &ext_subspace_dict);
          amt = sizeof (struct som_subspace_dictionary_record);
-         if (bfd_bwrite ((void *) som_section_data (subsection)->subspace_dict,
-                        amt, abfd) != amt)
+         if (bfd_bwrite (&ext_subspace_dict, amt, abfd) != amt)
            return FALSE;
        }
       /* Goto the next section.  */
@@ -3754,6 +4126,7 @@ som_finish_writing (bfd *abfd)
           subsection != NULL;
           subsection = subsection->next)
        {
+         struct som_external_subspace_dictionary_record ext_subspace_dict;
 
          /* Skip any section which does not correspond to a space or
             subspace, or which SEC_ALLOC set (and therefore handled
@@ -3785,9 +4158,10 @@ som_finish_writing (bfd *abfd)
          som_section_data (subsection)->subspace_dict->space_index = i;
 
          /* Dump this subspace header.  */
+         som_swap_subspace_dictionary_record_out
+           (som_section_data (subsection)->subspace_dict, &ext_subspace_dict);
          amt = sizeof (struct som_subspace_dictionary_record);
-         if (bfd_bwrite ((void *) som_section_data (subsection)->subspace_dict,
-                        amt, abfd) != amt)
+         if (bfd_bwrite (&ext_subspace_dict, amt, abfd) != amt)
            return FALSE;
        }
       /* Goto the next section.  */
@@ -3806,14 +4180,17 @@ som_finish_writing (bfd *abfd)
   section = abfd->sections;
   for (i = 0; i < num_spaces; i++)
     {
+      struct som_external_space_dictionary_record ext_space_dict;
+
       /* Find a space.  */
       while (!som_is_space (section))
        section = section->next;
 
       /* Dump its header.  */
-      amt = sizeof (struct space_dictionary_record);
-      if (bfd_bwrite ((void *) som_section_data (section)->space_dict,
-                    amt, abfd) != amt)
+      som_swap_space_dictionary_out (som_section_data (section)->space_dict,
+                                    &ext_space_dict);
+      amt = sizeof (struct som_external_space_dictionary_record);
+      if (bfd_bwrite (&ext_space_dict, amt, abfd) != amt)
        return FALSE;
 
       /* Goto the next section.  */
@@ -3823,12 +4200,17 @@ som_finish_writing (bfd *abfd)
   /* Write the compilation unit record if there is one.  */
   if (obj_som_compilation_unit (abfd))
     {
+      struct som_external_compilation_unit ext_comp_unit;
+
       location = obj_som_file_hdr (abfd)->compiler_location;
       if (bfd_seek (abfd, location, SEEK_SET) != 0)
        return FALSE;
 
-      amt = COMPUNITSZ;
-      if (bfd_bwrite ((void *) obj_som_compilation_unit (abfd), amt, abfd) != amt)
+      som_swap_compilation_unit_out
+       (obj_som_compilation_unit (abfd), &ext_comp_unit);
+
+      amt = sizeof (struct som_external_compilation_unit);
+      if (bfd_bwrite (&ext_comp_unit, amt, abfd) != amt)
        return FALSE;
     }
 
@@ -3843,16 +4225,17 @@ som_finish_writing (bfd *abfd)
   else
     obj_som_file_hdr (abfd)->system_id = CPU_PA_RISC1_0;
 
-  /* Compute the checksum for the file header just before writing
+  /* Swap and compute the checksum for the file header just before writing
      the header to disk.  */
-  obj_som_file_hdr (abfd)->checksum = som_compute_checksum (abfd);
+  som_swap_header_out (obj_som_file_hdr (abfd), &ext_header);
+  bfd_putb32 (som_compute_checksum (&ext_header), ext_header.checksum);
 
   /* Only thing left to do is write out the file header.  It is always
      at location zero.  Seek there and write it.  */
   if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0)
     return FALSE;
-  amt = sizeof (struct header);
-  if (bfd_bwrite ((void *) obj_som_file_hdr (abfd), amt, abfd) != amt)
+  amt = sizeof (struct som_external_header);
+  if (bfd_bwrite (&ext_header, amt, abfd) != amt)
     return FALSE;
 
   /* Now write the exec header.  */
@@ -3860,6 +4243,7 @@ som_finish_writing (bfd *abfd)
     {
       long tmp, som_length;
       struct som_exec_auxhdr *exec_header;
+      struct som_external_exec_auxhdr ext_exec_header;
 
       exec_header = obj_som_exec_hdr (abfd);
       exec_header->exec_entry = bfd_get_start_address (abfd);
@@ -3885,12 +4269,14 @@ som_finish_writing (bfd *abfd)
          return FALSE;
        }
 
+      som_swap_exec_auxhdr_out (exec_header, &ext_exec_header);
+
       if (bfd_seek (abfd, obj_som_file_hdr (abfd)->aux_header_location,
                    SEEK_SET) != 0)
        return FALSE;
 
-      amt = AUX_HDR_SIZE;
-      if (bfd_bwrite ((void *) exec_header, amt, abfd) != amt)
+      amt = sizeof (ext_exec_header);
+      if (bfd_bwrite (&ext_exec_header, amt, abfd) != amt)
        return FALSE;
     }
   return TRUE;
@@ -3898,14 +4284,15 @@ som_finish_writing (bfd *abfd)
 
 /* Compute and return the checksum for a SOM file header.  */
 
-static unsigned long
-som_compute_checksum (bfd *abfd)
+static uint32_t
+som_compute_checksum (struct som_external_header *hdr)
 {
-  unsigned long checksum, count, i;
-  unsigned long *buffer = (unsigned long *) obj_som_file_hdr (abfd);
+  size_t count, i;
+  uint32_t checksum;
+  uint32_t *buffer = (uint32_t *) hdr;
 
   checksum = 0;
-  count = sizeof (struct header) / sizeof (unsigned long);
+  count = sizeof (*hdr) / sizeof (*buffer);
   for (i = 0; i < count; i++)
     checksum ^= *(buffer + i);
 
@@ -3969,7 +4356,9 @@ som_bfd_derive_misc_symbol_info (bfd *abfd ATTRIBUTE_UNUSED,
         section (ST_DATA for DATA sections, ST_CODE for CODE sections).  */
       else if (som_symbol_data (sym)->som_type == SYMBOL_TYPE_UNKNOWN)
        {
-         if (sym->section->flags & SEC_CODE)
+         if (bfd_is_abs_section (sym->section))
+           info->symbol_type = ST_ABSOLUTE;
+         else if (sym->section->flags & SEC_CODE)
            info->symbol_type = ST_CODE;
          else
            info->symbol_type = ST_DATA;
@@ -4076,58 +4465,71 @@ som_build_and_write_symbol_table (bfd *abfd)
   unsigned int num_syms = bfd_get_symcount (abfd);
   file_ptr symtab_location = obj_som_file_hdr (abfd)->symbol_location;
   asymbol **bfd_syms = obj_som_sorted_syms (abfd);
-  struct symbol_dictionary_record *som_symtab = NULL;
+  struct som_external_symbol_dictionary_record *som_symtab = NULL;
   unsigned int i;
   bfd_size_type symtab_size;
+  size_t amt;
 
   /* Compute total symbol table size and allocate a chunk of memory
      to hold the symbol table as we build it.  */
-  symtab_size = num_syms;
-  symtab_size *= sizeof (struct symbol_dictionary_record);
-  som_symtab = bfd_zmalloc (symtab_size);
-  if (som_symtab == NULL && symtab_size != 0)
+  if (_bfd_mul_overflow (num_syms,
+                        sizeof (struct som_external_symbol_dictionary_record),
+                        &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return FALSE;
+    }
+  som_symtab = bfd_zmalloc (amt);
+  if (som_symtab == NULL && num_syms != 0)
     goto error_return;
 
   /* Walk over each symbol.  */
   for (i = 0; i < num_syms; i++)
     {
       struct som_misc_symbol_info info;
+      unsigned int flags;
 
       /* This is really an index into the symbol strings table.
         By the time we get here, the index has already been
         computed and stored into the name field in the BFD symbol.  */
-      som_symtab[i].name.n_strx = som_symbol_data(bfd_syms[i])->stringtab_offset;
+      bfd_putb32 (som_symbol_data (bfd_syms[i])->stringtab_offset,
+                 som_symtab[i].name);
 
       /* Derive SOM information from the BFD symbol.  */
       som_bfd_derive_misc_symbol_info (abfd, bfd_syms[i], &info);
 
       /* Now use it.  */
-      som_symtab[i].symbol_type = info.symbol_type;
-      som_symtab[i].symbol_scope = info.symbol_scope;
-      som_symtab[i].arg_reloc = info.arg_reloc;
-      som_symtab[i].symbol_info = info.symbol_info;
-      som_symtab[i].xleast = 3;
-      som_symtab[i].symbol_value = info.symbol_value | info.priv_level;
-      som_symtab[i].secondary_def = info.secondary_def;
-      som_symtab[i].is_comdat = info.is_comdat;
-      som_symtab[i].is_common = info.is_common;
-      som_symtab[i].dup_common = info.dup_common;
+      flags = (info.symbol_type << SOM_SYMBOL_TYPE_SH)
+       | (info.symbol_scope << SOM_SYMBOL_SCOPE_SH)
+       | (info.arg_reloc << SOM_SYMBOL_ARG_RELOC_SH)
+       | (3 << SOM_SYMBOL_XLEAST_SH)
+       | (info.secondary_def ? SOM_SYMBOL_SECONDARY_DEF : 0)
+       | (info.is_common ? SOM_SYMBOL_IS_COMMON : 0)
+       | (info.dup_common ? SOM_SYMBOL_DUP_COMMON : 0);
+      bfd_putb32 (flags, som_symtab[i].flags);
+
+      flags = (info.symbol_info << SOM_SYMBOL_SYMBOL_INFO_SH)
+       | (info.is_comdat ? SOM_SYMBOL_IS_COMDAT : 0);
+      bfd_putb32 (flags, som_symtab[i].info);
+      bfd_putb32 (info.symbol_value | info.priv_level,
+                 som_symtab[i].symbol_value);
     }
 
   /* Everything is ready, seek to the right location and
      scribble out the symbol table.  */
   if (bfd_seek (abfd, symtab_location, SEEK_SET) != 0)
-    return FALSE;
+    goto error_return;
 
+  symtab_size = num_syms;
+  symtab_size *= sizeof (struct som_external_symbol_dictionary_record);
   if (bfd_bwrite ((void *) som_symtab, symtab_size, abfd) != symtab_size)
     goto error_return;
 
-  if (som_symtab != NULL)
-    free (som_symtab);
+  free (som_symtab);
   return TRUE;
+
  error_return:
-  if (som_symtab != NULL)
-    free (som_symtab);
+  free (som_symtab);
   return FALSE;
 }
 
@@ -4172,15 +4574,11 @@ som_slurp_string_table (bfd *abfd)
     }
 
   /* Allocate and read in the string table.  */
-  amt = obj_som_stringtab_size (abfd);
-  stringtab = bfd_zmalloc (amt);
-  if (stringtab == NULL)
-    return FALSE;
-
   if (bfd_seek (abfd, obj_som_str_filepos (abfd), SEEK_SET) != 0)
     return FALSE;
-
-  if (bfd_bread (stringtab, amt, abfd) != amt)
+  amt = obj_som_stringtab_size (abfd);
+  stringtab = (char *) _bfd_malloc_and_read (abfd, amt, amt);
+  if (stringtab == NULL)
     return FALSE;
 
   /* Save our results and return success.  */
@@ -4202,29 +4600,33 @@ som_get_symtab_upper_bound (bfd *abfd)
 
 /* Convert from a SOM subspace index to a BFD section.  */
 
-static asection *
-bfd_section_from_som_symbol (bfd *abfd, struct symbol_dictionary_record *symbol)
+asection *
+bfd_section_from_som_symbol
+  (bfd *abfd, struct som_external_symbol_dictionary_record *symbol)
 {
   asection *section;
+  unsigned int flags = bfd_getb32 (symbol->flags);
+  unsigned int symbol_type = (flags >> SOM_SYMBOL_TYPE_SH) & SOM_SYMBOL_TYPE_MASK;
 
   /* The meaning of the symbol_info field changes for functions
      within executables.  So only use the quick symbol_info mapping for
      incomplete objects and non-function symbols in executables.  */
   if ((abfd->flags & (EXEC_P | DYNAMIC)) == 0
-      || (symbol->symbol_type != ST_ENTRY
-         && symbol->symbol_type != ST_PRI_PROG
-         && symbol->symbol_type != ST_SEC_PROG
-         && symbol->symbol_type != ST_MILLICODE))
+      || (symbol_type != ST_ENTRY
+         && symbol_type != ST_PRI_PROG
+         && symbol_type != ST_SEC_PROG
+         && symbol_type != ST_MILLICODE))
     {
-      int index = symbol->symbol_info;
+      int idx = (bfd_getb32 (symbol->info) >> SOM_SYMBOL_SYMBOL_INFO_SH)
+       & SOM_SYMBOL_SYMBOL_INFO_MASK;
 
       for (section = abfd->sections; section != NULL; section = section->next)
-       if (section->target_index == index && som_is_subspace (section))
+       if (section->target_index == idx && som_is_subspace (section))
          return section;
     }
   else
     {
-      unsigned int value = symbol->symbol_value;
+      unsigned int value = bfd_getb32 (symbol->symbol_value);
 
       /* For executables we will have to use the symbol's address and
         find out what section would contain that address.   Yuk.  */
@@ -4245,12 +4647,12 @@ bfd_section_from_som_symbol (bfd *abfd, struct symbol_dictionary_record *symbol)
 static unsigned int
 som_slurp_symbol_table (bfd *abfd)
 {
-  int symbol_count = bfd_get_symcount (abfd);
-  int symsize = sizeof (struct symbol_dictionary_record);
+  unsigned int symbol_count = bfd_get_symcount (abfd);
+  size_t symsize = sizeof (struct som_external_symbol_dictionary_record);
   char *stringtab;
-  struct symbol_dictionary_record *buf = NULL, *bufp, *endbufp;
-  som_symbol_type *sym, *symbase;
-  bfd_size_type amt;
+  struct som_external_symbol_dictionary_record *buf = NULL, *bufp, *endbufp;
+  som_symbol_type *sym, *symbase = NULL;
+  size_t amt;
 
   /* Return saved value if it exists.  */
   if (obj_som_symtab (abfd) != NULL)
@@ -4265,63 +4667,74 @@ som_slurp_symbol_table (bfd *abfd)
 
   stringtab = obj_som_stringtab (abfd);
 
-  amt = symbol_count;
-  amt *= sizeof (som_symbol_type);
-  symbase = bfd_zmalloc (amt);
-  if (symbase == NULL)
-    goto error_return;
-
   /* Read in the external SOM representation.  */
-  amt = symbol_count;
-  amt *= symsize;
-  buf = bfd_malloc (amt);
-  if (buf == NULL && amt != 0)
-    goto error_return;
+  if (_bfd_mul_overflow (symbol_count, symsize, &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      goto error_return;
+    }
   if (bfd_seek (abfd, obj_som_sym_filepos (abfd), SEEK_SET) != 0)
     goto error_return;
-  if (bfd_bread (buf, amt, abfd) != amt)
+  buf = (struct som_external_symbol_dictionary_record *)
+    _bfd_malloc_and_read (abfd, amt, amt);
+  if (buf == NULL)
+    goto error_return;
+
+  if (_bfd_mul_overflow (symbol_count, sizeof (som_symbol_type), &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      goto error_return;
+    }
+  symbase = bfd_zmalloc (amt);
+  if (symbase == NULL)
     goto error_return;
 
   /* Iterate over all the symbols and internalize them.  */
   endbufp = buf + symbol_count;
   for (bufp = buf, sym = symbase; bufp < endbufp; ++bufp)
     {
+      unsigned int flags = bfd_getb32 (bufp->flags);
+      unsigned int symbol_type =
+       (flags >> SOM_SYMBOL_TYPE_SH) & SOM_SYMBOL_TYPE_MASK;
+      unsigned int symbol_scope =
+       (flags >> SOM_SYMBOL_SCOPE_SH) & SOM_SYMBOL_SCOPE_MASK;
+
       /* I don't think we care about these.  */
-      if (bufp->symbol_type == ST_SYM_EXT
-         || bufp->symbol_type == ST_ARG_EXT)
+      if (symbol_type == ST_SYM_EXT || symbol_type == ST_ARG_EXT)
        continue;
 
       /* Set some private data we care about.  */
-      if (bufp->symbol_type == ST_NULL)
+      if (symbol_type == ST_NULL)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_UNKNOWN;
-      else if (bufp->symbol_type == ST_ABSOLUTE)
+      else if (symbol_type == ST_ABSOLUTE)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_ABSOLUTE;
-      else if (bufp->symbol_type == ST_DATA)
+      else if (symbol_type == ST_DATA)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_DATA;
-      else if (bufp->symbol_type == ST_CODE)
+      else if (symbol_type == ST_CODE)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_CODE;
-      else if (bufp->symbol_type == ST_PRI_PROG)
+      else if (symbol_type == ST_PRI_PROG)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_PRI_PROG;
-      else if (bufp->symbol_type == ST_SEC_PROG)
+      else if (symbol_type == ST_SEC_PROG)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_SEC_PROG;
-      else if (bufp->symbol_type == ST_ENTRY)
+      else if (symbol_type == ST_ENTRY)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_ENTRY;
-      else if (bufp->symbol_type == ST_MILLICODE)
+      else if (symbol_type == ST_MILLICODE)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_MILLICODE;
-      else if (bufp->symbol_type == ST_PLABEL)
+      else if (symbol_type == ST_PLABEL)
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_PLABEL;
       else
        som_symbol_data (sym)->som_type = SYMBOL_TYPE_UNKNOWN;
-      som_symbol_data (sym)->tc_data.ap.hppa_arg_reloc = bufp->arg_reloc;
+      som_symbol_data (sym)->tc_data.ap.hppa_arg_reloc =
+       (flags >> SOM_SYMBOL_ARG_RELOC_SH) & SOM_SYMBOL_ARG_RELOC_MASK;
 
       /* Some reasonable defaults.  */
       sym->symbol.the_bfd = abfd;
-      sym->symbol.name = bufp->name.n_strx + stringtab;
-      sym->symbol.value = bufp->symbol_value;
+      sym->symbol.name = bfd_getb32 (bufp->name) + stringtab;
+      sym->symbol.value = bfd_getb32 (bufp->symbol_value);
       sym->symbol.section = 0;
       sym->symbol.flags = 0;
 
-      switch (bufp->symbol_type)
+      switch (symbol_type)
        {
        case ST_ENTRY:
        case ST_MILLICODE:
@@ -4340,7 +4753,7 @@ som_slurp_symbol_table (bfd *abfd)
          sym->symbol.value &= ~0x3;
          /* If the symbol's scope is SS_UNSAT, then these are
             undefined function symbols.  */
-         if (bufp->symbol_scope == SS_UNSAT)
+         if (symbol_scope == SS_UNSAT)
            sym->symbol.flags |= BSF_FUNCTION;
 
        default:
@@ -4348,12 +4761,12 @@ som_slurp_symbol_table (bfd *abfd)
        }
 
       /* Handle scoping and section information.  */
-      switch (bufp->symbol_scope)
+      switch (symbol_scope)
        {
        /* symbol_info field is undefined for SS_EXTERNAL and SS_UNSAT symbols,
           so the section associated with this symbol can't be known.  */
        case SS_EXTERNAL:
-         if (bufp->symbol_type != ST_STORAGE)
+         if (symbol_type != ST_STORAGE)
            sym->symbol.section = bfd_und_section_ptr;
          else
            sym->symbol.section = bfd_com_section_ptr;
@@ -4361,7 +4774,7 @@ som_slurp_symbol_table (bfd *abfd)
          break;
 
        case SS_UNSAT:
-         if (bufp->symbol_type != ST_STORAGE)
+         if (symbol_type != ST_STORAGE)
            sym->symbol.section = bfd_und_section_ptr;
          else
            sym->symbol.section = bfd_com_section_ptr;
@@ -4381,7 +4794,7 @@ som_slurp_symbol_table (bfd *abfd)
        }
 
       /* Check for a weak symbol.  */
-      if (bufp->secondary_def)
+      if (flags & SOM_SYMBOL_SECONDARY_DEF)
        sym->symbol.flags |= BSF_WEAK;
 
       /* Mark section symbols and symbols used by the debugger.
@@ -4405,18 +4818,17 @@ som_slurp_symbol_table (bfd *abfd)
 
   /* We modify the symbol count to record the number of BFD symbols we
      created.  */
-  bfd_get_symcount (abfd) = sym - symbase;
+  abfd->symcount = sym - symbase;
 
   /* Save our results and return success.  */
   obj_som_symtab (abfd) = symbase;
  successful_return:
-  if (buf != NULL)
-    free (buf);
+  free (buf);
   return (TRUE);
 
  error_return:
-  if (buf != NULL)
-    free (buf);
+  free (symbase);
+  free (buf);
   return FALSE;
 }
 
@@ -4448,14 +4860,14 @@ som_canonicalize_symtab (bfd *abfd, asymbol **location)
 static asymbol *
 som_make_empty_symbol (bfd *abfd)
 {
-  bfd_size_type amt = sizeof (som_symbol_type);
-  som_symbol_type *new = bfd_zalloc (abfd, amt);
+  size_t amt = sizeof (som_symbol_type);
+  som_symbol_type *new_symbol_type = bfd_zalloc (abfd, amt);
 
-  if (new == NULL)
+  if (new_symbol_type == NULL)
     return NULL;
-  new->symbol.the_bfd = abfd;
+  new_symbol_type->symbol.the_bfd = abfd;
 
-  return &new->symbol;
+  return &new_symbol_type->symbol;
 }
 
 /* Print symbol information.  */
@@ -4625,7 +5037,7 @@ som_set_reloc_info (unsigned char *fixup,
                  push (v);
                }
              else
-               /* An operator.  Pop two two values from the stack and
+               /* An operator.  Pop two values from the stack and
                   use them as operands to the given operation.  Push
                   the result of the operation back on the stack.  */
                switch (c)
@@ -4811,8 +5223,7 @@ som_set_reloc_info (unsigned char *fixup,
                      if (!bfd_malloc_and_get_section (section->owner, section,
                                                       &contents))
                        {
-                         if (contents != NULL)
-                           free (contents);
+                         free (contents);
                          return (unsigned) -1;
                        }
                      section->contents = contents;
@@ -4862,7 +5273,7 @@ som_slurp_reloc_table (bfd *abfd,
   unsigned int fixup_stream_size;
   arelent *internal_relocs;
   unsigned int num_relocs;
-  bfd_size_type amt;
+  size_t amt;
 
   fixup_stream_size = som_section_data (section)->reloc_size;
   /* If there were no relocations, then there is nothing to do.  */
@@ -4873,17 +5284,13 @@ som_slurp_reloc_table (bfd *abfd,
      parsed.  We must do so now to know how many relocations exist.  */
   if (section->reloc_count == (unsigned) -1)
     {
-      amt = fixup_stream_size;
-      external_relocs = bfd_malloc (amt);
-      if (external_relocs == NULL)
-       return FALSE;
       /* Read in the external forms.  */
-      if (bfd_seek (abfd,
-                   obj_som_reloc_filepos (abfd) + section->rel_filepos,
-                   SEEK_SET)
-         != 0)
+      if (bfd_seek (abfd, obj_som_reloc_filepos (abfd) + section->rel_filepos,
+                   SEEK_SET) != 0)
        return FALSE;
-      if (bfd_bread (external_relocs, amt, abfd) != amt)
+      amt = fixup_stream_size;
+      external_relocs = _bfd_malloc_and_read (abfd, amt, amt);
+      if (external_relocs == NULL)
        return FALSE;
 
       /* Let callers know how many relocations found.
@@ -4906,9 +5313,12 @@ som_slurp_reloc_table (bfd *abfd,
   if (section->relocation != NULL)
     return TRUE;
 
-  amt = num_relocs;
-  amt *= sizeof (arelent);
-  internal_relocs = bfd_zalloc (abfd, (amt));
+  if (_bfd_mul_overflow (num_relocs, sizeof (arelent), &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      return FALSE;
+    }
+  internal_relocs = bfd_zalloc (abfd, amt);
   if (internal_relocs == NULL)
     return FALSE;
 
@@ -4939,8 +5349,11 @@ som_get_reloc_upper_bound (bfd *abfd, sec_ptr asect)
        return -1;
       return (asect->reloc_count + 1) * sizeof (arelent *);
     }
-  /* There are no relocations.  */
-  return 0;
+
+  /* There are no relocations.  Return enough space to hold the
+     NULL pointer which will be installed if som_canonicalize_reloc
+     is called.  */
+  return sizeof (arelent *);
 }
 
 /* Convert relocations from SOM (external) form into BFD internal
@@ -4968,7 +5381,7 @@ som_canonicalize_reloc (bfd *abfd,
   return section->reloc_count;
 }
 
-extern const bfd_target som_vec;
+extern const bfd_target hppa_som_vec;
 
 /* A hook to set up object file dependent section information.  */
 
@@ -4977,7 +5390,7 @@ som_new_section_hook (bfd *abfd, asection *newsect)
 {
   if (!newsect->used_by_bfd)
     {
-      bfd_size_type amt = sizeof (struct som_section_data_struct);
+      size_t amt = sizeof (struct som_section_data_struct);
 
       newsect->used_by_bfd = bfd_zalloc (abfd, amt);
       if (!newsect->used_by_bfd)
@@ -5023,7 +5436,7 @@ som_bfd_copy_private_section_data (bfd *ibfd,
                                   bfd *obfd,
                                   asection *osection)
 {
-  bfd_size_type amt;
+  size_t amt;
 
   /* One day we may try to grok other private data.  */
   if (ibfd->xvec->flavour != bfd_target_som_flavour
@@ -5042,8 +5455,18 @@ som_bfd_copy_private_section_data (bfd *ibfd,
 
   /* Reparent if necessary.  */
   if (som_section_data (osection)->copy_data->container)
-    som_section_data (osection)->copy_data->container =
-      som_section_data (osection)->copy_data->container->output_section;
+    {
+      if (som_section_data (osection)->copy_data->container->output_section)
+       som_section_data (osection)->copy_data->container =
+         som_section_data (osection)->copy_data->container->output_section;
+      else
+       {
+         /* User has specified a subspace without its containing space.  */
+         _bfd_error_handler (_("%pB[%pA]: no output section for space %pA"),
+           obfd, osection, som_section_data (osection)->copy_data->container);
+         return FALSE;
+       }
+    }
 
   return TRUE;
 }
@@ -5077,7 +5500,7 @@ static bfd_boolean
 som_bfd_print_private_bfd_data (bfd *abfd, void *farg)
 {
   struct som_exec_auxhdr *exec_header;
-  struct aux_id* auxhdr;
+  struct som_aux_id* auxhdr;
   FILE *f;
 
   f = (FILE *) farg;
@@ -5101,7 +5524,7 @@ som_bfd_print_private_bfd_data (bfd *abfd, void *farg)
       fprintf (f, "  length             %#x\n", auxhdr->length);
 
       /* Note that, depending on the HP-UX version, the following fields can be
-         either ints, or longs.  */
+        either ints, or longs.  */
 
       fprintf (f, "  text size          %#lx\n", (long) exec_header->exec_tsize);
       fprintf (f, "  text memory offset %#lx\n", (long) exec_header->exec_tmem);
@@ -5131,7 +5554,7 @@ bfd_som_set_section_attributes (asection *section,
   /* Allocate memory to hold the magic information.  */
   if (som_section_data (section)->copy_data == NULL)
     {
-      bfd_size_type amt = sizeof (struct som_copyable_section_data_struct);
+      size_t amt = sizeof (struct som_copyable_section_data_struct);
 
       som_section_data (section)->copy_data = bfd_zalloc (section->owner, amt);
       if (som_section_data (section)->copy_data == NULL)
@@ -5151,7 +5574,7 @@ bfd_som_set_section_attributes (asection *section,
 bfd_boolean
 bfd_som_set_subsection_attributes (asection *section,
                                   asection *container,
-                                  int access,
+                                  int access_ctr,
                                   unsigned int sort_key,
                                   int quadrant,
                                   int comdat,
@@ -5161,14 +5584,14 @@ bfd_som_set_subsection_attributes (asection *section,
   /* Allocate memory to hold the magic information.  */
   if (som_section_data (section)->copy_data == NULL)
     {
-      bfd_size_type amt = sizeof (struct som_copyable_section_data_struct);
+      size_t amt = sizeof (struct som_copyable_section_data_struct);
 
       som_section_data (section)->copy_data = bfd_zalloc (section->owner, amt);
       if (som_section_data (section)->copy_data == NULL)
        return FALSE;
     }
   som_section_data (section)->copy_data->sort_key = sort_key;
-  som_section_data (section)->copy_data->access_control_bits = access;
+  som_section_data (section)->copy_data->access_control_bits = access_ctr;
   som_section_data (section)->copy_data->quadrant = quadrant;
   som_section_data (section)->copy_data->container = container;
   som_section_data (section)->copy_data->is_comdat = comdat;
@@ -5195,7 +5618,7 @@ bfd_som_set_symbol_type (asymbol *symbol, unsigned int type)
 bfd_boolean
 bfd_som_attach_aux_hdr (bfd *abfd, int type, char *string)
 {
-  bfd_size_type amt;
+  size_t amt;
 
   if (type == VERSION_AUX_ID)
     {
@@ -5204,32 +5627,32 @@ bfd_som_attach_aux_hdr (bfd *abfd, int type, char *string)
 
       if (len % 4)
        pad = (4 - (len % 4));
-      amt = sizeof (struct aux_id) + sizeof (unsigned int) + len + pad;
+      amt = sizeof (struct som_string_auxhdr) + len + pad;
       obj_som_version_hdr (abfd) = bfd_zalloc (abfd, amt);
       if (!obj_som_version_hdr (abfd))
        return FALSE;
       obj_som_version_hdr (abfd)->header_id.type = VERSION_AUX_ID;
-      obj_som_version_hdr (abfd)->header_id.length = len + pad;
-      obj_som_version_hdr (abfd)->header_id.length += sizeof (int);
+      obj_som_version_hdr (abfd)->header_id.length = 4 + len + pad;
       obj_som_version_hdr (abfd)->string_length = len;
-      strncpy (obj_som_version_hdr (abfd)->user_string, string, len);
+      memcpy (obj_som_version_hdr (abfd)->string, string, len);
+      memset (obj_som_version_hdr (abfd)->string + len, 0, pad);
     }
   else if (type == COPYRIGHT_AUX_ID)
     {
-      int len = strlen (string);
+      size_t len = strlen (string);
       int pad = 0;
 
       if (len % 4)
        pad = (4 - (len % 4));
-      amt = sizeof (struct aux_id) + sizeof (unsigned int) + len + pad;
+      amt = sizeof (struct som_string_auxhdr) + len + pad;
       obj_som_copyright_hdr (abfd) = bfd_zalloc (abfd, amt);
       if (!obj_som_copyright_hdr (abfd))
        return FALSE;
       obj_som_copyright_hdr (abfd)->header_id.type = COPYRIGHT_AUX_ID;
-      obj_som_copyright_hdr (abfd)->header_id.length = len + pad;
-      obj_som_copyright_hdr (abfd)->header_id.length += sizeof (int);
+      obj_som_copyright_hdr (abfd)->header_id.length = len + pad + 4;
       obj_som_copyright_hdr (abfd)->string_length = len;
-      strcpy (obj_som_copyright_hdr (abfd)->copyright, string);
+      memcpy (obj_som_copyright_hdr (abfd)->string, string, len);
+      memset (obj_som_copyright_hdr (abfd)->string + len, 0, pad);
     }
   return TRUE;
 }
@@ -5244,18 +5667,20 @@ bfd_som_attach_compilation_unit (bfd *abfd,
                                 const char *product_id,
                                 const char *version_id)
 {
-  COMPUNIT *n = (COMPUNIT *) bfd_zalloc (abfd, (bfd_size_type) COMPUNITSZ);
+  struct som_compilation_unit *n;
 
+  n = (struct som_compilation_unit *) bfd_zalloc
+    (abfd, (bfd_size_type) sizeof (*n));
   if (n == NULL)
     return FALSE;
 
 #define STRDUP(f) \
   if (f != NULL) \
     { \
-      n->f.n_name = bfd_alloc (abfd, (bfd_size_type) strlen (f) + 1); \
-      if (n->f.n_name == NULL) \
+      n->f.name = bfd_alloc (abfd, (bfd_size_type) strlen (f) + 1); \
+      if (n->f.name == NULL) \
        return FALSE; \
-      strcpy (n->f.n_name, f); \
+      strcpy (n->f.name, f); \
     }
 
   STRDUP (name);
@@ -5332,22 +5757,26 @@ som_set_arch_mach (bfd *abfd,
 
 static bfd_boolean
 som_find_nearest_line (bfd *abfd,
-                      asection *section,
                       asymbol **symbols,
+                      asection *section,
                       bfd_vma offset,
                       const char **filename_ptr,
                       const char **functionname_ptr,
-                      unsigned int *line_ptr)
+                      unsigned int *line_ptr,
+                      unsigned int *discriminator_ptr)
 {
   bfd_boolean found;
   asymbol *func;
   bfd_vma low_func;
   asymbol **p;
 
+  if (discriminator_ptr)
+    *discriminator_ptr = 0;
+
   if (! _bfd_stab_section_find_nearest_line (abfd, symbols, section, offset,
-                                             & found, filename_ptr,
-                                             functionname_ptr, line_ptr,
-                                             & somdata (abfd).line_info))
+                                            & found, filename_ptr,
+                                            functionname_ptr, line_ptr,
+                                            & somdata (abfd).line_info))
     return FALSE;
 
   if (found)
@@ -5361,9 +5790,9 @@ som_find_nearest_line (bfd *abfd,
   low_func = 0;
 
   for (p = symbols; *p != NULL; p++)
-    { 
+    {
       som_symbol_type *q = (som_symbol_type *) *p;
-  
+
       if (q->som_type == SYMBOL_TYPE_ENTRY
          && q->symbol.section == section
          && q->symbol.value >= low_func
@@ -5388,8 +5817,7 @@ static int
 som_sizeof_headers (bfd *abfd ATTRIBUTE_UNUSED,
                    struct bfd_link_info *info ATTRIBUTE_UNUSED)
 {
-  (*_bfd_error_handler) (_("som_sizeof_headers unimplemented"));
-  fflush (stderr);
+  _bfd_error_handler (_("som_sizeof_headers unimplemented"));
   abort ();
   return 0;
 }
@@ -5476,73 +5904,86 @@ som_get_symbol_info (bfd *ignore_abfd ATTRIBUTE_UNUSED,
 
 static bfd_boolean
 som_bfd_count_ar_symbols (bfd *abfd,
-                         struct lst_header *lst_header,
+                         struct som_lst_header *lst_header,
                          symindex *count)
 {
   unsigned int i;
-  unsigned int *hash_table = NULL;
-  bfd_size_type amt;
-  file_ptr lst_filepos = bfd_tell (abfd) - sizeof (struct lst_header);
+  unsigned char *hash_table;
+  size_t amt;
+  file_ptr lst_filepos;
+
+  lst_filepos = bfd_tell (abfd) - sizeof (struct som_external_lst_header);
 
-  amt = lst_header->hash_size;
-  amt *= sizeof (unsigned int);
-  hash_table = bfd_malloc (amt);
+  /* Read in the hash table.  The hash table is an array of 32-bit
+     file offsets which point to the hash chains.  */
+  if (_bfd_mul_overflow (lst_header->hash_size, 4, &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      return FALSE;
+    }
+  hash_table = _bfd_malloc_and_read (abfd, amt, amt);
   if (hash_table == NULL && lst_header->hash_size != 0)
     goto error_return;
 
   /* Don't forget to initialize the counter!  */
   *count = 0;
 
-  /* Read in the hash table.  The has table is an array of 32bit file offsets
-     which point to the hash chains.  */
-  if (bfd_bread ((void *) hash_table, amt, abfd) != amt)
-    goto error_return;
-
   /* Walk each chain counting the number of symbols found on that particular
      chain.  */
   for (i = 0; i < lst_header->hash_size; i++)
     {
-      struct lst_symbol_record lst_symbol;
+      struct som_external_lst_symbol_record ext_lst_symbol;
+      unsigned int hash_val = bfd_getb32 (hash_table + 4 * i);
 
       /* An empty chain has zero as it's file offset.  */
-      if (hash_table[i] == 0)
+      if (hash_val == 0)
        continue;
 
       /* Seek to the first symbol in this hash chain.  */
-      if (bfd_seek (abfd, lst_filepos + hash_table[i], SEEK_SET) != 0)
+      if (bfd_seek (abfd, lst_filepos + hash_val, SEEK_SET) != 0)
        goto error_return;
 
       /* Read in this symbol and update the counter.  */
-      amt = sizeof (lst_symbol);
-      if (bfd_bread ((void *) &lst_symbol, amt, abfd) != amt)
+      amt = sizeof (ext_lst_symbol);
+      if (bfd_bread ((void *) &ext_lst_symbol, amt, abfd) != amt)
        goto error_return;
 
       (*count)++;
 
       /* Now iterate through the rest of the symbols on this chain.  */
-      while (lst_symbol.next_entry)
+      while (1)
        {
+         unsigned int next_entry = bfd_getb32 (ext_lst_symbol.next_entry);
+
+         if (next_entry == 0)
+           break;
+
+         /* Assume symbols on a chain are in increasing file offset
+            order.  Otherwise we can loop here with fuzzed input.  */
+         if (next_entry < hash_val + sizeof (ext_lst_symbol))
+           {
+             bfd_set_error (bfd_error_bad_value);
+             goto error_return;
+           }
+         hash_val = next_entry;
 
          /* Seek to the next symbol.  */
-         if (bfd_seek (abfd, lst_filepos + lst_symbol.next_entry, SEEK_SET)
-             != 0)
+         if (bfd_seek (abfd, lst_filepos + next_entry, SEEK_SET) != 0)
            goto error_return;
 
          /* Read the symbol in and update the counter.  */
-         amt = sizeof (lst_symbol);
-         if (bfd_bread ((void *) &lst_symbol, amt, abfd) != amt)
+         amt = sizeof (ext_lst_symbol);
+         if (bfd_bread ((void *) &ext_lst_symbol, amt, abfd) != amt)
            goto error_return;
 
          (*count)++;
        }
     }
-  if (hash_table != NULL)
-    free (hash_table);
+  free (hash_table);
   return TRUE;
 
  error_return:
-  if (hash_table != NULL)
-    free (hash_table);
+  free (hash_table);
   return FALSE;
 }
 
@@ -5551,25 +5992,28 @@ som_bfd_count_ar_symbols (bfd *abfd,
 
 static bfd_boolean
 som_bfd_fill_in_ar_symbols (bfd *abfd,
-                           struct lst_header *lst_header,
+                           struct som_lst_header *lst_header,
                            carsym **syms)
 {
-  unsigned int i, len;
+  unsigned int i;
   carsym *set = syms[0];
-  unsigned int *hash_table = NULL;
-  struct som_entry *som_dict = NULL;
-  bfd_size_type amt;
-  file_ptr lst_filepos = bfd_tell (abfd) - sizeof (struct lst_header);
+  unsigned char *hash_table;
+  struct som_external_som_entry *som_dict = NULL;
+  size_t amt;
+  file_ptr lst_filepos;
+  unsigned int string_loc;
 
-  amt = lst_header->hash_size;
-  amt *= sizeof (unsigned int);
-  hash_table = bfd_malloc (amt);
-  if (hash_table == NULL && lst_header->hash_size != 0)
-    goto error_return;
+  lst_filepos = bfd_tell (abfd) - sizeof (struct som_external_lst_header);
 
   /* Read in the hash table.  The has table is an array of 32bit file offsets
      which point to the hash chains.  */
-  if (bfd_bread ((void *) hash_table, amt, abfd) != amt)
+  if (_bfd_mul_overflow (lst_header->hash_size, 4, &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      return FALSE;
+    }
+  hash_table = _bfd_malloc_and_read (abfd, amt, amt);
+  if (hash_table == NULL && lst_header->hash_size != 0)
     goto error_return;
 
   /* Seek to and read in the SOM dictionary.  We will need this to fill
@@ -5577,26 +6021,36 @@ som_bfd_fill_in_ar_symbols (bfd *abfd,
   if (bfd_seek (abfd, lst_filepos + lst_header->dir_loc, SEEK_SET) != 0)
     goto error_return;
 
-  amt = lst_header->module_count;
-  amt *= sizeof (struct som_entry);
-  som_dict = bfd_malloc (amt);
+  if (_bfd_mul_overflow (lst_header->module_count,
+                        sizeof (struct som_external_som_entry), &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      goto error_return;
+    }
+  som_dict = (struct som_external_som_entry *)
+    _bfd_malloc_and_read (abfd, amt, amt);
   if (som_dict == NULL && lst_header->module_count != 0)
     goto error_return;
 
-  if (bfd_bread ((void *) som_dict, amt, abfd) != amt)
-    goto error_return;
+  string_loc = lst_header->string_loc;
 
   /* Walk each chain filling in the carsyms as we go along.  */
   for (i = 0; i < lst_header->hash_size; i++)
     {
-      struct lst_symbol_record lst_symbol;
+      struct som_external_lst_symbol_record lst_symbol;
+      unsigned int hash_val;
+      size_t len;
+      unsigned char ext_len[4];
+      char *name;
+      unsigned int ndx;
 
       /* An empty chain has zero as it's file offset.  */
-      if (hash_table[i] == 0)
+      hash_val = bfd_getb32 (hash_table + 4 * i);
+      if (hash_val == 0)
        continue;
 
       /* Seek to and read the first symbol on the chain.  */
-      if (bfd_seek (abfd, lst_filepos + hash_table[i], SEEK_SET) != 0)
+      if (bfd_seek (abfd, lst_filepos + hash_val, SEEK_SET) != 0)
        goto error_return;
 
       amt = sizeof (lst_symbol);
@@ -5610,36 +6064,50 @@ som_bfd_fill_in_ar_symbols (bfd *abfd,
         and index into it.  Well, according to the SOM ABI the string
         index can point *anywhere* in the archive to save space, so just
         using the string table would not be safe.  */
-      if (bfd_seek (abfd, lst_filepos + lst_header->string_loc
-                           + lst_symbol.name.n_strx - 4, SEEK_SET) != 0)
+      if (bfd_seek (abfd, (lst_filepos + string_loc
+                          + bfd_getb32 (lst_symbol.name) - 4), SEEK_SET) != 0)
        goto error_return;
 
-      if (bfd_bread (&len, (bfd_size_type) 4, abfd) != 4)
+      if (bfd_bread (&ext_len, (bfd_size_type) 4, abfd) != 4)
        goto error_return;
+      len = bfd_getb32 (ext_len);
 
       /* Allocate space for the name and null terminate it too.  */
-      set->name = bfd_zalloc (abfd, (bfd_size_type) len + 1);
-      if (!set->name)
-       goto error_return;
-      if (bfd_bread (set->name, (bfd_size_type) len, abfd) != len)
+      if (len == (size_t) -1)
+       {
+         bfd_set_error (bfd_error_no_memory);
+         goto error_return;
+       }
+      name = (char *) _bfd_alloc_and_read (abfd, len + 1, len);
+      if (!name)
        goto error_return;
-
-      set->name[len] = 0;
+      name[len] = 0;
+      set->name = name;
 
       /* Fill in the file offset.  Note that the "location" field points
         to the SOM itself, not the ar_hdr in front of it.  */
-      set->file_offset = som_dict[lst_symbol.som_index].location
-                         - sizeof (struct ar_hdr);
+      ndx = bfd_getb32 (lst_symbol.som_index);
+      if (ndx >= lst_header->module_count)
+       {
+         bfd_set_error (bfd_error_bad_value);
+         goto error_return;
+       }
+      set->file_offset
+       = bfd_getb32 (som_dict[ndx].location) - sizeof (struct ar_hdr);
 
       /* Go to the next symbol.  */
       set++;
 
       /* Iterate through the rest of the chain.  */
-      while (lst_symbol.next_entry)
+      while (1)
        {
+         unsigned int next_entry = bfd_getb32 (lst_symbol.next_entry);
+
+         if (next_entry == 0)
+           break;
+
          /* Seek to the next symbol and read it in.  */
-         if (bfd_seek (abfd, lst_filepos + lst_symbol.next_entry, SEEK_SET)
-             != 0)
+         if (bfd_seek (abfd, lst_filepos + next_entry, SEEK_SET) != 0)
            goto error_return;
 
          amt = sizeof (lst_symbol);
@@ -5647,26 +6115,36 @@ som_bfd_fill_in_ar_symbols (bfd *abfd,
            goto error_return;
 
          /* Seek to the name length & string and read them in.  */
-         if (bfd_seek (abfd, lst_filepos + lst_header->string_loc
-                               + lst_symbol.name.n_strx - 4, SEEK_SET) != 0)
+         if (bfd_seek (abfd, lst_filepos + string_loc
+                       + bfd_getb32 (lst_symbol.name) - 4, SEEK_SET) != 0)
            goto error_return;
 
-         if (bfd_bread (&len, (bfd_size_type) 4, abfd) != 4)
+         if (bfd_bread (&ext_len, (bfd_size_type) 4, abfd) != 4)
            goto error_return;
+         len = bfd_getb32 (ext_len);
 
          /* Allocate space for the name and null terminate it too.  */
-         set->name = bfd_zalloc (abfd, (bfd_size_type) len + 1);
-         if (!set->name)
-           goto error_return;
-
-         if (bfd_bread (set->name, (bfd_size_type) len, abfd) != len)
+         if (len == (size_t) -1)
+           {
+             bfd_set_error (bfd_error_no_memory);
+             goto error_return;
+           }
+         name = (char *) _bfd_alloc_and_read (abfd, len + 1, len);
+         if (!name)
            goto error_return;
-         set->name[len] = 0;
+         name[len] = 0;
+         set->name = name;
 
          /* Fill in the file offset.  Note that the "location" field points
             to the SOM itself, not the ar_hdr in front of it.  */
-         set->file_offset = som_dict[lst_symbol.som_index].location
-                              - sizeof (struct ar_hdr);
+         ndx = bfd_getb32 (lst_symbol.som_index);
+         if (ndx >= lst_header->module_count)
+           {
+             bfd_set_error (bfd_error_bad_value);
+             goto error_return;
+           }
+         set->file_offset
+           = bfd_getb32 (som_dict[ndx].location) - sizeof (struct ar_hdr);
 
          /* Go on to the next symbol.  */
          set++;
@@ -5674,17 +6152,13 @@ som_bfd_fill_in_ar_symbols (bfd *abfd,
     }
   /* If we haven't died by now, then we successfully read the entire
      archive symbol table.  */
-  if (hash_table != NULL)
-    free (hash_table);
-  if (som_dict != NULL)
-    free (som_dict);
+  free (hash_table);
+  free (som_dict);
   return TRUE;
 
  error_return:
-  if (hash_table != NULL)
-    free (hash_table);
-  if (som_dict != NULL)
-    free (som_dict);
+  free (hash_table);
+  free (som_dict);
   return FALSE;
 }
 
@@ -5693,12 +6167,13 @@ som_bfd_fill_in_ar_symbols (bfd *abfd,
 static bfd_boolean
 som_slurp_armap (bfd *abfd)
 {
-  struct lst_header lst_header;
+  struct som_external_lst_header ext_lst_header;
+  struct som_lst_header lst_header;
   struct ar_hdr ar_header;
   unsigned int parsed_size;
   struct artdata *ardata = bfd_ardata (abfd);
   char nextname[17];
-  bfd_size_type amt = 16;
+  size_t amt = 16;
   int i = bfd_bread ((void *) nextname, amt, abfd);
 
   /* Special cases.  */
@@ -5713,7 +6188,7 @@ som_slurp_armap (bfd *abfd)
   /* For archives without .o files there is no symbol table.  */
   if (! CONST_STRNEQ (nextname, "/               "))
     {
-      bfd_has_map (abfd) = FALSE;
+      abfd->has_armap = FALSE;
       return TRUE;
     }
 
@@ -5742,10 +6217,12 @@ som_slurp_armap (bfd *abfd)
 
   /* Read in the library symbol table.  We'll make heavy use of this
      in just a minute.  */
-  amt = sizeof (struct lst_header);
-  if (bfd_bread ((void *) &lst_header, amt, abfd) != amt)
+  amt = sizeof (struct som_external_lst_header);
+  if (bfd_bread ((void *) &ext_lst_header, amt, abfd) != amt)
     return FALSE;
 
+  som_swap_lst_header_in (&ext_lst_header, &lst_header);
+
   /* Sanity check.  */
   if (lst_header.a_magic != LIBMAGIC)
     {
@@ -5759,13 +6236,17 @@ som_slurp_armap (bfd *abfd)
 
   /* Get back to the start of the library symbol table.  */
   if (bfd_seek (abfd, (ardata->first_file_filepos - parsed_size
-                      + sizeof (struct lst_header)), SEEK_SET) != 0)
+                      + sizeof (struct som_external_lst_header)),
+               SEEK_SET) != 0)
     return FALSE;
 
   /* Initialize the cache and allocate space for the library symbols.  */
   ardata->cache = 0;
-  amt = ardata->symdef_count;
-  amt *= sizeof (carsym);
+  if (_bfd_mul_overflow (ardata->symdef_count, sizeof (carsym), &amt))
+    {
+      bfd_set_error (bfd_error_file_too_big);
+      return FALSE;
+    }
   ardata->symdefs = bfd_alloc (abfd, amt);
   if (!ardata->symdefs)
     return FALSE;
@@ -5780,7 +6261,7 @@ som_slurp_armap (bfd *abfd)
     return FALSE;
 
   /* Notify the generic archive code that we have a symbol map.  */
-  bfd_has_map (abfd) = TRUE;
+  abfd->has_armap = TRUE;
   return TRUE;
 }
 
@@ -5850,7 +6331,7 @@ som_bfd_prep_for_ar_write (bfd *abfd,
          /* Bump the various counters, being careful to honor
             alignment considerations in the string table.  */
          (*num_syms)++;
-         *stringsize = *stringsize + strlen (sym->symbol.name) + 5;
+         *stringsize += strlen (sym->symbol.name) + 5;
          while (*stringsize % 4)
            (*stringsize)++;
        }
@@ -5883,41 +6364,52 @@ static bfd_boolean
 som_bfd_ar_write_symbol_stuff (bfd *abfd,
                               unsigned int nsyms,
                               unsigned int string_size,
-                              struct lst_header lst,
+                              struct som_external_lst_header lst,
                               unsigned elength)
 {
-  file_ptr lst_filepos;
   char *strings = NULL, *p;
-  struct lst_symbol_record *lst_syms = NULL, *curr_lst_sym;
+  struct som_external_lst_symbol_record *lst_syms = NULL, *curr_lst_sym;
   bfd *curr_bfd;
-  unsigned int *hash_table = NULL;
-  struct som_entry *som_dict = NULL;
-  struct lst_symbol_record **last_hash_entry = NULL;
+  unsigned char *hash_table = NULL;
+  struct som_external_som_entry *som_dict = NULL;
+  struct som_external_lst_symbol_record **last_hash_entry = NULL;
   unsigned int curr_som_offset, som_index = 0;
-  bfd_size_type amt;
+  size_t amt;
+  unsigned int module_count;
+  unsigned int hash_size;
 
-  amt = lst.hash_size;
-  amt *= sizeof (unsigned int);
+  hash_size = bfd_getb32 (lst.hash_size);
+  if (_bfd_mul_overflow (hash_size, 4, &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return FALSE;
+    }
   hash_table = bfd_zmalloc (amt);
-  if (hash_table == NULL && lst.hash_size != 0)
+  if (hash_table == NULL && hash_size != 0)
     goto error_return;
 
-  amt = lst.module_count;
-  amt *= sizeof (struct som_entry);
+  module_count = bfd_getb32 (lst.module_count);
+  if (_bfd_mul_overflow (module_count,
+                        sizeof (struct som_external_som_entry), &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      goto error_return;
+    }
   som_dict = bfd_zmalloc (amt);
-  if (som_dict == NULL && lst.module_count != 0)
+  if (som_dict == NULL && module_count != 0)
     goto error_return;
 
-  amt = lst.hash_size;
-  amt *= sizeof (struct lst_symbol_record *);
+  if (_bfd_mul_overflow (hash_size,
+                        sizeof (struct som_external_lst_symbol_record *),
+                        &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      goto error_return;
+    }
   last_hash_entry = bfd_zmalloc (amt);
-  if (last_hash_entry == NULL && lst.hash_size != 0)
+  if (last_hash_entry == NULL && hash_size != 0)
     goto error_return;
 
-  /* Lots of fields are file positions relative to the start
-     of the lst record.  So save its location.  */
-  lst_filepos = bfd_tell (abfd) - sizeof (struct lst_header);
-
   /* Symbols have som_index fields, so we have to keep track of the
      index of each SOM in the archive.
 
@@ -5930,7 +6422,7 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
   /* We add in the size of the archive header twice as the location
      in the SOM dictionary is the actual offset of the SOM, not the
      archive header before the SOM.  */
-  curr_som_offset = 8 + 2 * sizeof (struct ar_hdr) + lst.file_end;
+  curr_som_offset = 8 + 2 * sizeof (struct ar_hdr) + bfd_getb32 (lst.file_end);
 
   /* Make room for the archive header and the contents of the
      extended string table.  Note that elength includes the size
@@ -5942,12 +6434,16 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
   curr_som_offset = (curr_som_offset + 0x1) & ~0x1;
 
   /* FIXME should be done with buffers just like everything else...  */
-  amt = nsyms;
-  amt *= sizeof (struct lst_symbol_record);
+  if (_bfd_mul_overflow (nsyms,
+                        sizeof (struct som_external_lst_symbol_record), &amt))
+    {
+      bfd_set_error (bfd_error_no_memory);
+      goto error_return;
+    }
   lst_syms = bfd_malloc (amt);
   if (lst_syms == NULL && nsyms != 0)
     goto error_return;
-  strings = bfd_malloc ((bfd_size_type) string_size);
+  strings = bfd_malloc (string_size);
   if (strings == NULL && string_size != 0)
     goto error_return;
 
@@ -5980,6 +6476,11 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
       for (i = 0; i < curr_count; i++, sym++)
        {
          struct som_misc_symbol_info info;
+         struct som_external_lst_symbol_record *last;
+         unsigned int symbol_pos;
+         unsigned int slen;
+         unsigned int symbol_key;
+         unsigned int flags;
 
          /* Derive SOM information from the BFD symbol.  */
          som_bfd_derive_misc_symbol_info (curr_bfd, &sym->symbol, &info);
@@ -6001,72 +6502,74 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
 
          /* If this is the first symbol from this SOM, then update
             the SOM dictionary too.  */
-         if (som_dict[som_index].location == 0)
+         if (bfd_getb32 (som_dict[som_index].location) == 0)
            {
-             som_dict[som_index].location = curr_som_offset;
-             som_dict[som_index].length = arelt_size (curr_bfd);
+             bfd_putb32 (curr_som_offset, som_dict[som_index].location);
+             bfd_putb32 (arelt_size (curr_bfd), som_dict[som_index].length);
            }
 
+         symbol_key = som_bfd_ar_symbol_hash (&sym->symbol);
+
          /* Fill in the lst symbol record.  */
-         curr_lst_sym->hidden = 0;
-         curr_lst_sym->secondary_def = info.secondary_def;
-         curr_lst_sym->symbol_type = info.symbol_type;
-         curr_lst_sym->symbol_scope = info.symbol_scope;
-         curr_lst_sym->check_level = 0;
-         curr_lst_sym->must_qualify = 0;
-         curr_lst_sym->initially_frozen = 0;
-         curr_lst_sym->memory_resident = 0;
-         curr_lst_sym->is_common = bfd_is_com_section (sym->symbol.section);
-         curr_lst_sym->dup_common = info.dup_common;
-         curr_lst_sym->xleast = 3;
-         curr_lst_sym->arg_reloc = info.arg_reloc;
-         curr_lst_sym->name.n_strx = p - strings + 4;
-         curr_lst_sym->qualifier_name.n_strx = 0;
-         curr_lst_sym->symbol_info = info.symbol_info;
-         curr_lst_sym->symbol_value = info.symbol_value | info.priv_level;
-         curr_lst_sym->symbol_descriptor = 0;
+         flags = 0;
+         if (info.secondary_def)
+           flags |= LST_SYMBOL_SECONDARY_DEF;
+         flags |= info.symbol_type << LST_SYMBOL_SYMBOL_TYPE_SH;
+         flags |= info.symbol_scope << LST_SYMBOL_SYMBOL_SCOPE_SH;
+         if (bfd_is_com_section (sym->symbol.section))
+           flags |= LST_SYMBOL_IS_COMMON;
+         if (info.dup_common)
+           flags |= LST_SYMBOL_DUP_COMMON;
+         flags |= 3 << LST_SYMBOL_XLEAST_SH;
+         flags |= info.arg_reloc << LST_SYMBOL_ARG_RELOC_SH;
+         bfd_putb32 (flags, curr_lst_sym->flags);
+         bfd_putb32 (p - strings + 4, curr_lst_sym->name);
+         bfd_putb32 (0, curr_lst_sym->qualifier_name);
+         bfd_putb32 (info.symbol_info, curr_lst_sym->symbol_info);
+         bfd_putb32 (info.symbol_value | info.priv_level,
+                     curr_lst_sym->symbol_value);
+         bfd_putb32 (0, curr_lst_sym->symbol_descriptor);
          curr_lst_sym->reserved = 0;
-         curr_lst_sym->som_index = som_index;
-         curr_lst_sym->symbol_key = som_bfd_ar_symbol_hash (&sym->symbol);
-         curr_lst_sym->next_entry = 0;
+         bfd_putb32 (som_index, curr_lst_sym->som_index);
+         bfd_putb32 (symbol_key, curr_lst_sym->symbol_key);
+         bfd_putb32 (0, curr_lst_sym->next_entry);
 
          /* Insert into the hash table.  */
-         if (hash_table[curr_lst_sym->symbol_key % lst.hash_size])
+         symbol_pos =
+           (curr_lst_sym - lst_syms)
+           * sizeof (struct som_external_lst_symbol_record)
+           + hash_size * 4
+           + module_count * sizeof (struct som_external_som_entry)
+           + sizeof (struct som_external_lst_header);
+         last = last_hash_entry[symbol_key % hash_size];
+         if (last != NULL)
            {
-             struct lst_symbol_record *tmp;
-
              /* There is already something at the head of this hash chain,
                 so tack this symbol onto the end of the chain.  */
-             tmp = last_hash_entry[curr_lst_sym->symbol_key % lst.hash_size];
-             tmp->next_entry
-               = (curr_lst_sym - lst_syms) * sizeof (struct lst_symbol_record)
-                 + lst.hash_size * 4
-                 + lst.module_count * sizeof (struct som_entry)
-                 + sizeof (struct lst_header);
+             bfd_putb32 (symbol_pos, last->next_entry);
            }
          else
            /* First entry in this hash chain.  */
-           hash_table[curr_lst_sym->symbol_key % lst.hash_size]
-             = (curr_lst_sym - lst_syms) * sizeof (struct lst_symbol_record)
-             + lst.hash_size * 4
-             + lst.module_count * sizeof (struct som_entry)
-             + sizeof (struct lst_header);
+           bfd_putb32 (symbol_pos, hash_table + 4 * (symbol_key % hash_size));
 
          /* Keep track of the last symbol we added to this chain so we can
             easily update its next_entry pointer.  */
-         last_hash_entry[curr_lst_sym->symbol_key % lst.hash_size]
-           = curr_lst_sym;
+         last_hash_entry[symbol_key % hash_size] = curr_lst_sym;
 
          /* Update the string table.  */
-         bfd_put_32 (abfd, strlen (sym->symbol.name), p);
+         slen = strlen (sym->symbol.name);
+         bfd_put_32 (abfd, slen, p);
          p += 4;
-         strcpy (p, sym->symbol.name);
-         p += strlen (sym->symbol.name) + 1;
-         while ((int) p % 4)
+         slen++; /* Nul terminator.  */
+         memcpy (p, sym->symbol.name, slen);
+         p += slen;
+         while (slen % 4)
            {
              bfd_put_8 (abfd, 0, p);
              p++;
+             slen++;
            }
+         BFD_ASSERT (p <= strings + string_size);
 
          /* Head to the next symbol.  */
          curr_lst_sym++;
@@ -6085,17 +6588,17 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
     }
 
   /* Now scribble out the hash table.  */
-  amt = lst.hash_size * 4;
+  amt = (size_t) hash_size * 4;
   if (bfd_bwrite ((void *) hash_table, amt, abfd) != amt)
     goto error_return;
 
   /* Then the SOM dictionary.  */
-  amt = lst.module_count * sizeof (struct som_entry);
+  amt = (size_t) module_count * sizeof (struct som_external_som_entry);
   if (bfd_bwrite ((void *) som_dict, amt, abfd) != amt)
     goto error_return;
 
   /* The library symbols.  */
-  amt = nsyms * sizeof (struct lst_symbol_record);
+  amt = (size_t) nsyms * sizeof (struct som_external_lst_symbol_record);
   if (bfd_bwrite ((void *) lst_syms, amt, abfd) != amt)
     goto error_return;
 
@@ -6104,29 +6607,19 @@ som_bfd_ar_write_symbol_stuff (bfd *abfd,
   if (bfd_bwrite ((void *) strings, amt, abfd) != amt)
     goto error_return;
 
-  if (hash_table != NULL)
-    free (hash_table);
-  if (som_dict != NULL)
-    free (som_dict);
-  if (last_hash_entry != NULL)
-    free (last_hash_entry);
-  if (lst_syms != NULL)
-    free (lst_syms);
-  if (strings != NULL)
-    free (strings);
+  free (hash_table);
+  free (som_dict);
+  free (last_hash_entry);
+  free (lst_syms);
+  free (strings);
   return TRUE;
 
  error_return:
-  if (hash_table != NULL)
-    free (hash_table);
-  if (som_dict != NULL)
-    free (som_dict);
-  if (last_hash_entry != NULL)
-    free (last_hash_entry);
-  if (lst_syms != NULL)
-    free (lst_syms);
-  if (strings != NULL)
-    free (strings);
+  free (hash_table);
+  free (som_dict);
+  free (last_hash_entry);
+  free (lst_syms);
+  free (strings);
 
   return FALSE;
 }
@@ -6146,12 +6639,14 @@ som_write_armap (bfd *abfd,
   struct stat statbuf;
   unsigned int i, lst_size, nsyms, stringsize;
   struct ar_hdr hdr;
-  struct lst_header lst;
-  int *p;
-  bfd_size_type amt;
+  struct som_external_lst_header lst;
+  unsigned char *p;
+  size_t amt;
+  unsigned int csum;
+  unsigned int module_count;
 
   /* We'll use this for the archive's date and mode later.  */
-  if (stat (abfd->filename, &statbuf) != 0)
+  if (stat (bfd_get_filename (abfd), &statbuf) != 0)
     {
       bfd_set_error (bfd_error_system_call);
       return FALSE;
@@ -6160,78 +6655,86 @@ som_write_armap (bfd *abfd,
   bfd_ardata (abfd)->armap_timestamp = statbuf.st_mtime + 60;
 
   /* Account for the lst header first.  */
-  lst_size = sizeof (struct lst_header);
+  lst_size = sizeof (struct som_external_lst_header);
 
   /* Start building the LST header.  */
   /* FIXME:  Do we need to examine each element to determine the
      largest id number?  */
-  lst.system_id = CPU_PA_RISC1_0;
-  lst.a_magic = LIBMAGIC;
-  lst.version_id = VERSION_ID;
-  lst.file_time.secs = 0;
-  lst.file_time.nanosecs = 0;
+  bfd_putb16 (CPU_PA_RISC1_0, &lst.system_id);
+  bfd_putb16 (LIBMAGIC, &lst.a_magic);
+  bfd_putb32 (VERSION_ID, &lst.version_id);
+  bfd_putb32 (0, &lst.file_time.secs);
+  bfd_putb32 (0, &lst.file_time.nanosecs);
 
-  lst.hash_loc = lst_size;
-  lst.hash_size = SOM_LST_HASH_SIZE;
+  bfd_putb32 (lst_size, &lst.hash_loc);
+  bfd_putb32 (SOM_LST_HASH_SIZE, &lst.hash_size);
 
   /* Hash table is a SOM_LST_HASH_SIZE 32bit offsets.  */
   lst_size += 4 * SOM_LST_HASH_SIZE;
 
   /* We need to count the number of SOMs in this archive.  */
   curr_bfd = abfd->archive_head;
-  lst.module_count = 0;
+  module_count = 0;
   while (curr_bfd != NULL)
     {
       /* Only true SOM objects count.  */
       if (curr_bfd->format == bfd_object
          && curr_bfd->xvec->flavour == bfd_target_som_flavour)
-       lst.module_count++;
+       module_count++;
       curr_bfd = curr_bfd->archive_next;
     }
-  lst.module_limit = lst.module_count;
-  lst.dir_loc = lst_size;
-  lst_size += sizeof (struct som_entry) * lst.module_count;
+  bfd_putb32 (module_count, &lst.module_count);
+  bfd_putb32 (module_count, &lst.module_limit);
+  bfd_putb32 (lst_size, &lst.dir_loc);
+  lst_size += sizeof (struct som_external_som_entry) * module_count;
 
   /* We don't support import/export tables, auxiliary headers,
      or free lists yet.  Make the linker work a little harder
      to make our life easier.  */
 
-  lst.export_loc = 0;
-  lst.export_count = 0;
-  lst.import_loc = 0;
-  lst.aux_loc = 0;
-  lst.aux_size = 0;
+  bfd_putb32 (0, &lst.export_loc);
+  bfd_putb32 (0, &lst.export_count);
+  bfd_putb32 (0, &lst.import_loc);
+  bfd_putb32 (0, &lst.aux_loc);
+  bfd_putb32 (0, &lst.aux_size);
 
   /* Count how many symbols we will have on the hash chains and the
      size of the associated string table.  */
   if (! som_bfd_prep_for_ar_write (abfd, &nsyms, &stringsize))
     return FALSE;
 
-  lst_size += sizeof (struct lst_symbol_record) * nsyms;
+  lst_size += sizeof (struct som_external_lst_symbol_record) * nsyms;
 
   /* For the string table.  One day we might actually use this info
      to avoid small seeks/reads when reading archives.  */
-  lst.string_loc = lst_size;
-  lst.string_size = stringsize;
+  bfd_putb32 (lst_size, &lst.string_loc);
+  bfd_putb32 (stringsize, &lst.string_size);
   lst_size += stringsize;
 
   /* SOM ABI says this must be zero.  */
-  lst.free_list = 0;
-  lst.file_end = lst_size;
+  bfd_putb32 (0, &lst.free_list);
+  bfd_putb32 (lst_size, &lst.file_end);
 
   /* Compute the checksum.  Must happen after the entire lst header
      has filled in.  */
-  p = (int *) &lst;
-  lst.checksum = 0;
-  for (i = 0; i < sizeof (struct lst_header) / sizeof (int) - 1; i++)
-    lst.checksum ^= *p++;
-
-  sprintf (hdr.ar_name, "/               ");
-  sprintf (hdr.ar_date, "%ld", bfd_ardata (abfd)->armap_timestamp);
-  sprintf (hdr.ar_uid, "%ld", (long) getuid ());
-  sprintf (hdr.ar_gid, "%ld", (long) getgid ());
-  sprintf (hdr.ar_mode, "%-8o", (unsigned int) statbuf.st_mode);
-  sprintf (hdr.ar_size, "%-10d", (int) lst_size);
+  p = (unsigned char *) &lst;
+  csum = 0;
+  for (i = 0; i < sizeof (struct som_external_lst_header) - sizeof (int);
+       i += 4)
+    csum ^= bfd_getb32 (&p[i]);
+  bfd_putb32 (csum, &lst.checksum);
+
+  sprintf (hdr.ar_name, "/              ");
+  _bfd_ar_spacepad (hdr.ar_date, sizeof (hdr.ar_date), "%-12ld",
+                   bfd_ardata (abfd)->armap_timestamp);
+  _bfd_ar_spacepad (hdr.ar_uid, sizeof (hdr.ar_uid), "%ld",
+                   statbuf.st_uid);
+  _bfd_ar_spacepad (hdr.ar_gid, sizeof (hdr.ar_gid), "%ld",
+                   statbuf.st_gid);
+  _bfd_ar_spacepad (hdr.ar_mode, sizeof (hdr.ar_mode), "%-8o",
+                   (unsigned int)statbuf.st_mode);
+  _bfd_ar_spacepad (hdr.ar_size, sizeof (hdr.ar_size), "%-10d",
+                   (int) lst_size);
   hdr.ar_fmag[0] = '`';
   hdr.ar_fmag[1] = '\012';
 
@@ -6246,7 +6749,7 @@ som_write_armap (bfd *abfd,
     return FALSE;
 
   /* Now scribble out the lst header.  */
-  amt = sizeof (struct lst_header);
+  amt = sizeof (struct som_external_lst_header);
   if (bfd_bwrite ((void *) &lst, amt, abfd) != amt)
     return FALSE;
 
@@ -6264,26 +6767,25 @@ som_write_armap (bfd *abfd,
 static bfd_boolean
 som_bfd_free_cached_info (bfd *abfd)
 {
-  asection *o;
-
-  if (bfd_get_format (abfd) != bfd_object)
-    return TRUE;
-
-#define FREE(x) if (x != NULL) { free (x); x = NULL; }
-  /* Free the native string and symbol tables.  */
-  FREE (obj_som_symtab (abfd));
-  FREE (obj_som_stringtab (abfd));
-  for (o = abfd->sections; o != NULL; o = o->next)
+  if (bfd_get_format (abfd) == bfd_object)
     {
-      /* Free the native relocations.  */
-      o->reloc_count = (unsigned) -1;
-      FREE (som_section_data (o)->reloc_stream);
-      /* Free the generic relocations.  */
-      FREE (o->relocation);
-    }
+      asection *o;
+
+#define FREE(x) do { free (x); x = NULL; } while (0)
+      /* Free the native string and symbol tables.  */
+      FREE (obj_som_symtab (abfd));
+      FREE (obj_som_stringtab (abfd));
+      for (o = abfd->sections; o != NULL; o = o->next)
+       {
+         /* Free the native relocations.  */
+         o->reloc_count = (unsigned) -1;
+         FREE (som_section_data (o)->reloc_stream);
+         /* Do not free the generic relocations as they are objalloc'ed.  */
+       }
 #undef FREE
+    }
 
-  return TRUE;
+  return _bfd_generic_close_and_cleanup (abfd);
 }
 
 /* End of miscellaneous support functions.  */
@@ -6296,39 +6798,50 @@ som_bfd_link_split_section (bfd *abfd ATTRIBUTE_UNUSED, asection *sec)
   return som_is_subspace (sec) && sec->size > 240000;
 }
 
-#define        som_close_and_cleanup                   som_bfd_free_cached_info
-#define som_read_ar_hdr                                _bfd_generic_read_ar_hdr
-#define som_openr_next_archived_file           bfd_generic_openr_next_archived_file
-#define som_get_elt_at_index                   _bfd_generic_get_elt_at_index
-#define som_generic_stat_arch_elt              bfd_generic_stat_arch_elt
-#define som_truncate_arname                    bfd_bsd_truncate_arname
-#define som_slurp_extended_name_table          _bfd_slurp_extended_name_table
-#define som_construct_extended_name_table       _bfd_archive_coff_construct_extended_name_table
-#define som_update_armap_timestamp             bfd_true
-#define som_bfd_is_target_special_symbol   ((bfd_boolean (*) (bfd *, asymbol *)) bfd_false)
-#define som_get_lineno                         _bfd_nosymbols_get_lineno
-#define som_bfd_make_debug_symbol              _bfd_nosymbols_bfd_make_debug_symbol
-#define som_read_minisymbols                   _bfd_generic_read_minisymbols
-#define som_minisymbol_to_symbol               _bfd_generic_minisymbol_to_symbol
-#define som_get_section_contents_in_window      _bfd_generic_get_section_contents_in_window
-#define som_bfd_get_relocated_section_contents  bfd_generic_get_relocated_section_contents
-#define som_bfd_relax_section                   bfd_generic_relax_section
-#define som_bfd_link_hash_table_create          _bfd_generic_link_hash_table_create
-#define som_bfd_link_hash_table_free            _bfd_generic_link_hash_table_free
-#define som_bfd_link_add_symbols                _bfd_generic_link_add_symbols
-#define som_bfd_link_just_syms                  _bfd_generic_link_just_syms
-#define som_bfd_final_link                      _bfd_generic_final_link
-#define som_bfd_gc_sections                    bfd_generic_gc_sections
-#define som_bfd_merge_sections                 bfd_generic_merge_sections
-#define som_bfd_is_group_section               bfd_generic_is_group_section
-#define som_bfd_discard_group                  bfd_generic_discard_group
-#define som_section_already_linked              _bfd_generic_section_already_linked
+#define som_find_line                          _bfd_nosymbols_find_line
+#define som_get_symbol_version_string          _bfd_nosymbols_get_symbol_version_string
+#define        som_close_and_cleanup                   som_bfd_free_cached_info
+#define som_read_ar_hdr                                _bfd_generic_read_ar_hdr
+#define som_write_ar_hdr                       _bfd_generic_write_ar_hdr
+#define som_openr_next_archived_file           bfd_generic_openr_next_archived_file
+#define som_get_elt_at_index                   _bfd_generic_get_elt_at_index
+#define som_generic_stat_arch_elt              bfd_generic_stat_arch_elt
+#define som_truncate_arname                    bfd_bsd_truncate_arname
+#define som_slurp_extended_name_table          _bfd_slurp_extended_name_table
+#define som_construct_extended_name_table      _bfd_archive_coff_construct_extended_name_table
+#define som_update_armap_timestamp             _bfd_bool_bfd_true
+#define som_bfd_is_target_special_symbol        _bfd_bool_bfd_asymbol_false
+#define som_get_lineno                         _bfd_nosymbols_get_lineno
+#define som_bfd_make_debug_symbol              _bfd_nosymbols_bfd_make_debug_symbol
+#define som_read_minisymbols                   _bfd_generic_read_minisymbols
+#define som_minisymbol_to_symbol               _bfd_generic_minisymbol_to_symbol
+#define som_get_section_contents_in_window     _bfd_generic_get_section_contents_in_window
+#define som_bfd_get_relocated_section_contents bfd_generic_get_relocated_section_contents
+#define som_bfd_relax_section                  bfd_generic_relax_section
+#define som_bfd_link_hash_table_create         _bfd_generic_link_hash_table_create
+#define som_bfd_link_add_symbols               _bfd_generic_link_add_symbols
+#define som_bfd_link_just_syms                 _bfd_generic_link_just_syms
+#define som_bfd_copy_link_hash_symbol_type \
+  _bfd_generic_copy_link_hash_symbol_type
+#define som_bfd_final_link                     _bfd_generic_final_link
+#define som_bfd_gc_sections                    bfd_generic_gc_sections
+#define som_bfd_lookup_section_flags           bfd_generic_lookup_section_flags
+#define som_bfd_merge_sections                 bfd_generic_merge_sections
+#define som_bfd_is_group_section               bfd_generic_is_group_section
+#define som_bfd_group_name                     bfd_generic_group_name
+#define som_bfd_discard_group                  bfd_generic_discard_group
+#define som_section_already_linked             _bfd_generic_section_already_linked
+#define som_bfd_define_common_symbol           bfd_generic_define_common_symbol
+#define som_bfd_link_hide_symbol               _bfd_generic_link_hide_symbol
+#define som_bfd_define_start_stop              bfd_generic_define_start_stop
 #define som_bfd_merge_private_bfd_data         _bfd_generic_bfd_merge_private_bfd_data
 #define som_bfd_copy_private_header_data       _bfd_generic_bfd_copy_private_header_data
 #define som_bfd_set_private_flags              _bfd_generic_bfd_set_private_flags
 #define som_find_inliner_info                  _bfd_nosymbols_find_inliner_info
+#define som_bfd_link_check_relocs              _bfd_generic_link_check_relocs
+#define som_set_reloc                          _bfd_generic_set_reloc
 
-const bfd_target som_vec =
+const bfd_target hppa_som_vec =
 {
   "som",                       /* Name.  */
   bfd_target_som_flavour,
@@ -6345,6 +6858,7 @@ const bfd_target som_vec =
   0,
   '/',                         /* AR_pad_char.  */
   14,                          /* AR_max_namelen.  */
+  0,                           /* match priority.  */
   bfd_getb64, bfd_getb_signed_64, bfd_putb64,
   bfd_getb32, bfd_getb_signed_32, bfd_putb32,
   bfd_getb16, bfd_getb_signed_16, bfd_putb16,  /* Data.  */
@@ -6357,16 +6871,16 @@ const bfd_target som_vec =
    _bfd_dummy_target
   },
   {
-    bfd_false,
+    _bfd_bool_bfd_false_error,
     som_mkobject,
     _bfd_generic_mkarchive,
-    bfd_false
+    _bfd_bool_bfd_false_error
   },
   {
-    bfd_false,
+    _bfd_bool_bfd_false_error,
     som_write_object_contents,
     _bfd_write_archive_contents,
-    bfd_false,
+    _bfd_bool_bfd_false_error,
   },
 #undef som
 
@@ -6385,4 +6899,3 @@ const bfd_target som_vec =
   NULL
 };
 
-#endif /* HOST_HPPAHPUX || HOST_HPPABSD || HOST_HPPAOSF */