* aoutx.h (NAME(aout,canonicalize_reloc)): Don't error out if
[deliverable/binutils-gdb.git] / bfd / som.c
index d399e0f11e3ff2f04b3bb0054b5d402773097dff..7c913b1cfa13eb23a252f1cf59e628773d807835 100644 (file)
--- a/bfd/som.c
+++ b/bfd/som.c
 /* Max number of SOMs to be found in an archive.  */
 #define SOM_LST_MODULE_LIMIT 1024
 
+/* Generic alignment macro.  */
+#define SOM_ALIGN(val, alignment) \
+  (((val) + (alignment) - 1) & ~((alignment) - 1))
+
 /* SOM allows any one of the four previous relocations to be reused
    with a "R_PREV_FIXUP" relocation entry.  Since R_PREV_FIXUP
    relocations are always a single byte, using a R_PREV_FIXUP instead
@@ -129,28 +133,33 @@ static boolean som_mkobject PARAMS ((bfd *));
 static bfd_target * som_object_setup PARAMS ((bfd *,
                                              struct header *,
                                              struct som_exec_auxhdr *));
-static asection * make_unique_section PARAMS ((bfd *, CONST char *, int));
 static boolean setup_sections PARAMS ((bfd *, struct header *));
 static bfd_target * som_object_p PARAMS ((bfd *));
 static boolean som_write_object_contents PARAMS ((bfd *));
 static boolean som_slurp_string_table PARAMS ((bfd *));
 static unsigned int som_slurp_symbol_table PARAMS ((bfd *));
-static unsigned int som_get_symtab_upper_bound PARAMS ((bfd *));
-static unsigned int som_canonicalize_reloc PARAMS ((bfd *, sec_ptr,
-                                                   arelent **, asymbol **));
-static unsigned int som_get_reloc_upper_bound PARAMS ((bfd *, sec_ptr));
+static long som_get_symtab_upper_bound PARAMS ((bfd *));
+static long som_canonicalize_reloc PARAMS ((bfd *, sec_ptr,
+                                           arelent **, asymbol **));
+static long som_get_reloc_upper_bound PARAMS ((bfd *, sec_ptr));
 static unsigned int som_set_reloc_info PARAMS ((unsigned char *, unsigned int,
                                                arelent *, asection *,
                                                asymbol **, boolean));
 static boolean som_slurp_reloc_table PARAMS ((bfd *, asection *,
                                              asymbol **, boolean));
-static unsigned int som_get_symtab PARAMS ((bfd *, asymbol **));
+static long som_get_symtab PARAMS ((bfd *, asymbol **));
 static asymbol * som_make_empty_symbol PARAMS ((bfd *));
 static void som_print_symbol PARAMS ((bfd *, PTR,
                                      asymbol *, bfd_print_symbol_type));
 static boolean som_new_section_hook PARAMS ((bfd *, asection *));
+static boolean som_bfd_copy_private_section_data PARAMS ((bfd *, asection *,
+                                                         bfd *, asection *));
+static boolean som_bfd_copy_private_bfd_data PARAMS ((bfd *, bfd *));
+static boolean som_bfd_is_local_label PARAMS ((bfd *, asymbol *));
 static boolean som_set_section_contents PARAMS ((bfd *, sec_ptr, PTR,
                                                 file_ptr, bfd_size_type));
+static boolean som_get_section_contents PARAMS ((bfd *, sec_ptr, PTR,
+                                                file_ptr, bfd_size_type));
 static boolean som_set_arch_mach PARAMS ((bfd *, enum bfd_architecture,
                                          unsigned long));
 static boolean som_find_nearest_line PARAMS ((bfd *, asection *,
@@ -159,8 +168,8 @@ static boolean som_find_nearest_line PARAMS ((bfd *, asection *,
                                              CONST char **,
                                              unsigned int *));
 static void som_get_symbol_info PARAMS ((bfd *, asymbol *, symbol_info *));
-static asection * som_section_from_subspace_index PARAMS ((bfd *, 
-                                                          unsigned int));
+static asection * bfd_section_from_som_symbol PARAMS ((bfd *, 
+                                       struct symbol_dictionary_record *));
 static int log2 PARAMS ((unsigned int));
 static bfd_reloc_status_type hppa_som_reloc PARAMS ((bfd *, arelent *,
                                                     asymbol *, PTR,
@@ -213,7 +222,6 @@ static boolean som_bfd_fill_in_ar_symbols PARAMS ((bfd *, struct lst_header *,
                                                   carsym **syms));
 static boolean som_slurp_armap PARAMS ((bfd *));
 static boolean som_write_armap PARAMS ((bfd *));
-static boolean som_slurp_extended_name_table PARAMS ((bfd *));
 static void som_bfd_derive_misc_symbol_info PARAMS ((bfd *, asymbol *,
                                             struct som_misc_symbol_info *));
 static boolean som_bfd_prep_for_ar_write PARAMS ((bfd *, unsigned int *,
@@ -222,6 +230,10 @@ static unsigned int som_bfd_ar_symbol_hash PARAMS ((asymbol *));
 static boolean som_bfd_ar_write_symbol_stuff PARAMS ((bfd *, unsigned int,
                                                      unsigned int,
                                                      struct lst_header));
+static CONST char *normalize PARAMS ((CONST char *file));
+static boolean som_is_space PARAMS ((asection *));
+static boolean som_is_subspace PARAMS ((asection *));
+static boolean som_is_container PARAMS ((asection *, asection *));
        
 /* Map SOM section names to POSIX/BSD single-character symbol types.
 
@@ -955,7 +967,6 @@ static reloc_howto_type som_hppa_howto_table[] =
   {R_RESERVED, 0, 0, 32, false, 0, 0, hppa_som_reloc, "R_RESERVED"},
   {R_RESERVED, 0, 0, 32, false, 0, 0, hppa_som_reloc, "R_RESERVED"}};
   
-
 /* Initialize the SOM relocation queue.  By definition the queue holds
    the last four multibyte fixups.  */
   
@@ -1585,6 +1596,16 @@ som_object_setup (abfd, file_hdrp, aux_hdrp)
   obj_som_str_filepos (abfd) = file_hdrp->symbol_strings_location;
   obj_som_reloc_filepos (abfd) = file_hdrp->fixup_request_location;
 
+  obj_som_exec_data (abfd) = (struct som_exec_data *)
+    bfd_zalloc (abfd, sizeof (struct som_exec_data ));
+  if (obj_som_exec_data (abfd) == NULL)
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return NULL;
+    }
+
+  obj_som_exec_data (abfd)->system_id = file_hdrp->system_id;
+  obj_som_exec_data (abfd)->exec_flags = aux_hdrp->exec_flags;
   return abfd->xvec;
 }
 
@@ -1606,7 +1627,7 @@ setup_sections (abfd, file_hdr)
   /* First, read in space names */
 
   space_strings = malloc (file_hdr->space_strings_size);
-  if (!space_strings)
+  if (!space_strings && file_hdr->space_strings_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -1651,9 +1672,10 @@ setup_sections (abfd, file_hdr)
        space_asect->flags |= SEC_DEBUGGING;
 
       /* Set up all the attributes for the space.  */
-      bfd_som_set_section_attributes (space_asect, space.is_defined,
-                                     space.is_private, space.sort_key,
-                                     space.space_number);
+      if (bfd_som_set_section_attributes (space_asect, space.is_defined,
+                                         space.is_private, space.sort_key,
+                                         space.space_number) == false)
+       goto error_return;
 
       /* Now, read in the first subspace for this space */
       if (bfd_seek (abfd, file_hdr->subspace_location
@@ -1704,14 +1726,14 @@ setup_sections (abfd, file_hdr)
            goto error_return;
 
          /* Store private information about the section.  */
-         bfd_som_set_subsection_attributes (subspace_asect, space_asect,
-                                            subspace.access_control_bits,
-                                            subspace.sort_key,
-                                            subspace.quadrant);
+         if (bfd_som_set_subsection_attributes (subspace_asect, space_asect,
+                                                subspace.access_control_bits,
+                                                subspace.sort_key,
+                                                subspace.quadrant) == false)
+           goto error_return;
 
          /* Keep an easy mapping between subspaces and sections.  */
-         som_section_data (subspace_asect)->subspace_index 
-           = total_subspaces++;
+         subspace_asect->target_index = total_subspaces++;
 
          /* Set SEC_READONLY and SEC_CODE/SEC_DATA as specified
             by the access_control_bits in the subspace header.  */
@@ -1825,7 +1847,8 @@ som_object_p (abfd)
 
   if (bfd_read ((PTR) & file_hdr, 1, FILE_HDR_SIZE, abfd) != FILE_HDR_SIZE)
     {
-      bfd_set_error (bfd_error_system_call);
+      if (bfd_get_error () != bfd_error_system_call)
+       bfd_set_error (bfd_error_wrong_format);
       return 0;
     }
 
@@ -1874,7 +1897,8 @@ som_object_p (abfd)
     {
       if (bfd_read ((PTR) & aux_hdr, 1, AUX_HDR_SIZE, abfd) != AUX_HDR_SIZE)
        {
-         bfd_set_error (bfd_error_wrong_format);
+         if (bfd_get_error () != bfd_error_system_call)
+           bfd_set_error (bfd_error_wrong_format);
          return 0;
        }
     }
@@ -1900,14 +1924,6 @@ som_mkobject (abfd)
   abfd->tdata.som_data = (struct som_data_struct *)
     bfd_zalloc (abfd, sizeof (struct som_data_struct));
   if (abfd->tdata.som_data == NULL)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return false;
-    }
-  obj_som_file_hdr (abfd)
-    = (struct header *) bfd_zalloc (abfd, sizeof (struct header));
-  if (obj_som_file_hdr (abfd) == NULL)
-
     {
       bfd_set_error (bfd_error_no_memory);
       return false;
@@ -1923,12 +1939,25 @@ static boolean
 som_prep_headers (abfd)
      bfd *abfd;
 {
-  struct header *file_hdr = obj_som_file_hdr (abfd);
+  struct header *file_hdr;
   asection *section;
 
+  /* Make and attach a file header to the BFD.  */
+  file_hdr = (struct header *) bfd_zalloc (abfd, sizeof (struct header));
+  if (file_hdr == NULL)
+
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return false;
+    }
+  obj_som_file_hdr (abfd) = file_hdr;
+
   /* FIXME.  This should really be conditional based on whether or not
      PA1.1 instructions/registers have been used.  */
-  file_hdr->system_id = CPU_PA_RISC1_0;
+  if (abfd->flags & EXEC_P)
+    file_hdr->system_id = obj_som_exec_data (abfd)->system_id;
+  else
+    file_hdr->system_id = CPU_PA_RISC1_0;
 
   if (abfd->flags & EXEC_P)
     {
@@ -1951,15 +1980,9 @@ som_prep_headers (abfd)
   file_hdr->file_time.secs = 0;
   file_hdr->file_time.nanosecs = 0; 
 
-  if (abfd->flags & EXEC_P)
-    abort ();
-  else
-    {
-      file_hdr->entry_space = 0;
-      file_hdr->entry_subspace = 0;
-      file_hdr->entry_offset = 0;
-    }
-  
+  file_hdr->entry_space = 0;
+  file_hdr->entry_subspace = 0;
+  file_hdr->entry_offset = 0;
   file_hdr->presumed_dp = 0;
 
   /* Now iterate over the sections translating information from
@@ -1969,48 +1992,134 @@ som_prep_headers (abfd)
     {
       /* Ignore anything which has not been marked as a space or
         subspace.  */
-      if (som_section_data (section)->is_space == 0
-
-         && som_section_data (section)->is_subspace == 0)
+      if (!som_is_space (section) && !som_is_subspace (section))
        continue;
-
-      if (som_section_data (section)->is_space)
+      
+      if (som_is_space (section))
        {
+         /* Allocate space for the space dictionary.  */
+         som_section_data (section)->space_dict
+           = (struct space_dictionary_record *)
+             bfd_zalloc (abfd, sizeof (struct space_dictionary_record));
+         if (som_section_data (section)->space_dict == NULL)
+           {
+             bfd_set_error (bfd_error_no_memory);
+             return false;
+           }
          /* Set space attributes.  Note most attributes of SOM spaces
             are set based on the subspaces it contains.  */
-         som_section_data (section)->space_dict.loader_fix_index = -1;
-         som_section_data (section)->space_dict.init_pointer_index = -1;
+         som_section_data (section)->space_dict->loader_fix_index = -1;
+         som_section_data (section)->space_dict->init_pointer_index = -1;
+
+         /* Set more attributes that were stuffed away in private data.  */
+         som_section_data (section)->space_dict->sort_key = 
+           som_section_data (section)->copy_data->sort_key;
+         som_section_data (section)->space_dict->is_defined = 
+           som_section_data (section)->copy_data->is_defined;
+         som_section_data (section)->space_dict->is_private = 
+           som_section_data (section)->copy_data->is_private;
+         som_section_data (section)->space_dict->space_number =
+           section->target_index;
        }
       else
        {
+         /* Allocate space for the subspace dictionary.  */
+         som_section_data (section)->subspace_dict
+           = (struct subspace_dictionary_record *)
+             bfd_zalloc (abfd, sizeof (struct subspace_dictionary_record));
+         if (som_section_data (section)->subspace_dict == NULL)
+           {
+             bfd_set_error (bfd_error_no_memory);
+             return false;
+           }
+
          /* Set subspace attributes.  Basic stuff is done here, additional
             attributes are filled in later as more information becomes
             available.  */
          if (section->flags & SEC_IS_COMMON)
            {
-             som_section_data (section)->subspace_dict.dup_common = 1;
-             som_section_data (section)->subspace_dict.is_common = 1;
+             som_section_data (section)->subspace_dict->dup_common = 1;
+             som_section_data (section)->subspace_dict->is_common = 1;
            }
 
          if (section->flags & SEC_ALLOC)
-           som_section_data (section)->subspace_dict.is_loadable = 1;
+           som_section_data (section)->subspace_dict->is_loadable = 1;
 
          if (section->flags & SEC_CODE)
-           som_section_data (section)->subspace_dict.code_only = 1;
+           som_section_data (section)->subspace_dict->code_only = 1;
 
-         som_section_data (section)->subspace_dict.subspace_start = 
+         som_section_data (section)->subspace_dict->subspace_start = 
            section->vma;
-         som_section_data (section)->subspace_dict.subspace_length =
+         som_section_data (section)->subspace_dict->subspace_length =
            bfd_section_size (abfd, section);
-         som_section_data (section)->subspace_dict.initialization_length =
+         som_section_data (section)->subspace_dict->initialization_length =
            bfd_section_size (abfd, section);
-         som_section_data (section)->subspace_dict.alignment = 
+         som_section_data (section)->subspace_dict->alignment = 
            1 << section->alignment_power;
+
+         /* Set more attributes that were stuffed away in private data.  */
+         som_section_data (section)->subspace_dict->sort_key =
+           som_section_data (section)->copy_data->sort_key;
+         som_section_data (section)->subspace_dict->access_control_bits =
+           som_section_data (section)->copy_data->access_control_bits;
+         som_section_data (section)->subspace_dict->quadrant =
+           som_section_data (section)->copy_data->quadrant;
        }
     }
   return true;
 }
 
+/* Return true if the given section is a SOM space, false otherwise.  */
+
+static boolean
+som_is_space (section)
+     asection *section;
+{
+  /* If no copy data is available, then it's neither a space nor a
+     subspace.  */
+  if (som_section_data (section)->copy_data == NULL)
+    return false;
+
+  /* If the containing space isn't the same as the given section,
+     then this isn't a space.  */
+  if (som_section_data (section)->copy_data->container != section)
+    return false;
+
+  /* OK.  Must be a space.  */
+  return true;
+}
+
+/* Return true if the given section is a SOM subspace, false otherwise.  */
+
+static boolean
+som_is_subspace (section)
+     asection *section;
+{
+  /* If no copy data is available, then it's neither a space nor a
+     subspace.  */
+  if (som_section_data (section)->copy_data == NULL)
+    return false;
+
+  /* If the containing space is the same as the given section,
+     then this isn't a subspace.  */
+  if (som_section_data (section)->copy_data->container == section)
+    return false;
+
+  /* OK.  Must be a subspace.  */
+  return true;
+}
+
+/* Return true if the given space containins the given subspace.  It
+   is safe to assume space really is a space, and subspace really
+   is a subspace.  */
+
+static boolean
+som_is_container (space, subspace)
+     asection *space, *subspace;
+{
+  return som_section_data (subspace)->copy_data->container == space;
+}
+
 /* Count and return the number of spaces attached to the given BFD.  */
 
 static unsigned long
@@ -2021,7 +2130,7 @@ som_count_spaces (abfd)
   asection *section;
 
   for (section = abfd->sections; section != NULL; section = section->next)
-    count += som_section_data (section)->is_space;
+      count += som_is_space (section);
 
   return count;
 }
@@ -2036,7 +2145,7 @@ som_count_subspaces (abfd)
   asection *section;
 
   for (section = abfd->sections; section != NULL; section = section->next)
-    count += som_section_data (section)->is_subspace;
+    count += som_is_subspace (section);
 
   return count;
 }
@@ -2095,15 +2204,11 @@ som_prep_for_fixups (abfd, syms, num_syms)
     {
       /* Handle a section symbol; these have no pointers back to the 
         SOM symbol info.  So we just use the pointer field (udata)
-        to hold the relocation count.
-
-        FIXME.  While we're here set the name of any section symbol
-        to something which will not screw GDB.  How do other formats
-        deal with this?!?  */
-      if (som_symbol_data (syms[i]) == NULL)
+        to hold the relocation count.  */
+      if (som_symbol_data (syms[i]) == NULL
+         || syms[i]->flags & BSF_SECTION_SYM)
        {
          syms[i]->flags |= BSF_SECTION_SYM;
-         syms[i]->name = "L$0\002";
          syms[i]->udata = (PTR) 0;
        }
       else
@@ -2205,7 +2310,7 @@ som_write_fixups (abfd, current_offset, total_reloc_sizep)
       asection *subsection;
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
       /* Now iterate through each of its subspaces.  */
@@ -2216,33 +2321,30 @@ som_write_fixups (abfd, current_offset, total_reloc_sizep)
          int reloc_offset, current_rounding_mode;
 
          /* Find a subspace of this space.  */
-         if (som_section_data (subsection)->is_subspace == 0
-             || som_section_data (subsection)->containing_space != section)
+         if (!som_is_subspace (subsection)
+             || !som_is_container (section, subsection))
            continue;
 
          /* If this subspace had no relocations, then we're finished 
             with it.  */
          if (subsection->reloc_count <= 0)
            {
-             som_section_data (subsection)->subspace_dict.fixup_request_index
+             som_section_data (subsection)->subspace_dict->fixup_request_index
                = -1;
              continue;
            }
 
          /* This subspace has some relocations.  Put the relocation stream
             index into the subspace record.  */
-         som_section_data (subsection)->subspace_dict.fixup_request_index
+         som_section_data (subsection)->subspace_dict->fixup_request_index
            = total_reloc_size;
 
          /* To make life easier start over with a clean slate for 
             each subspace.  Seek to the start of the relocation stream
             for this subspace in preparation for writing out its fixup
             stream.  */
-         if (bfd_seek (abfd, current_offset + total_reloc_size, SEEK_SET) != 0)
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+         if (bfd_seek (abfd, current_offset + total_reloc_size, SEEK_SET) < 0)
+           return false;
 
          /* Buffer space has already been allocated.  Just perform some
             initialization here.  */
@@ -2278,10 +2380,8 @@ som_write_fixups (abfd, current_offset, total_reloc_sizep)
                {
                  if (bfd_write ((PTR) tmp_space, p - tmp_space, 1, abfd)
                      != p - tmp_space)
-                   {
-                     bfd_set_error (bfd_error_system_call);
-                     return false;
-                   }
+                   return false;
+
                  p = tmp_space;
                  som_initialize_reloc_queue (reloc_queue);
                }
@@ -2448,14 +2548,11 @@ som_write_fixups (abfd, current_offset, total_reloc_sizep)
          /* Scribble out the relocations.  */
          if (bfd_write ((PTR) tmp_space, p - tmp_space, 1, abfd)
              != p - tmp_space)
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+           return false;
          p = tmp_space;
 
          total_reloc_size += subspace_reloc_size;
-         som_section_data (subsection)->subspace_dict.fixup_request_quantity
+         som_section_data (subsection)->subspace_dict->fixup_request_quantity
            = subspace_reloc_size;
        }
       section = section->next;
@@ -2484,11 +2581,8 @@ som_write_space_strings (abfd, current_offset, string_sizep)
 
   /* Seek to the start of the space strings in preparation for writing
      them out.  */
-  if (bfd_seek (abfd, current_offset, SEEK_SET) != 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+  if (bfd_seek (abfd, current_offset, SEEK_SET) < 0)
+    return false;
 
   /* Walk through all the spaces and subspaces (order is not important)
      building up and writing string table entries for their names.  */
@@ -2498,8 +2592,7 @@ som_write_space_strings (abfd, current_offset, string_sizep)
 
       /* Only work with space/subspaces; avoid any other sections
         which might have been made (.text for example).  */
-      if (som_section_data (section)->is_space == 0
-         && som_section_data (section)->is_subspace == 0)
+      if (!som_is_space (section) && !som_is_subspace (section))
        continue;
 
       /* Get the length of the space/subspace name.  */
@@ -2512,10 +2605,7 @@ som_write_space_strings (abfd, current_offset, string_sizep)
        {
          if (bfd_write ((PTR) &tmp_space[0], p - tmp_space, 1, abfd)
              != p - tmp_space) 
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+           return false;
          /* Reset to beginning of the buffer space.  */
          p = tmp_space;
        }
@@ -2527,10 +2617,10 @@ som_write_space_strings (abfd, current_offset, string_sizep)
       strings_size += 4;
 
       /* Record the index in the space/subspace records.  */
-      if (som_section_data (section)->is_space)
-       som_section_data (section)->space_dict.name.n_strx = strings_size;
+      if (som_is_space (section))
+       som_section_data (section)->space_dict->name.n_strx = strings_size;
       else
-       som_section_data (section)->subspace_dict.name.n_strx = strings_size;
+       som_section_data (section)->subspace_dict->name.n_strx = strings_size;
 
       /* Next comes the string itself + a null terminator.  */
       strcpy (p, section->name);
@@ -2549,10 +2639,7 @@ som_write_space_strings (abfd, current_offset, string_sizep)
   /* Done with the space/subspace strings.  Write out any information
      contained in a partial block.  */
   if (bfd_write ((PTR) &tmp_space[0], p - tmp_space, 1, abfd) != p - tmp_space)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
   *string_sizep = strings_size;
   return true;
 }
@@ -2580,11 +2667,8 @@ som_write_symbol_strings (abfd, current_offset, syms, num_syms, string_sizep)
 
   /* Seek to the start of the space strings in preparation for writing
      them out.  */
-  if (bfd_seek (abfd, current_offset, SEEK_SET) != 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+  if (bfd_seek (abfd, current_offset, SEEK_SET) < 0)
+    return false;
 
   for (i = 0; i < num_syms; i++)
     {
@@ -2596,10 +2680,7 @@ som_write_symbol_strings (abfd, current_offset, syms, num_syms, string_sizep)
        {
          if (bfd_write ((PTR) &tmp_space[0], p - tmp_space, 1, abfd)
              != p - tmp_space)
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+           return false;
          /* Reset to beginning of the buffer space.  */
          p = tmp_space;
        }
@@ -2631,10 +2712,7 @@ som_write_symbol_strings (abfd, current_offset, syms, num_syms, string_sizep)
 
   /* Scribble out any partial block.  */
   if (bfd_write ((PTR) &tmp_space[0], p - tmp_space, 1, abfd) != p - tmp_space)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   *string_sizep = strings_size;
   return true;
@@ -2655,6 +2733,7 @@ som_begin_writing (abfd)
   asection *section;
   asymbol **syms = bfd_get_outsymbols (abfd);
   unsigned int total_subspaces = 0;
+  struct som_exec_auxhdr exec_header;
 
   /* The file header will always be first in an object file, 
      everything else can be in random locations.  To keep things
@@ -2674,21 +2753,30 @@ som_begin_writing (abfd)
      we support only the copyright and version headers.  */
   obj_som_file_hdr (abfd)->aux_header_location = current_offset;
   obj_som_file_hdr (abfd)->aux_header_size = 0;
+  if (abfd->flags & EXEC_P)
+    {
+      /* 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 (exec_header);
+      obj_som_file_hdr (abfd)->aux_header_size += sizeof (exec_header);
+      memset (&exec_header, 0, sizeof (exec_header));
+      exec_header.som_auxhdr.type = HPUX_AUX_ID;
+      exec_header.som_auxhdr.length = 40;
+    }
   if (obj_som_version_hdr (abfd) != NULL)
     {
       unsigned int len;
 
-      bfd_seek (abfd, current_offset, SEEK_SET);
+      if (bfd_seek (abfd, current_offset, SEEK_SET) < 0)
+       return false;
 
       /* Write the aux_id structure and the string length.  */
       len = sizeof (struct aux_id) + sizeof (unsigned int);
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
       if (bfd_write ((PTR) obj_som_version_hdr (abfd), len, 1, abfd) != len)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
 
       /* Write the version string.  */
       len = obj_som_version_hdr (abfd)->header_id.length - sizeof (int);
@@ -2696,27 +2784,22 @@ som_begin_writing (abfd)
       current_offset += len;
       if (bfd_write ((PTR) obj_som_version_hdr (abfd)->user_string,
                     len, 1, abfd) != len)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
     }
 
   if (obj_som_copyright_hdr (abfd) != NULL)
     {
       unsigned int len;
 
-      bfd_seek (abfd, current_offset, SEEK_SET);
+      if (bfd_seek (abfd, current_offset, SEEK_SET) < 0)
+       return false;
 
       /* Write the aux_id structure and the string length.  */
       len = sizeof (struct aux_id) + sizeof (unsigned int);
       obj_som_file_hdr (abfd)->aux_header_size += len;
       current_offset += len;
       if (bfd_write ((PTR) obj_som_copyright_hdr (abfd), len, 1, abfd) != len)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
 
       /* Write the copyright string.  */
       len = obj_som_copyright_hdr (abfd)->header_id.length - sizeof (int);
@@ -2724,10 +2807,7 @@ som_begin_writing (abfd)
       current_offset += len;
       if (bfd_write ((PTR) obj_som_copyright_hdr (abfd)->copyright,
                     len, 1, abfd) != len)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
     }
 
   /* Next comes the initialization pointers; we have no initialization
@@ -2834,43 +2914,122 @@ som_begin_writing (abfd)
   obj_som_file_hdr (abfd)->compiler_location = current_offset;
   obj_som_file_hdr (abfd)->compiler_total = 0;
 
-  /* Now compute the file positions for the loadable subspaces.  */
+  /* Now compute the file positions for the loadable subspaces, taking
+     care to make sure everything stays properly aligned.  */
 
   section = abfd->sections;
   for (i = 0; i < num_spaces; i++)
     {
       asection *subsection;
+      int first_subspace;
+      unsigned int subspace_offset = 0;
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
+      first_subspace = 1;
       /* Now look for all its subspaces.  */
       for (subsection = abfd->sections;
           subsection != NULL;
           subsection = subsection->next)
        {
-         
-         if (som_section_data (subsection)->is_subspace == 0
-             || som_section_data (subsection)->containing_space != section
+
+         if (!som_is_subspace (subsection)
+             || !som_is_container (section, subsection)
              || (subsection->flags & SEC_ALLOC) == 0)
            continue;
 
-         som_section_data (subsection)->subspace_index = total_subspaces++;
+         /* If this is the first subspace in the space, and we are
+            building an executable, then take care to make sure all
+            the alignments are correct and update the exec header.  */
+         if (first_subspace
+             && (abfd->flags & EXEC_P))
+           {
+             /* Demand paged executables have each space aligned to a
+                page boundary.  Sharable executables (write-protected
+                text) have just the private (aka data & bss) space aligned
+                to a page boundary.  Ugh.  Not true for HPUX.
+
+                The HPUX kernel requires the text to always be page aligned
+                within the file regardless of the executable's type.  */
+             if (abfd->flags & D_PAGED
+                 || (subsection->flags & SEC_CODE)
+                 || ((abfd->flags & WP_TEXT)
+                     && (subsection->flags & SEC_DATA)))
+               current_offset = SOM_ALIGN (current_offset, PA_PAGESIZE);
+
+             /* Update the exec header.  */
+             if (subsection->flags & SEC_CODE && exec_header.exec_tfile == 0)
+               {
+                 exec_header.exec_tmem = section->vma;
+                 exec_header.exec_tfile = current_offset;
+               }
+             if (subsection->flags & SEC_DATA && exec_header.exec_dfile == 0)
+               {
+                 exec_header.exec_dmem = section->vma;
+                 exec_header.exec_dfile = current_offset;
+               }
+
+             /* Keep track of exactly where we are within a particular
+                space.  This is necessary as the braindamaged HPUX
+                loader will create holes between subspaces *and* 
+                subspace alignments are *NOT* preserved.  What a crock.  */
+             subspace_offset = subsection->vma;
+
+             /* Only do this for the first subspace within each space.  */
+             first_subspace = 0;
+           }
+         else if (abfd->flags & EXEC_P)
+           {
+             /* The braindamaged HPUX loader may have created a hole
+                between two subspaces.  It is *not* sufficient to use
+                the alignment specifications within the subspaces to
+                account for these holes -- I've run into at least one
+                case where the loader left one code subspace unaligned
+                in a final executable.
+
+                To combat this we keep a current offset within each space,
+                and use the subspace vma fields to detect and preserve
+                holes.  What a crock!
+
+                ps.  This is not necessary for unloadable space/subspaces.  */
+             current_offset += subsection->vma - subspace_offset;
+             if (subsection->flags & SEC_CODE)
+               exec_header.exec_tsize += subsection->vma - subspace_offset;
+             else
+               exec_header.exec_dsize += subsection->vma - subspace_offset;
+             subspace_offset += subsection->vma - subspace_offset;
+           }
+
+
+         subsection->target_index = total_subspaces++;
          /* This is real data to be loaded from the file.  */
          if (subsection->flags & SEC_LOAD)
            {
-             som_section_data (subsection)->subspace_dict.file_loc_init_value
+             /* Update the size of the code & data.  */
+             if (abfd->flags & EXEC_P
+                 && subsection->flags & SEC_CODE)
+               exec_header.exec_tsize += subsection->_cooked_size;
+             else if (abfd->flags & EXEC_P
+                      && subsection->flags & SEC_DATA)
+               exec_header.exec_dsize += subsection->_cooked_size;
+             som_section_data (subsection)->subspace_dict->file_loc_init_value
                = current_offset;
-             section->filepos = current_offset;
+             subsection->filepos = current_offset;
              current_offset += bfd_section_size (abfd, subsection); 
+             subspace_offset += bfd_section_size (abfd, subsection);
            }
          /* Looks like uninitialized data.  */
          else
            {
-             som_section_data (subsection)->subspace_dict.file_loc_init_value
+             /* Update the size of the bss section.  */
+             if (abfd->flags & EXEC_P)
+               exec_header.exec_bsize += subsection->_cooked_size;
+
+             som_section_data (subsection)->subspace_dict->file_loc_init_value
                = 0;
-             som_section_data (subsection)->subspace_dict.
+             som_section_data (subsection)->subspace_dict->
                initialization_length = 0;
            }
        }
@@ -2878,7 +3037,12 @@ som_begin_writing (abfd)
       section = section->next; 
     }
 
-  /* Finally compute the file positions for unloadable subspaces.  */
+  /* Finally compute the file positions for unloadable subspaces.
+     If building an executable, start the unloadable stuff on its
+     own page.  */
+
+  if (abfd->flags & EXEC_P)
+    current_offset = SOM_ALIGN (current_offset, PA_PAGESIZE);
 
   obj_som_file_hdr (abfd)->unloadable_sp_location = current_offset;
   section = abfd->sections;
@@ -2887,35 +3051,38 @@ som_begin_writing (abfd)
       asection *subsection;
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
+      if (abfd->flags & EXEC_P)
+       current_offset = SOM_ALIGN (current_offset, PA_PAGESIZE);
+
       /* Now look for all its subspaces.  */
       for (subsection = abfd->sections;
           subsection != NULL;
           subsection = subsection->next)
        {
          
-         if (som_section_data (subsection)->is_subspace == 0
-             || som_section_data (subsection)->containing_space != section
+         if (!som_is_subspace (subsection)
+             || !som_is_container (section, subsection)
              || (subsection->flags & SEC_ALLOC) != 0)
            continue;
 
-         som_section_data (subsection)->subspace_index = total_subspaces++;
+         subsection->target_index = total_subspaces;
          /* This is real data to be loaded from the file.  */
          if ((subsection->flags & SEC_LOAD) == 0)
            {
-             som_section_data (subsection)->subspace_dict.file_loc_init_value
+             som_section_data (subsection)->subspace_dict->file_loc_init_value
                = current_offset;
-             section->filepos = current_offset;
+             subsection->filepos = current_offset;
              current_offset += bfd_section_size (abfd, subsection); 
            }
          /* Looks like uninitialized data.  */
          else
            {
-             som_section_data (subsection)->subspace_dict.file_loc_init_value
+             som_section_data (subsection)->subspace_dict->file_loc_init_value
                = 0;
-             som_section_data (subsection)->subspace_dict.
+             som_section_data (subsection)->subspace_dict->
                initialization_length = bfd_section_size (abfd, subsection);
            }
        }
@@ -2923,6 +3090,16 @@ som_begin_writing (abfd)
       section = section->next; 
     }
 
+  /* If building an executable, then make sure to seek to and write
+     one byte at the end of the file to make sure any necessary
+     zeros are filled in.  Ugh.  */
+  if (abfd->flags & EXEC_P)
+    current_offset = SOM_ALIGN (current_offset, PA_PAGESIZE);
+  if (bfd_seek (abfd, current_offset - 1, SEEK_SET) < 0)
+    return false;
+  if (bfd_write ((PTR) "", 1, 1, abfd) != 1)
+    return false;
+
   obj_som_file_hdr (abfd)->unloadable_sp_size
     = current_offset - obj_som_file_hdr (abfd)->unloadable_sp_location;
 
@@ -2932,6 +3109,33 @@ som_begin_writing (abfd)
 
   /* Done.  Store the total size of the SOM.  */
   obj_som_file_hdr (abfd)->som_length = current_offset;
+
+  /* Now write the exec header.  */
+  if (abfd->flags & EXEC_P)
+    {
+      long tmp;
+
+      exec_header.exec_entry = bfd_get_start_address (abfd);
+      exec_header.exec_flags = obj_som_exec_data (abfd)->exec_flags;
+
+      /* Oh joys.  Ram some of the BSS data into the DATA section
+        to be compatable with how the hp linker makes objects
+        (saves memory space).  */
+      tmp = exec_header.exec_dsize;
+      tmp = SOM_ALIGN (tmp, PA_PAGESIZE);
+      exec_header.exec_bsize -= (tmp - exec_header.exec_dsize);
+      if (exec_header.exec_bsize < 0)
+       exec_header.exec_bsize = 0;
+      exec_header.exec_dsize = tmp;
+
+      if (bfd_seek (abfd, obj_som_file_hdr (abfd)->aux_header_location,
+                   SEEK_SET) < 0)
+       return false;
+
+      if (bfd_write ((PTR) &exec_header, AUX_HDR_SIZE, 1, abfd)
+         != AUX_HDR_SIZE)
+       return false;
+    }
   return true;
 }
 
@@ -2952,7 +3156,9 @@ som_write_headers (abfd)
 
   /* Seek to the start of the subspace dictionary records.  */
   location = obj_som_file_hdr (abfd)->subspace_location;
-  bfd_seek (abfd, location, SEEK_SET);
+  if (bfd_seek (abfd, location, SEEK_SET) < 0)
+    return false;
+
   section = abfd->sections;
   /* Now for each loadable space write out records for its subspaces.  */
   for (i = 0; i < num_spaces; i++)
@@ -2960,7 +3166,7 @@ som_write_headers (abfd)
       asection *subsection;
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
       /* Now look for all its subspaces.  */
@@ -2972,8 +3178,8 @@ som_write_headers (abfd)
          /* Skip any section which does not correspond to a space
             or subspace.  Or does not have SEC_ALLOC set (and therefore
             has no real bits on the disk).  */
-         if (som_section_data (subsection)->is_subspace == 0
-             || som_section_data (subsection)->containing_space != section
+         if (!som_is_subspace (subsection)
+             || !som_is_container (section, subsection)
              || (subsection->flags & SEC_ALLOC) == 0)
            continue;
 
@@ -2981,30 +3187,27 @@ som_write_headers (abfd)
             the index of the subspace in its containing space.  Also
             set "is_loadable" in the containing space.  */
 
-         if (som_section_data (section)->space_dict.subspace_quantity == 0)
+         if (som_section_data (section)->space_dict->subspace_quantity == 0)
            {
-             som_section_data (section)->space_dict.is_loadable = 1;
-             som_section_data (section)->space_dict.subspace_index
+             som_section_data (section)->space_dict->is_loadable = 1;
+             som_section_data (section)->space_dict->subspace_index
                = subspace_index;
            }
 
          /* Increment the number of subspaces seen and the number of
             subspaces contained within the current space.  */
          subspace_index++;
-         som_section_data (section)->space_dict.subspace_quantity++;
+         som_section_data (section)->space_dict->subspace_quantity++;
 
          /* Mark the index of the current space within the subspace's
             dictionary record.  */
-         som_section_data (subsection)->subspace_dict.space_index = i;
+         som_section_data (subsection)->subspace_dict->space_index = i;
          
          /* Dump the current subspace header.  */
-         if (bfd_write ((PTR) &som_section_data (subsection)->subspace_dict,
+         if (bfd_write ((PTR) som_section_data (subsection)->subspace_dict,
                         sizeof (struct subspace_dictionary_record), 1, abfd)
              != sizeof (struct subspace_dictionary_record))
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+           return false;
        }
       /* Goto the next section.  */
       section = section->next; 
@@ -3018,7 +3221,7 @@ som_write_headers (abfd)
       asection *subsection;
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
       /* Now look for all its subspaces.  */
@@ -3029,10 +3232,10 @@ som_write_headers (abfd)
          
          /* Skip any section which does not correspond to a space or
             subspace, or which SEC_ALLOC set (and therefore handled
-            in the loadable spaces/subspaces code above.  */
+            in the loadable spaces/subspaces code above).  */
 
-         if (som_section_data (subsection)->is_subspace == 0
-             || som_section_data (subsection)->containing_space != section
+         if (!som_is_subspace (subsection)
+             || !som_is_container (section, subsection)
              || (subsection->flags & SEC_ALLOC) != 0)
            continue;
 
@@ -3040,30 +3243,27 @@ som_write_headers (abfd)
             the index of the subspace in its containing space.  Clear
             "is_loadable".  */
 
-         if (som_section_data (section)->space_dict.subspace_quantity == 0)
+         if (som_section_data (section)->space_dict->subspace_quantity == 0)
            {
-             som_section_data (section)->space_dict.is_loadable = 0;
-             som_section_data (section)->space_dict.subspace_index
+             som_section_data (section)->space_dict->is_loadable = 0;
+             som_section_data (section)->space_dict->subspace_index
                = subspace_index;
            }
 
          /* Increment the number of subspaces seen and the number of
             subspaces contained within the current space.  */
-         som_section_data (section)->space_dict.subspace_quantity++;
+         som_section_data (section)->space_dict->subspace_quantity++;
          subspace_index++; 
 
          /* Mark the index of the current space within the subspace's
             dictionary record.  */
-         som_section_data (subsection)->subspace_dict.space_index = i;
+         som_section_data (subsection)->subspace_dict->space_index = i;
          
          /* Dump this subspace header.  */
-         if (bfd_write ((PTR) &som_section_data (subsection)->subspace_dict,
+         if (bfd_write ((PTR) som_section_data (subsection)->subspace_dict,
                         sizeof (struct subspace_dictionary_record), 1, abfd)
              != sizeof (struct subspace_dictionary_record))
-           {
-             bfd_set_error (bfd_error_system_call);
-             return false;
-           }
+           return false;
        }
       /* Goto the next section.  */
       section = section->next; 
@@ -3075,24 +3275,22 @@ som_write_headers (abfd)
      Seek to the right location and start writing the space
      dictionary records.  */
   location = obj_som_file_hdr (abfd)->space_location;
-  bfd_seek (abfd, location, SEEK_SET);
+  if (bfd_seek (abfd, location, SEEK_SET) < 0)
+    return false;
 
   section = abfd->sections;
   for (i = 0; i < num_spaces; i++)
     {
 
       /* Find a space.  */
-      while (som_section_data (section)->is_space == 0)
+      while (!som_is_space (section))
        section = section->next;
 
       /* Dump its header  */
-      if (bfd_write ((PTR) &som_section_data (section)->space_dict,
+      if (bfd_write ((PTR) som_section_data (section)->space_dict,
                     sizeof (struct space_dictionary_record), 1, abfd)
          != sizeof (struct space_dictionary_record))
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
 
       /* Goto the next section.  */
       section = section->next;
@@ -3100,14 +3298,12 @@ som_write_headers (abfd)
 
   /* Only thing left to do is write out the file header.  It is always
      at location zero.  Seek there and write it.  */
-  bfd_seek (abfd, (file_ptr) 0, SEEK_SET);
+  if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) < 0)
+    return false;
   if (bfd_write ((PTR) obj_som_file_hdr (abfd),
                 sizeof (struct header), 1, abfd)
       != sizeof (struct header))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
   return true;
 }
 
@@ -3225,7 +3421,7 @@ som_bfd_derive_misc_symbol_info (abfd, sym, info)
   /* For all other symbols, the symbol_info field contains the 
      subspace index of the space this symbol is contained in.  */
   else
-    info->symbol_info = som_section_data (sym->section)->subspace_index;
+    info->symbol_info = sym->section->target_index;
 
   /* Set the symbol's value.  */
   info->symbol_value = sym->value + sym->section->vma;
@@ -3248,7 +3444,7 @@ som_build_and_write_symbol_table (abfd)
      to hold the symbol table as we build it.  */
   symtab_size = num_syms * sizeof (struct symbol_dictionary_record);
   som_symtab = (struct symbol_dictionary_record *) malloc (symtab_size);
-  if (som_symtab == NULL)
+  if (som_symtab == NULL && symtab_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -3279,16 +3475,10 @@ som_build_and_write_symbol_table (abfd)
   /* Everything is ready, seek to the right location and
      scribble out the symbol table.  */
   if (bfd_seek (abfd, symtab_location, SEEK_SET) != 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    return false;
 
   if (bfd_write ((PTR) som_symtab, symtab_size, 1, abfd) != symtab_size)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   if (som_symtab != NULL)
     free (som_symtab);
@@ -3340,8 +3530,17 @@ som_slurp_string_table (abfd)
   if (obj_som_stringtab (abfd) != NULL)
     return true;
 
+  /* I don't think this can currently happen, and I'm not sure it should
+     really be an error, but it's better than getting unpredictable results
+     from the host's malloc when passed a size of zero.  */
+  if (obj_som_stringtab_size (abfd) == 0)
+    {
+      bfd_set_error (bfd_error_no_symbols);
+      return false;
+    }
+
   /* Allocate and read in the string table.  */
-  stringtab = bfd_zalloc (abfd, obj_som_stringtab_size (abfd));
+  stringtab = malloc (obj_som_stringtab_size (abfd));
   if (stringtab == NULL)
     {
       bfd_set_error (bfd_error_no_memory);
@@ -3349,17 +3548,11 @@ som_slurp_string_table (abfd)
     }
 
   if (bfd_seek (abfd, obj_som_str_filepos (abfd), SEEK_SET) < 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
   
   if (bfd_read (stringtab, obj_som_stringtab_size (abfd), 1, abfd)
       != obj_som_stringtab_size (abfd))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* Save our results and return success. */
   obj_som_stringtab (abfd) = stringtab;
@@ -3369,12 +3562,12 @@ som_slurp_string_table (abfd)
 /* Return the amount of data (in bytes) required to hold the symbol
    table for this object.  */
 
-static unsigned int
+static long
 som_get_symtab_upper_bound (abfd)
      bfd *abfd;
 {
   if (!som_slurp_symbol_table (abfd))
-    return 0;
+    return -1;
 
   return (bfd_get_symcount (abfd) + 1) * (sizeof (asymbol *));
 }
@@ -3382,18 +3575,46 @@ som_get_symtab_upper_bound (abfd)
 /* Convert from a SOM subspace index to a BFD section.  */
 
 static asection *
-som_section_from_subspace_index (abfd, index)
+bfd_section_from_som_symbol (abfd, symbol)
      bfd *abfd;
-     unsigned int index;
+     struct symbol_dictionary_record *symbol;
 {
   asection *section;
 
-  for (section = abfd->sections; section != NULL; section = section->next)
-    if (som_section_data (section)->subspace_index == index)
-      return section;
+  /* 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) == 0
+      || (symbol->symbol_type != ST_ENTRY
+         && symbol->symbol_type != ST_PRI_PROG
+         && symbol->symbol_type != ST_SEC_PROG
+         && symbol->symbol_type != ST_MILLICODE))
+    {
+      unsigned int index = symbol->symbol_info;
+      for (section = abfd->sections; section != NULL; section = section->next)
+       if (section->target_index == index)
+         return section;
 
-  /* Should never happen.  */
-  abort();
+      /* Should never happen.  */
+      abort();
+    }
+  else
+    {
+      unsigned int value = symbol->symbol_value;
+      unsigned int found = 0;
+
+      /* For executables we will have to use the symbol's address and
+        find out what section would contain that address.   Yuk.  */
+      for (section = abfd->sections; section; section = section->next)
+       {
+         if (value >= section->vma
+             && value <= section->vma + section->_cooked_size)
+           return section;
+       }
+
+      /* Should never happen.  */
+      abort ();
+    }
 }
 
 /* Read and save the symbol table associated with the given BFD.  */
@@ -3422,7 +3643,7 @@ som_slurp_symbol_table (abfd)
   stringtab = obj_som_stringtab (abfd);
 
   symbase = (som_symbol_type *)
-    bfd_zalloc (abfd, symbol_count * sizeof (som_symbol_type));
+    malloc (symbol_count * sizeof (som_symbol_type));
   if (symbase == NULL)
     {
       bfd_set_error (bfd_error_no_memory);
@@ -3431,22 +3652,16 @@ som_slurp_symbol_table (abfd)
 
   /* Read in the external SOM representation.  */
   buf = malloc (symbol_count * symsize);
-  if (buf == NULL)
+  if (buf == NULL && symbol_count * symsize != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
     }
   if (bfd_seek (abfd, obj_som_sym_filepos (abfd), SEEK_SET) < 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
   if (bfd_read (buf, symbol_count * symsize, 1, abfd) 
       != symbol_count * symsize)
-    {
-      bfd_set_error (bfd_error_no_symbols);
-      goto error_return;
-    }
+    goto error_return;
 
   /* Iterate over all the symbols and internalize them.  */
   endbufp = buf + symbol_count;
@@ -3528,8 +3743,7 @@ som_slurp_symbol_table (abfd)
 
        case SS_UNIVERSAL:
          sym->symbol.flags |= (BSF_EXPORT | BSF_GLOBAL);
-         sym->symbol.section
-           = som_section_from_subspace_index (abfd, bufp->symbol_info);
+         sym->symbol.section = bfd_section_from_som_symbol (abfd, bufp);
          sym->symbol.value -= sym->symbol.section->vma;
          break;
 
@@ -3540,16 +3754,21 @@ som_slurp_symbol_table (abfd)
 #endif
        case SS_LOCAL:
          sym->symbol.flags |= BSF_LOCAL;
-         sym->symbol.section
-           = som_section_from_subspace_index (abfd, bufp->symbol_info);
+         sym->symbol.section = bfd_section_from_som_symbol (abfd, bufp);
          sym->symbol.value -= sym->symbol.section->vma;
          break;
        }
 
       /* Mark section symbols and symbols used by the debugger.  */
-      if (!strcmp (sym->symbol.name, "L$0\002"))
+      if (sym->symbol.name[0] == '$'
+         && sym->symbol.name[strlen (sym->symbol.name) - 1] == '$')
        sym->symbol.flags |= BSF_SECTION_SYM;
-      else if (!strncmp (sym->symbol.name, "L$0", 3))
+      else if (!strncmp (sym->symbol.name, "L$0\002", 4))
+       {
+         sym->symbol.flags |= BSF_SECTION_SYM;
+         sym->symbol.name = sym->symbol.section->name;
+       }
+      else if (!strncmp (sym->symbol.name, "L$0\001", 4))
        sym->symbol.flags |= BSF_DEBUGGING;
 
       /* Note increment at bottom of loop, since we skip some symbols
@@ -3573,7 +3792,7 @@ som_slurp_symbol_table (abfd)
 /* Canonicalize a SOM symbol table.  Return the number of entries
    in the symbol table.  */
 
-static unsigned int
+static long
 som_get_symtab (abfd, location)
      bfd *abfd;
      asymbol **location;
@@ -3582,7 +3801,7 @@ som_get_symtab (abfd, location)
   som_symbol_type *symbase;
 
   if (!som_slurp_symbol_table (abfd))
-    return 0;
+    return -1;
 
   i = bfd_get_symcount (abfd);
   symbase = obj_som_symtab (abfd);
@@ -3644,6 +3863,14 @@ som_print_symbol (ignore_abfd, afile, symbol, how)
     }
 }
 
+static boolean
+som_bfd_is_local_label (abfd, sym)
+     bfd *abfd;
+     asymbol *sym;
+{
+  return (sym->name[0] == 'L' && sym->name[1] == '$');
+}
+
 /* Count or process variable-length SOM fixup records.
 
    To avoid code duplication we use this code both to compute the number
@@ -3904,7 +4131,7 @@ som_slurp_reloc_table (abfd, section, symbols, just_count)
      parsed.  We must do so now to know how many relocations exist.  */
   if (section->reloc_count == -1)
     {
-      external_relocs = (char *) bfd_zalloc (abfd, fixup_stream_size);
+      external_relocs = (char *) malloc (fixup_stream_size);
       if (external_relocs == (char *) NULL)
        {
          bfd_set_error (bfd_error_no_memory);
@@ -3915,16 +4142,11 @@ som_slurp_reloc_table (abfd, section, symbols, just_count)
                    obj_som_reloc_filepos (abfd) + section->rel_filepos,
                    SEEK_SET)
          != 0)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
       if (bfd_read (external_relocs, 1, fixup_stream_size, abfd)
          != fixup_stream_size)
-       {
-         bfd_set_error (bfd_error_system_call);
-         return false;
-       }
+       return false;
+
       /* Let callers know how many relocations found.
         also save the relocation stream as we will
         need it again.  */
@@ -3945,8 +4167,7 @@ som_slurp_reloc_table (abfd, section, symbols, just_count)
   if (section->relocation != (arelent *) NULL)
     return true;
 
-  internal_relocs = (arelent *) bfd_zalloc (abfd,
-                                           num_relocs * sizeof (arelent));
+  internal_relocs = (arelent *) malloc (num_relocs * sizeof (arelent));
   if (internal_relocs == (arelent *) NULL)
     {
       bfd_set_error (bfd_error_no_memory);
@@ -3965,7 +4186,7 @@ som_slurp_reloc_table (abfd, section, symbols, just_count)
 /* Return the number of bytes required to store the relocation
    information associated with the given section.  */ 
 
-static unsigned int
+static long
 som_get_reloc_upper_bound (abfd, asect)
      bfd *abfd;
      sec_ptr asect;
@@ -3974,18 +4195,18 @@ som_get_reloc_upper_bound (abfd, asect)
      and parse it to determine how many relocations exist.  */
   if (asect->flags & SEC_RELOC)
     {
-      if (som_slurp_reloc_table (abfd, asect, NULL, true))
-       return (asect->reloc_count + 1) * sizeof (arelent);
+      if (! som_slurp_reloc_table (abfd, asect, NULL, true))
+       return false;
+      return (asect->reloc_count + 1) * sizeof (arelent);
     }
-  /* Either there are no relocations or an error occurred while 
-     reading and parsing the relocation stream.  */ 
+  /* There are no relocations.  */
   return 0;
 }
 
 /* Convert relocations from SOM (external) form into BFD internal
    form.  Return the number of relocations.  */
 
-static unsigned int
+static long
 som_canonicalize_reloc (abfd, section, relptr, symbols)
      bfd *abfd;
      sec_ptr section;
@@ -3996,12 +4217,10 @@ som_canonicalize_reloc (abfd, section, relptr, symbols)
   int count;
 
   if (som_slurp_reloc_table (abfd, section, symbols, false) == false)
-    return 0;
+    return -1;
 
   count = section->reloc_count;
   tblptr = section->relocation;
-  if (tblptr == (arelent *) NULL)
-    return 0;
 
   while (count--)
     *relptr++ = tblptr++;
@@ -4028,18 +4247,78 @@ som_new_section_hook (abfd, newsect)
     }
   newsect->alignment_power = 3;
 
-  /* Initialize the subspace_index field to -1 so that it does
-     not match a subspace with an index of 0.  */
-  som_section_data (newsect)->subspace_index = -1;
-
   /* We allow more than three sections internally */
   return true;
 }
 
+/* Copy any private info we understand from the input section
+   to the output section.  */
+static boolean
+som_bfd_copy_private_section_data (ibfd, isection, obfd, osection)
+     bfd *ibfd;
+     asection *isection;
+     bfd *obfd;
+     asection *osection;
+{
+  /* One day we may try to grok other private data.  */
+  if (ibfd->xvec->flavour != bfd_target_som_flavour
+      || obfd->xvec->flavour != bfd_target_som_flavour
+      || (!som_is_space (isection) && !som_is_subspace (isection)))
+    return false;
+
+  som_section_data (osection)->copy_data
+    = (struct som_copyable_section_data_struct *)
+      bfd_zalloc (obfd, sizeof (struct som_copyable_section_data_struct));
+  if (som_section_data (osection)->copy_data == NULL)
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return false;
+    }
+
+  memcpy (som_section_data (osection)->copy_data,
+         som_section_data (isection)->copy_data,
+         sizeof (struct som_copyable_section_data_struct));
+
+  /* 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;
+
+  return true;
+}
+
+/* Copy any private info we understand from the input bfd
+   to the output bfd.  */
+
+static boolean
+som_bfd_copy_private_bfd_data (ibfd, obfd)
+     bfd *ibfd, *obfd;
+{
+  /* One day we may try to grok other private data.  */
+  if (ibfd->xvec->flavour != bfd_target_som_flavour
+      || obfd->xvec->flavour != bfd_target_som_flavour)
+    return false;
+
+  /* Allocate some memory to hold the data we need.  */
+  obj_som_exec_data (obfd) = (struct som_exec_data *)
+    bfd_zalloc (obfd, sizeof (struct som_exec_data));
+  if (obj_som_exec_data (obfd) == NULL)
+    {
+      bfd_set_error (bfd_error_no_memory);
+      return false;
+    }
+
+  /* Now copy the data.  */
+  memcpy (obj_som_exec_data (obfd), obj_som_exec_data (ibfd),
+         sizeof (struct som_exec_data));
+
+  return true;
+}
+
 /* Set backend info for sections which can not be described
    in the BFD data structures.  */
 
-void
+boolean
 bfd_som_set_section_attributes (section, defined, private, sort_key, spnum)
      asection *section;
      int defined;
@@ -4047,20 +4326,31 @@ bfd_som_set_section_attributes (section, defined, private, sort_key, spnum)
      unsigned int sort_key;
      int spnum;
 {
-  struct space_dictionary_record *space_dict;
-
-  som_section_data (section)->is_space = 1;
-  space_dict = &som_section_data (section)->space_dict;
-  space_dict->is_defined = defined;
-  space_dict->is_private = private;
-  space_dict->sort_key = sort_key;
-  space_dict->space_number = spnum;
+  /* Allocate memory to hold the magic information.  */
+  if (som_section_data (section)->copy_data == NULL)
+    {
+      som_section_data (section)->copy_data
+       = (struct som_copyable_section_data_struct *)
+         bfd_zalloc (section->owner,
+                     sizeof (struct som_copyable_section_data_struct));
+      if (som_section_data (section)->copy_data == NULL)
+       {
+         bfd_set_error (bfd_error_no_memory);
+         return false;
+       }
+    }
+  som_section_data (section)->copy_data->sort_key = sort_key;
+  som_section_data (section)->copy_data->is_defined = defined;
+  som_section_data (section)->copy_data->is_private = private;
+  som_section_data (section)->copy_data->container = section;
+  section->target_index = spnum;
+  return true;
 }
 
 /* Set backend info for subsections which can not be described 
    in the BFD data structures.  */
 
-void
+boolean
 bfd_som_set_subsection_attributes (section, container, access,
                                   sort_key, quadrant)
      asection *section;
@@ -4069,13 +4359,24 @@ bfd_som_set_subsection_attributes (section, container, access,
      unsigned int sort_key;
      int quadrant;
 {
-  struct subspace_dictionary_record *subspace_dict;
-  som_section_data (section)->is_subspace = 1;
-  subspace_dict = &som_section_data (section)->subspace_dict;
-  subspace_dict->access_control_bits = access;
-  subspace_dict->sort_key = sort_key;
-  subspace_dict->quadrant = quadrant;
-  som_section_data (section)->containing_space = container;
+  /* Allocate memory to hold the magic information.  */
+  if (som_section_data (section)->copy_data == NULL)
+    {
+      som_section_data (section)->copy_data
+       = (struct som_copyable_section_data_struct *)
+         bfd_zalloc (section->owner,
+                     sizeof (struct som_copyable_section_data_struct));
+      if (som_section_data (section)->copy_data == NULL)
+       {
+         bfd_set_error (bfd_error_no_memory);
+         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->quadrant = quadrant;
+  som_section_data (section)->copy_data->container = container;
+  return true;
 }
 
 /* Set the full SOM symbol type.  SOM needs far more symbol information
@@ -4145,7 +4446,7 @@ bfd_som_attach_aux_hdr (abfd, type, string)
                            + sizeof (unsigned int) + len + pad);
       if (!obj_som_copyright_hdr (abfd))
        {
-         bfd_set_error (bfd_error_no_error);
+         bfd_set_error (bfd_error_no_memory);
          return false;
        }
       obj_som_copyright_hdr (abfd)->header_id.type = COPYRIGHT_AUX_ID;
@@ -4157,6 +4458,23 @@ bfd_som_attach_aux_hdr (abfd, type, string)
   return true;
 }
 
+static boolean
+som_get_section_contents (abfd, section, location, offset, count)
+     bfd *abfd;
+     sec_ptr section;
+     PTR location;
+     file_ptr offset;
+     bfd_size_type count;
+{
+  if (count == 0 || ((section->flags & (SEC_LOAD | SEC_DEBUGGING)) == 0))
+    return true;
+  if ((bfd_size_type)(offset+count) > section->_raw_size
+      || bfd_seek (abfd, (file_ptr)(section->filepos + offset), SEEK_SET) == -1
+      || bfd_read (location, (bfd_size_type)1, count, abfd) != count)
+    return (false); /* on error */
+  return (true);
+}
+
 static boolean
 som_set_section_contents (abfd, section, location, offset, count)
      bfd *abfd;
@@ -4178,24 +4496,18 @@ som_set_section_contents (abfd, section, location, offset, count)
 
   /* Only write subspaces which have "real" contents (eg. the contents
      are not generated at run time by the OS).  */
-  if (som_section_data (section)->is_subspace != 1
+  if (!som_is_subspace (section)
       || ((section->flags & (SEC_LOAD | SEC_DEBUGGING)) == 0))
     return true;
 
   /* Seek to the proper offset within the object file and write the
      data.  */
-  offset += som_section_data (section)->subspace_dict.file_loc_init_value; 
+  offset += som_section_data (section)->subspace_dict->file_loc_init_value; 
   if (bfd_seek (abfd, offset, SEEK_SET) == -1)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   if (bfd_write ((PTR) location, 1, count, abfd) != count)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
   return true;
 }
 
@@ -4309,7 +4621,7 @@ som_bfd_count_ar_symbols (abfd, lst_header, count)
 
   hash_table = 
     (unsigned int *) malloc (lst_header->hash_size * sizeof (unsigned int));
-  if (hash_table == NULL)
+  if (hash_table == NULL && lst_header->hash_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4322,10 +4634,7 @@ som_bfd_count_ar_symbols (abfd, lst_header, count)
      which point to the hash chains.  */
   if (bfd_read ((PTR) hash_table, lst_header->hash_size, 4, abfd)
       != lst_header->hash_size * 4)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* Walk each chain counting the number of symbols found on that particular
      chain.  */
@@ -4339,18 +4648,13 @@ som_bfd_count_ar_symbols (abfd, lst_header, count)
 
       /* Seek to the first symbol in this hash chain.  */
       if (bfd_seek (abfd, lst_filepos + hash_table[i], SEEK_SET) < 0)
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
 
       /* Read in this symbol and update the counter.  */
       if (bfd_read ((PTR) & lst_symbol, 1, sizeof (lst_symbol), abfd)
          != sizeof (lst_symbol))
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
+
       (*count)++;
 
       /* Now iterate through the rest of the symbols on this chain.  */
@@ -4360,18 +4664,13 @@ som_bfd_count_ar_symbols (abfd, lst_header, count)
          /* Seek to the next symbol.  */
          if (bfd_seek (abfd, lst_filepos + lst_symbol.next_entry, SEEK_SET)
              < 0)
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           goto error_return;
 
          /* Read the symbol in and update the counter.  */
          if (bfd_read ((PTR) & lst_symbol, 1, sizeof (lst_symbol), abfd)
              != sizeof (lst_symbol))
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           goto error_return;
+
          (*count)++;
        }
     }
@@ -4402,7 +4701,7 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
 
   hash_table = 
     (unsigned int *) malloc (lst_header->hash_size * sizeof (unsigned int));
-  if (hash_table == NULL)
+  if (hash_table == NULL && lst_header->hash_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4411,7 +4710,7 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
   som_dict =
     (struct som_entry *) malloc (lst_header->module_count
                                 * sizeof (struct som_entry));
-  if (som_dict == NULL)
+  if (som_dict == NULL && lst_header->module_count != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4421,26 +4720,17 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
      which point to the hash chains.  */
   if (bfd_read ((PTR) hash_table, lst_header->hash_size, 4, abfd)
       != lst_header->hash_size * 4)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* Seek to and read in the SOM dictionary.  We will need this to fill
      in the carsym's filepos field.  */
   if (bfd_seek (abfd, lst_filepos + lst_header->dir_loc, SEEK_SET) < 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   if (bfd_read ((PTR) som_dict, lst_header->module_count, 
                sizeof (struct som_entry), abfd)
       != lst_header->module_count * sizeof (struct som_entry))
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* Walk each chain filling in the carsyms as we go along.  */
   for (i = 0; i < lst_header->hash_size; i++)
@@ -4453,17 +4743,11 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
 
       /* Seek to and read the first symbol on the chain.  */
       if (bfd_seek (abfd, lst_filepos + hash_table[i], SEEK_SET) < 0)
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
 
       if (bfd_read ((PTR) & lst_symbol, 1, sizeof (lst_symbol), abfd)
          != sizeof (lst_symbol))
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
 
       /* Get the name of the symbol, first get the length which is stored
         as a 32bit integer just before the symbol.
@@ -4474,16 +4758,10 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
         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)
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
 
       if (bfd_read (&len, 1, 4, abfd) != 4)
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
 
       /* Allocate space for the name and null terminate it too.  */
       set->name = bfd_zalloc (abfd, len + 1);
@@ -4493,10 +4771,8 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
          goto error_return;
        }
       if (bfd_read (set->name, 1, len, abfd) != len)
-       {
-         bfd_set_error (bfd_error_system_call);
-         goto error_return;
-       }
+       goto error_return;
+
       set->name[len] = 0;
 
       /* Fill in the file offset.  Note that the "location" field points
@@ -4511,33 +4787,20 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
       while (lst_symbol.next_entry)
        {
          /* Seek to the next symbol and read it in.  */
-         if (bfd_seek (abfd, lst_filepos + lst_symbol.next_entry, SEEK_SET)
-             < 0)
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+         if (bfd_seek (abfd, lst_filepos + lst_symbol.next_entry, SEEK_SET) <0)
+           goto error_return;
 
          if (bfd_read ((PTR) & lst_symbol, 1, sizeof (lst_symbol), abfd)
              != sizeof (lst_symbol))
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           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)
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           goto error_return;
 
          if (bfd_read (&len, 1, 4, abfd) != 4)
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           goto error_return;
 
          /* Allocate space for the name and null terminate it too.  */
          set->name = bfd_zalloc (abfd, len + 1);
@@ -4546,11 +4809,9 @@ som_bfd_fill_in_ar_symbols (abfd, lst_header, syms)
              bfd_set_error (bfd_error_no_memory);
              goto error_return;
            }
+
          if (bfd_read (set->name, 1, len, abfd) != len)
-           {
-             bfd_set_error (bfd_error_system_call);
-             goto error_return;
-           }
+           goto error_return;
          set->name[len] = 0;
 
          /* Fill in the file offset.  Note that the "location" field points
@@ -4597,10 +4858,7 @@ som_slurp_armap (abfd)
     return false;
 
   if (bfd_seek (abfd, (file_ptr) - 16, SEEK_CUR) < 0)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* For archives without .o files there is no symbol table.  */
   if (strncmp (nextname, "/               ", 16))
@@ -4612,10 +4870,7 @@ som_slurp_armap (abfd)
   /* Read in and sanity check the archive header.  */
   if (bfd_read ((PTR) &ar_header, 1, sizeof (struct ar_hdr), abfd)
       != sizeof (struct ar_hdr))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   if (strncmp (ar_header.ar_fmag, ARFMAG, 2))
     {
@@ -4639,10 +4894,7 @@ som_slurp_armap (abfd)
      in just a minute.  */
   if (bfd_read ((PTR) & lst_header, 1, sizeof (struct lst_header), abfd)
       != sizeof (struct lst_header))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* Sanity check.  */
   if (lst_header.a_magic != LIBMAGIC)
@@ -4659,10 +4911,7 @@ som_slurp_armap (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)
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* Initializae the cache and allocate space for the library symbols.  */
   ardata->cache = 0;
@@ -4680,6 +4929,11 @@ som_slurp_armap (abfd)
       == false)
     return false;
 
+  /* Seek back to the "first" file in the archive.  Note the "first"
+     file may be the extended name table.  */
+  if (bfd_seek (abfd, ardata->first_file_filepos, SEEK_SET) < 0)
+    return false;
+
   /* Notify the generic archive code that we have a symbol map.  */
   bfd_has_map (abfd) = true;
   return true;
@@ -4707,6 +4961,14 @@ som_bfd_prep_for_ar_write (abfd, num_syms, stringsize)
       unsigned int curr_count, i;
       som_symbol_type *sym;
 
+      /* Don't bother for non-SOM objects.  */
+      if (curr_bfd->format != bfd_object
+         || curr_bfd->xvec->flavour != bfd_target_som_flavour)
+       {
+         curr_bfd = curr_bfd->next;
+         continue;
+       }
+
       /* Make sure the symbol table has been read, then snag a pointer
         to it.  It's a little slimey to grab the symbols via obj_som_symtab,
         but doing so avoids allocating lots of extra memory.  */
@@ -4769,6 +5031,19 @@ som_bfd_ar_symbol_hash (symbol)
          | (symbol->name[len-2] << 8) | symbol->name[len-1];
 }
 
+static CONST char *
+normalize (file)
+     CONST char *file;
+{
+  CONST char *filename = strrchr (file, '/');
+
+  if (filename != NULL)
+    filename++;
+  else
+    filename = file;
+  return filename;
+}
+
 /* Do the bulk of the work required to write the SOM library
    symbol table.  */
    
@@ -4781,15 +5056,16 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   file_ptr lst_filepos;
   char *strings = NULL, *p;
   struct lst_symbol_record *lst_syms = NULL, *curr_lst_sym;
-  bfd *curr_bfd = abfd->archive_head;
+  bfd *curr_bfd;
   unsigned int *hash_table = NULL;
   struct som_entry *som_dict = NULL;
   struct lst_symbol_record **last_hash_entry = NULL;
-  unsigned int curr_som_offset, som_index;
+  unsigned int curr_som_offset, som_index, extended_name_length = 0;
+  unsigned int maxname = abfd->xvec->ar_max_namelen;
 
   hash_table =
     (unsigned int *) malloc (lst.hash_size * sizeof (unsigned int));
-  if (hash_table == NULL)
+  if (hash_table == NULL && lst.hash_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4797,7 +5073,7 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   som_dict =
     (struct som_entry *) malloc (lst.module_count
                                 * sizeof (struct som_entry));
-  if (som_dict == NULL)
+  if (som_dict == NULL && lst.module_count != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4806,7 +5082,7 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   last_hash_entry =
     ((struct lst_symbol_record **)
      malloc (lst.hash_size * sizeof (struct lst_symbol_record *)));
-  if (last_hash_entry == NULL)
+  if (last_hash_entry == NULL && lst.hash_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4832,15 +5108,42 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   som_index = 0;
   curr_som_offset = 8 + 2 * sizeof (struct ar_hdr) + lst.file_end;
 
+  /* Yow!  We have to know the size of the extended name table
+     too.  */
+  for (curr_bfd = abfd->archive_head;
+       curr_bfd != NULL;
+       curr_bfd = curr_bfd->next)
+    {
+      CONST char *normal = normalize (curr_bfd->filename);
+      unsigned int thislen;
+
+      if (!normal)
+       {
+         bfd_set_error (bfd_error_no_memory);
+         return false;
+       }
+      thislen = strlen (normal);
+      if (thislen > maxname)
+       extended_name_length += thislen + 1;
+    }
+
+  /* Make room for the archive header and the contents of the
+     extended string table.  */
+  if (extended_name_length)
+    curr_som_offset += extended_name_length + sizeof (struct ar_hdr);
+
+  /* Make sure we're properly aligned.  */
+  curr_som_offset = (curr_som_offset + 0x1) & ~0x1;
+
   /* FIXME should be done with buffers just like everything else... */
   lst_syms = malloc (nsyms * sizeof (struct lst_symbol_record));
-  if (lst_syms == NULL)
+  if (lst_syms == NULL && nsyms != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
     }
   strings = malloc (string_size);
-  if (strings == NULL)
+  if (strings == NULL && string_size != 0)
     {
       bfd_set_error (bfd_error_no_memory);
       goto error_return;
@@ -4849,12 +5152,20 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   p = strings;
   curr_lst_sym = lst_syms;
 
-
+  curr_bfd = abfd->archive_head;
   while (curr_bfd != NULL)
     {
       unsigned int curr_count, i;
       som_symbol_type *sym;
 
+      /* Don't bother for non-SOM objects.  */
+      if (curr_bfd->format != bfd_object
+         || curr_bfd->xvec->flavour != bfd_target_som_flavour)
+       {
+         curr_bfd = curr_bfd->next;
+         continue;
+       }
+
       /* Make sure the symbol table has been read, then snag a pointer
         to it.  It's a little slimey to grab the symbols via obj_som_symtab,
         but doing so avoids allocating lots of extra memory.  */
@@ -4972,34 +5283,22 @@ som_bfd_ar_write_symbol_stuff (abfd, nsyms, string_size, lst)
   /* Now scribble out the hash table.  */
   if (bfd_write ((PTR) hash_table, lst.hash_size, 4, abfd)
       != lst.hash_size * 4)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* Then the SOM dictionary.  */
   if (bfd_write ((PTR) som_dict, lst.module_count,
                 sizeof (struct som_entry), abfd)
       != lst.module_count * sizeof (struct som_entry))
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* The library symbols.  */
   if (bfd_write ((PTR) lst_syms, nsyms, sizeof (struct lst_symbol_record), abfd)
       != nsyms * sizeof (struct lst_symbol_record))
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   /* And finally the strings.  */
   if (bfd_write ((PTR) strings, string_size, 1, abfd) != string_size)
-    {
-      bfd_set_error (bfd_error_system_call);
-      goto error_return;
-    }
+    goto error_return;
 
   if (hash_table != NULL)
     free (hash_table);
@@ -5073,7 +5372,10 @@ som_write_armap (abfd)
   lst.module_count = 0;
   while (curr_bfd != NULL)
     {
-      lst.module_count++;
+      /* Only true SOM objects count.  */
+      if (curr_bfd->format == bfd_object
+         && curr_bfd->xvec->flavour == bfd_target_som_flavour)
+       lst.module_count++;
       curr_bfd = curr_bfd->next;
     }
   lst.module_limit = lst.module_count;
@@ -5105,12 +5407,12 @@ som_write_armap (abfd)
 
   /* SOM ABI says this must be zero.  */
   lst.free_list = 0;
-
   lst.file_end = lst_size;
 
   /* 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++;
 
@@ -5131,18 +5433,12 @@ som_write_armap (abfd)
   /* Scribble out the ar header.  */
   if (bfd_write ((PTR) &hdr, 1, sizeof (struct ar_hdr), abfd)
       != sizeof (struct ar_hdr))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* Now scribble out the lst header.  */
   if (bfd_write ((PTR) &lst, 1, sizeof (struct lst_header), abfd)
       != sizeof (struct lst_header))
-    {
-      bfd_set_error (bfd_error_system_call);
-      return false;
-    }
+    return false;
 
   /* Build and write the armap.  */
   if (som_bfd_ar_write_symbol_stuff (abfd, nsyms, stringsize, lst) == false)
@@ -5152,13 +5448,29 @@ som_write_armap (abfd)
   return true;
 }
 
-/* Apparently the extened names are never used, even though they appear
-   in the SOM ABI.  Hmmm.  */
+/* Free all information we have cached for this BFD.  We can always
+   read it again later if we need it.  */
+
 static boolean
-som_slurp_extended_name_table (abfd)
+som_bfd_free_cached_info (abfd)
      bfd *abfd;
 {
-  bfd_ardata (abfd)->extended_names = NULL;
+  asection *o;
+
+#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 != (asection *) NULL; o = o->next)
+    {
+      /* Free the native relocations.  */
+      o->reloc_count = -1;
+      FREE (som_section_data (o)->reloc_stream);
+      /* Free the generic relocations.  */
+      FREE (o->relocation);
+    }
+#undef FREE
+
   return true;
 }
 
@@ -5171,10 +5483,10 @@ som_slurp_extended_name_table (abfd)
 #define som_openr_next_archived_file   bfd_generic_openr_next_archived_file
 #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_get_lineno                   (struct lineno_cache_entry *(*)())bfd_nullvoidptr
 #define        som_close_and_cleanup              bfd_generic_close_and_cleanup
-#define som_get_section_contents          bfd_generic_get_section_contents
 
 #define som_bfd_get_relocated_section_contents \
  bfd_generic_get_relocated_section_contents
@@ -5206,7 +5518,7 @@ bfd_target som_vec =
    predictable, and if so what is it */
   0,
   '/',                         /* ar_pad_char */
-  16,                          /* ar_max_namelen */
+  14,                          /* ar_max_namelen */
   3,                           /* minimum alignment */
   bfd_getb64, bfd_getb_signed_64, bfd_putb64,
   bfd_getb32, bfd_getb_signed_32, bfd_putb32,
This page took 0.049606 seconds and 4 git commands to generate.