* ld-scripts/script.t: Put .pr in .text, and .rw in .data, for
[deliverable/binutils-gdb.git] / binutils / objcopy.c
index 926ff6b09f6c1038a8f3c603f2f148dee348849f..befe5093491427a00a6ba4a2ab0e77d79712ba39 100644 (file)
 
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
+   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 \f
 #include "bfd.h"
 #include "sysdep.h"
+#include "progress.h"
 #include "bucomm.h"
 #include <getopt.h>
 #include "libiberty.h"
 
 static bfd_vma parse_vma PARAMS ((const char *, const char *));
+static flagword parse_flags PARAMS ((const char *));
+static struct section_list *find_section_list PARAMS ((const char *, boolean));
 static void setup_section PARAMS ((bfd *, asection *, PTR));
 static void copy_section PARAMS ((bfd *, asection *, PTR));
 static void get_sections PARAMS ((bfd *, asection *, PTR));
 static int compare_section_vma PARAMS ((const PTR, const PTR));
 static void add_strip_symbol PARAMS ((const char *));
-static int is_strip_symbol PARAMS ((const char *));
+static boolean is_strip_symbol PARAMS ((const char *));
+static boolean is_strip_section PARAMS ((bfd *, asection *));
 static unsigned int filter_symbols
   PARAMS ((bfd *, asymbol **, asymbol **, long));
 static void mark_symbols_used_in_relocations PARAMS ((bfd *, asection *, PTR));
@@ -51,6 +55,7 @@ enum strip_action
     strip_undef,
     strip_none,                        /* don't strip */
     strip_debug,               /* strip all debugger symbols */
+    strip_unneeded,            /* strip unnecessary symbols */
     strip_all                  /* strip all symbols */
   };
 
@@ -77,16 +82,20 @@ struct section_list
   const char *name;
   /* Whether this entry was used.  */
   boolean used;
-  /* Remaining fields only used if not on remove_sections list.  */
+  /* Whether to remove this section.  */
+  boolean remove;
   /* Whether to adjust or set VMA.  */
-  boolean adjust;
+  enum { ignore_vma, adjust_vma, set_vma } adjust;
   /* Amount to adjust by or set to.  */
   bfd_vma val;
+  /* Whether to set the section flags.  */
+  boolean set_flags;
+  /* What to set the section flags to.  */
+  flagword flags;
 };
 
-/* List of sections to remove.  */
-
-static struct section_list *remove_sections;
+static struct section_list *adjust_sections;
+static boolean sections_removed;
 
 /* Adjustments to the start address.  */
 static bfd_vma adjust_start = 0;
@@ -95,7 +104,6 @@ static bfd_vma set_start;
 
 /* Adjustments to section VMA's.  */
 static bfd_vma adjust_section_vma = 0;
-static struct section_list *adjust_sections;
 
 /* Filling gaps between sections.  */
 static boolean gap_fill_set = false;
@@ -105,6 +113,40 @@ static bfd_byte gap_fill = 0;
 static boolean pad_to_set = false;
 static bfd_vma pad_to;
 
+/* List of sections to add.  */
+
+struct section_add
+{
+  /* Next section to add.  */
+  struct section_add *next;
+  /* Name of section to add.  */
+  const char *name;
+  /* Name of file holding section contents.  */
+  const char *filename;
+  /* Size of file.  */
+  size_t size;
+  /* Contents of file.  */
+  bfd_byte *contents;
+  /* BFD section, after it has been added.  */
+  asection *section;
+};
+
+static struct section_add *add_sections;
+
+/* 150 isn't special; it's just an arbitrary non-ASCII char value.  */
+
+#define OPTION_ADD_SECTION 150
+#define OPTION_ADJUST_START (OPTION_ADD_SECTION + 1)
+#define OPTION_ADJUST_VMA (OPTION_ADJUST_START + 1)
+#define OPTION_ADJUST_SECTION_VMA (OPTION_ADJUST_VMA + 1)
+#define OPTION_ADJUST_WARNINGS (OPTION_ADJUST_SECTION_VMA + 1)
+#define OPTION_GAP_FILL (OPTION_ADJUST_WARNINGS + 1)
+#define OPTION_NO_ADJUST_WARNINGS (OPTION_GAP_FILL + 1)
+#define OPTION_PAD_TO (OPTION_NO_ADJUST_WARNINGS + 1)
+#define OPTION_SET_SECTION_FLAGS (OPTION_PAD_TO + 1)
+#define OPTION_SET_START (OPTION_SET_SECTION_FLAGS + 1)
+#define OPTION_STRIP_UNNEEDED (OPTION_SET_START + 1)
+
 /* Options to handle if running as "strip".  */
 
 static struct option strip_options[] =
@@ -115,11 +157,13 @@ static struct option strip_options[] =
   {"help", no_argument, 0, 'h'},
   {"input-format", required_argument, 0, 'I'}, /* Obsolete */
   {"input-target", required_argument, 0, 'I'},
+  {"keep-symbol", required_argument, 0, 'K'},
   {"output-format", required_argument, 0, 'O'},        /* Obsolete */
   {"output-target", required_argument, 0, 'O'},
   {"remove-section", required_argument, 0, 'R'},
   {"strip-all", no_argument, 0, 's'},
   {"strip-debug", no_argument, 0, 'S'},
+  {"strip-unneeded", no_argument, 0, OPTION_STRIP_UNNEEDED},
   {"strip-symbol", required_argument, 0, 'N'},
   {"target", required_argument, 0, 'F'},
   {"verbose", no_argument, 0, 'v'},
@@ -129,19 +173,9 @@ static struct option strip_options[] =
 
 /* Options to handle if running as "objcopy".  */
 
-/* 150 isn't special; it's just an arbitrary non-ASCII char value.  */
-
-#define OPTION_ADJUST_START 150
-#define OPTION_ADJUST_VMA (OPTION_ADJUST_START + 1)
-#define OPTION_ADJUST_SECTION_VMA (OPTION_ADJUST_VMA + 1)
-#define OPTION_ADJUST_WARNINGS (OPTION_ADJUST_SECTION_VMA + 1)
-#define OPTION_GAP_FILL (OPTION_ADJUST_WARNINGS + 1)
-#define OPTION_NO_ADJUST_WARNINGS (OPTION_GAP_FILL + 1)
-#define OPTION_PAD_TO (OPTION_NO_ADJUST_WARNINGS + 1)
-#define OPTION_SET_START (OPTION_PAD_TO + 1)
-
 static struct option copy_options[] =
 {
+  {"add-section", required_argument, 0, OPTION_ADD_SECTION},
   {"adjust-start", required_argument, 0, OPTION_ADJUST_START},
   {"adjust-vma", required_argument, 0, OPTION_ADJUST_VMA},
   {"adjust-section-vma", required_argument, 0, OPTION_ADJUST_SECTION_VMA},
@@ -155,14 +189,17 @@ static struct option copy_options[] =
   {"input-format", required_argument, 0, 'I'}, /* Obsolete */
   {"input-target", required_argument, 0, 'I'},
   {"interleave", required_argument, 0, 'i'},
+  {"keep-symbol", required_argument, 0, 'K'},
   {"no-adjust-warnings", no_argument, 0, OPTION_NO_ADJUST_WARNINGS},
   {"output-format", required_argument, 0, 'O'},        /* Obsolete */
   {"output-target", required_argument, 0, 'O'},
   {"pad-to", required_argument, 0, OPTION_PAD_TO},
   {"remove-section", required_argument, 0, 'R'},
+  {"set-section-flags", required_argument, 0, OPTION_SET_SECTION_FLAGS},
   {"set-start", required_argument, 0, OPTION_SET_START},
   {"strip-all", no_argument, 0, 'S'},
   {"strip-debug", no_argument, 0, 'g'},
+  {"strip-unneeded", no_argument, 0, OPTION_STRIP_UNNEEDED},
   {"strip-symbol", required_argument, 0, 'N'},
   {"target", required_argument, 0, 'F'},
   {"verbose", no_argument, 0, 'v'},
@@ -181,38 +218,44 @@ extern int is_strip;
 
 
 static void
-copy_usage (stream, status)
+copy_usage (stream, exit_status)
      FILE *stream;
-     int status;
+     int exit_status;
 {
   fprintf (stream, "\
 Usage: %s [-vVSgxX] [-I bfdname] [-O bfdname] [-F bfdname] [-b byte]\n\
        [-R section] [-i interleave] [--interleave=interleave] [--byte=byte]\n\
        [--input-target=bfdname] [--output-target=bfdname] [--target=bfdname]\n\
-       [--strip-all] [--strip-debug] [--discard-all] [--discard-locals]\n\
-       [--remove-section=section] [--gap-fill=val] [--pad-to=address]\n\
-       [--set-start=val] [--adjust-start=incr] [--adjust-vma=incr]\n\
-       [--adjust-section-vma=section{=,+,-}val] [--adjust-warnings]\n\
-       [--no-adjust-warnings] [--verbose] [--version] [--help]\n\
-       [--strip-symbol symbol] [-N symbol]\n\
-       in-file [out-file]\n",
+       [--strip-all] [--strip-debug] [--strip-unneeded] [--discard-all]\n\
+       [--discard-locals] [--remove-section=section] [--gap-fill=val]\n",
           program_name);
-  exit (status);
+  fprintf (stream, "\
+       [--pad-to=address] [--set-start=val] [--adjust-start=incr]\n\
+       [--adjust-vma=incr] [--adjust-section-vma=section{=,+,-}val]\n\
+       [--adjust-warnings] [--no-adjust-warnings]\n\
+       [--set-section-flags=section=flags] [--add-section=sectionname=filename]\n\
+       [--keep-symbol symbol] [-K symbol] [--strip-symbol symbol] [-N symbol]\n\
+       [--verbose] [--version] [--help]\n\
+       in-file [out-file]\n");
+  list_supported_targets (program_name, stream);
+  exit (exit_status);
 }
 
 static void
-strip_usage (stream, status)
+strip_usage (stream, exit_status)
      FILE *stream;
-     int status;
+     int exit_status;
 {
   fprintf (stream, "\
 Usage: %s [-vVsSgxX] [-I bfdname] [-O bfdname] [-F bfdname] [-R section]\n\
        [--input-target=bfdname] [--output-target=bfdname] [--target=bfdname]\n\
-       [--strip-all] [--strip-debug] [--discard-all] [--discard-locals]\n\
-       [--strip-symbol symbol] [-N symbol]\n\
-       [--remove-section=section] [--verbose] [--version] [--help] file...\n",
+       [--strip-all] [--strip-debug] [--strip-unneeded] [--discard-all]\n\
+       [--discard-locals] [--keep-symbol symbol] [-K symbol]\n\
+       [--strip-symbol symbol] [-N symbol] [--remove-section=section]\n\
+       [--verbose] [--version] [--help] file...\n",
           program_name);
-  exit (status);
+  list_supported_targets (program_name, stream);
+  exit (exit_status);
 }
 
 /* Parse a string into a VMA, with a fatal error if it can't be
@@ -235,38 +278,81 @@ parse_vma (s, arg)
   return ret;
 }
 
-/* Return the name of a temporary file in the same directory as FILENAME.  */
+/* Parse section flags into a flagword, with a fatal error if the
+   string can't be parsed.  */
 
-static char *
-make_tempname (filename)
-     char *filename;
+static flagword
+parse_flags (s)
+     const char *s;
 {
-  static char template[] = "stXXXXXX";
-  char *tmpname;
-  char *slash = strrchr (filename, '/');
+  flagword ret;
+  const char *snext;
+  int len;
 
-  if (slash != (char *) NULL)
-    {
-      *slash = 0;
-      tmpname = xmalloc (strlen (filename) + sizeof (template) + 1);
-      strcpy (tmpname, filename);
-      strcat (tmpname, "/");
-      strcat (tmpname, template);
-      mktemp (tmpname);
-      *slash = '/';
-    }
-  else
+  ret = SEC_NO_FLAGS;
+
+  do
     {
-      tmpname = xmalloc (sizeof (template));
-      strcpy (tmpname, template);
-      mktemp (tmpname);
+      snext = strchr (s, ',');
+      if (snext == NULL)
+       len = strlen (s);
+      else
+       {
+         len = snext - s;
+         ++snext;
+       }
+
+#define PARSE_FLAG(fname,fval) if (strncmp (fname, s, len) == 0) ret |= fval;
+      PARSE_FLAG ("alloc", SEC_ALLOC);
+      PARSE_FLAG ("load", SEC_LOAD);
+      PARSE_FLAG ("readonly", SEC_READONLY);
+      PARSE_FLAG ("code", SEC_CODE);
+      PARSE_FLAG ("data", SEC_DATA);
+      PARSE_FLAG ("rom", SEC_ROM);
+#undef PARSE_FLAG
+
+      s = snext;
     }
-  return tmpname;
+  while (s != NULL);
+
+  return ret;
+}
+
+/* Find and optionally add an entry in the adjust_sections list.  */
+
+static struct section_list *
+find_section_list (name, add)
+     const char *name;
+     boolean add;
+{
+  register struct section_list *p;
+
+  for (p = adjust_sections; p != NULL; p = p->next)
+    if (strcmp (p->name, name) == 0)
+      return p;
+
+  if (! add)
+    return NULL;
+
+  p = (struct section_list *) xmalloc (sizeof (struct section_list));
+  p->name = name;
+  p->used = false;
+  p->remove = false;
+  p->adjust = ignore_vma;
+  p->val = 0;
+  p->set_flags = false;
+  p->flags = 0;
+
+  p->next = adjust_sections;
+  adjust_sections = p;
+
+  return p;
 }
 
-/* Make a list of symbols to explicitly strip out. A linked list is 
-   good enough for a small number from the command line, but this will
-   slow things down a lot if many symbols are being deleted. */
+/* Make a list of symbols to explicitly strip out, or to keep.  A
+   linked list is good enough for a small number from the command
+   line, but this will slow things down a lot if many symbols are
+   being deleted. */
 
 struct symlist
 {
@@ -274,8 +360,17 @@ struct symlist
   struct symlist *next;
 };
 
+/* List of symbols to strip.  */
+
 static struct symlist *strip_specific_list = NULL;
 
+/* If this is false, we strip the symbols in strip_specific_list.
+   Otherwise, we keep only the symbols in the list.  */
+
+static boolean keep_symbols = false;
+
+/* Add a symbol to strip_specific_list.  */
+
 static void 
 add_strip_symbol (name)
      const char *name;
@@ -288,7 +383,10 @@ add_strip_symbol (name)
   strip_specific_list = tmp_list;
 }
 
-static int
+/* See whether a symbol should be stripped or kept based on
+   strip_specific_list and keep_symbols.  */
+
+static boolean
 is_strip_symbol (name)
      const char *name;
 {
@@ -297,9 +395,31 @@ is_strip_symbol (name)
   for (tmp_list = strip_specific_list; tmp_list; tmp_list = tmp_list->next)
     {
       if (strcmp (name, tmp_list->name) == 0)
-       return 1;
+       return keep_symbols ? false : true;
     }
-  return 0;
+  return keep_symbols;
+}
+
+/* See if a section is being removed.  */
+
+static boolean
+is_strip_section (abfd, sec)
+     bfd *abfd;
+     asection *sec;
+{
+  struct section_list *p;
+
+  if ((bfd_get_section_flags (abfd, sec) & SEC_DEBUGGING) != 0
+      && (strip_symbols == strip_debug
+         || strip_symbols == strip_unneeded
+         || strip_symbols == strip_all
+         || discard_locals == locals_all))
+    return true;
+
+  if (! sections_removed)
+    return false;
+  p = find_section_list (bfd_get_section_name (abfd, sec), false);
+  return p != NULL && p->remove ? true : false;
 }
 
 /* Choose which symbol entries to copy; put the result in OSYMS.
@@ -321,20 +441,25 @@ filter_symbols (abfd, osyms, isyms, symcount)
       flagword flags = sym->flags;
       int keep;
 
-      if ((flags & BSF_GLOBAL) /* Keep if external.  */
-         || (flags & BSF_KEEP) /* Keep if used in a relocation.  */
-         || bfd_is_und_section (bfd_get_section (sym))
-         || bfd_is_com_section (bfd_get_section (sym)))
+      if ((flags & BSF_KEEP) != 0)             /* Used in relocation.  */
        keep = 1;
+      else if ((flags & BSF_GLOBAL) != 0       /* Global symbol.  */
+              || bfd_is_und_section (bfd_get_section (sym))
+              || bfd_is_com_section (bfd_get_section (sym)))
+       keep = strip_symbols != strip_unneeded;
       else if ((flags & BSF_DEBUGGING) != 0)   /* Debugging symbol.  */
-       keep = strip_symbols != strip_debug;
+       keep = (strip_symbols != strip_debug
+               && strip_symbols != strip_unneeded);
       else                     /* Local symbol.  */
-       keep = discard_locals != locals_all
-         && (discard_locals != locals_start_L ||
-             ! bfd_is_local_label (abfd, sym));
+       keep = (strip_symbols != strip_unneeded
+               && (discard_locals != locals_all
+                   && (discard_locals != locals_start_L
+                       || ! bfd_is_local_label (abfd, sym))));
 
       if (keep && is_strip_symbol (bfd_asymbol_name (sym)))
        keep = 0;
+      if (keep && is_strip_section (abfd, bfd_get_section (sym)))
+       keep = 0;
 
       if (keep)
        to[dst_count++] = sym;
@@ -417,6 +542,51 @@ copy_object (ibfd, obfd)
      any output is done.  Thus, we traverse all sections multiple times.  */
   bfd_map_over_sections (ibfd, setup_section, (void *) obfd);
 
+  if (add_sections != NULL)
+    {
+      struct section_add *padd;
+      struct section_list *pset;
+
+      for (padd = add_sections; padd != NULL; padd = padd->next)
+       {
+         padd->section = bfd_make_section (obfd, padd->name);
+         if (padd->section == NULL)
+           {
+             fprintf (stderr, "%s: can't create section `%s': %s\n",
+                      program_name, padd->name,
+                      bfd_errmsg (bfd_get_error ()));
+             status = 1;
+             return;
+           }
+         else
+           {
+             flagword flags;
+             
+             if (! bfd_set_section_size (obfd, padd->section, padd->size))
+               nonfatal (bfd_get_filename (obfd));
+
+             pset = find_section_list (padd->name, false);
+             if (pset != NULL)
+               pset->used = true;
+
+             if (pset != NULL && pset->set_flags)
+               flags = pset->flags | SEC_HAS_CONTENTS;
+             else
+               flags = SEC_HAS_CONTENTS | SEC_READONLY | SEC_DATA;
+             if (! bfd_set_section_flags (obfd, padd->section, flags))
+               nonfatal (bfd_get_filename (obfd));
+
+             if (pset != NULL
+                 && (pset->adjust == adjust_vma
+                     || pset->adjust == set_vma))
+               {
+                 if (! bfd_set_section_vma (obfd, padd->section, pset->val))
+                   nonfatal (bfd_get_filename (obfd));
+               }
+           }
+       }
+    }
+
   if (gap_fill_set || pad_to_set)
     {
       asection **set;
@@ -503,7 +673,7 @@ copy_object (ibfd, obfd)
 
   /* Symbol filtering must happen after the output sections have
      been created, but before their contents are set.  */
-  if (strip_symbols == strip_all && discard_locals == locals_undef)
+  if (strip_symbols == strip_all)
     {
       osympp = isympp = NULL;
       symcount = 0;
@@ -526,8 +696,10 @@ copy_object (ibfd, obfd)
        }
 
       if (strip_symbols == strip_debug 
+         || strip_symbols == strip_unneeded
          || discard_locals != locals_undef
-         || strip_specific_list)
+         || strip_specific_list != NULL
+         || sections_removed)
        {
          /* Mark symbols used in output relocations so that they
             are kept, even if they are local labels or static symbols.
@@ -550,6 +722,20 @@ copy_object (ibfd, obfd)
   /* This has to happen after the symbol table has been set.  */
   bfd_map_over_sections (ibfd, copy_section, (void *) obfd);
 
+  if (add_sections != NULL)
+    {
+      struct section_add *padd;
+
+      for (padd = add_sections; padd != NULL; padd = padd->next)
+       {
+         if (! bfd_set_section_contents (obfd, padd->section,
+                                         (PTR) padd->contents,
+                                         (file_ptr) 0,
+                                         (bfd_size_type) padd->size))
+           nonfatal (bfd_get_filename (obfd));
+       }
+    }
+
   if (gap_fill_set || pad_to_set)
     {
       bfd_byte *buf;
@@ -560,7 +746,7 @@ copy_object (ibfd, obfd)
       if (max_gap > 8192)
        max_gap = 8192;
       buf = (bfd_byte *) xmalloc (max_gap);
-      memset (buf, gap_fill, max_gap);
+      memset (buf, gap_fill, (size_t) max_gap);
 
       c = bfd_count_sections (obfd);
       for (i = 0; i < c; i++)
@@ -634,13 +820,18 @@ copy_archive (ibfd, obfd, output_target)
     {
       struct name_list *next;
       char *name;
+      bfd *obfd;
     } *list, *l;
   bfd **ptr = &obfd->archive_head;
   bfd *this_element;
   char *dir = make_tempname (bfd_get_filename (obfd));
 
   /* Make a temp directory to hold the contents.  */
-  mkdir (dir, 0700);
+  if (mkdir (dir, 0700) != 0)
+    {
+      fatal ("cannot mkdir %s for archive copying (error: %s)",
+            dir, strerror (errno));
+    }
   obfd->has_armap = ibfd->has_armap;
 
   list = NULL;
@@ -677,6 +868,8 @@ copy_archive (ibfd, obfd, output_target)
       /* Open the newly output file and attach to our list.  */
       output_bfd = bfd_openr (output_name, output_target);
 
+      l->obfd = output_bfd;
+
       *ptr = output_bfd;
       ptr = &output_bfd->next;
 
@@ -693,15 +886,18 @@ copy_archive (ibfd, obfd, output_target)
       nonfatal (bfd_get_filename (obfd));
     }
 
-  /* Delete all the files that we opened.  */
-  for (l = list; l != NULL; l = l->next)
-    unlink (l->name);
-  rmdir (dir);
-
   if (!bfd_close (ibfd))
     {
       nonfatal (bfd_get_filename (ibfd));
     }
+
+  /* Delete all the files that we opened.  */
+  for (l = list; l != NULL; l = l->next)
+    {
+      bfd_close (l->obfd);
+      unlink (l->name);
+    }
+  rmdir (dir);
 }
 
 /* The top-level control.  */
@@ -793,22 +989,22 @@ setup_section (ibfd, isection, obfdarg)
   struct section_list *p;
   sec_ptr osection;
   bfd_vma vma;
+  flagword flags;
   char *err;
 
   if ((bfd_get_section_flags (ibfd, isection) & SEC_DEBUGGING) != 0
       && (strip_symbols == strip_debug
+         || strip_symbols == strip_unneeded
          || strip_symbols == strip_all
          || discard_locals == locals_all))
     return;
 
-  for (p = remove_sections; p != NULL; p = p->next)
-    {
-      if (strcmp (p->name, bfd_section_name (ibfd, isection)) == 0)
-       {
-         p->used = true;
-         return;
-       }
-    }
+  p = find_section_list (bfd_section_name (ibfd, isection), false);
+  if (p != NULL)
+    p->used = true;
+
+  if (p != NULL && p->remove)
+    return;
 
   osection = bfd_make_section_anyway (obfd, bfd_section_name (ibfd, isection));
   if (osection == NULL)
@@ -826,21 +1022,12 @@ setup_section (ibfd, isection, obfdarg)
     }
 
   vma = bfd_section_vma (ibfd, isection);
-  for (p = adjust_sections; p != NULL; p = p->next)
-    {
-      if (strcmp (p->name, bfd_section_name (ibfd, isection)) == 0)
-       {
-         if (p->adjust)
-           vma += p->val;
-         else
-           vma = p->val;
-         p->used = true;
-         break;
-       }
-    }
-  if (p == NULL)
+  if (p != NULL && p->adjust == adjust_vma)
+    vma += p->val;
+  else if (p != NULL && p->adjust == set_vma)
+    vma = p->val;
+  else
     vma += adjust_section_vma;
-
   if (! bfd_set_section_vma (obfd, osection, vma))
     {
       err = "vma";
@@ -856,8 +1043,10 @@ setup_section (ibfd, isection, obfdarg)
       goto loser;
     }
 
-  if (!bfd_set_section_flags (obfd, osection,
-                             bfd_get_section_flags (ibfd, isection)))
+  flags = bfd_get_section_flags (ibfd, isection);
+  if (p != NULL && p->set_flags)
+    flags = p->flags | (flags & SEC_HAS_CONTENTS);
+  if (!bfd_set_section_flags (obfd, osection, flags))
     {
       err = "flags";
       goto loser;
@@ -907,15 +1096,17 @@ copy_section (ibfd, isection, obfdarg)
 
   if ((bfd_get_section_flags (ibfd, isection) & SEC_DEBUGGING) != 0
       && (strip_symbols == strip_debug
+         || strip_symbols == strip_unneeded
          || strip_symbols == strip_all
          || discard_locals == locals_all))
     {
       return;
     }
 
-  for (p = remove_sections; p != NULL; p = p->next)
-    if (strcmp (p->name, bfd_section_name (ibfd, isection)) == 0)
-      return;
+  p = find_section_list (bfd_section_name (ibfd, isection), false);
+
+  if (p != NULL && p->remove)
+    return;
 
   osection = isection->output_section;
   size = bfd_get_section_size_before_reloc (isection);
@@ -1062,6 +1253,9 @@ mark_symbols_used_in_relocations (ibfd, isection, symbolsarg)
   if (relsize < 0)
     bfd_fatal (bfd_get_filename (ibfd));
 
+  if (relsize == 0)
+    return 0;
+
   relpp = (arelent **) xmalloc (relsize);
   relcount = bfd_canonicalize_reloc (ibfd, isection, relpp, symbols);
   if (relcount < 0)
@@ -1092,6 +1286,7 @@ simple_copy (from, to)
      char *from, *to;
 {
   int fromfd, tofd, nread;
+  int saved;
   char buf[COPY_BUF];
 
   fromfd = open (from, O_RDONLY);
@@ -1100,22 +1295,30 @@ simple_copy (from, to)
   tofd = open (to, O_WRONLY | O_CREAT | O_TRUNC);
   if (tofd < 0)
     {
+      saved = errno;
       close (fromfd);
+      errno = saved;
       return -1;
     }
   while ((nread = read (fromfd, buf, sizeof buf)) > 0)
     {
       if (write (tofd, buf, nread) != nread)
        {
+         saved = errno;
          close (fromfd);
          close (tofd);
+         errno = saved;
          return -1;
        }
     }
+  saved = errno;
   close (fromfd);
   close (tofd);
   if (nread < 0)
-    return -1;
+    {
+      errno = saved;
+      return -1;
+    }
   return 0;
 }
 
@@ -1153,12 +1356,27 @@ smart_rename (from, to)
          chmod (to, s.st_mode & 07777);
          chown (to, s.st_uid, s.st_gid);
        }
+      else
+       {
+         /* We have to clean up here. */
+         int saved = errno;
+         fprintf (stderr, "%s: %s: ", program_name, to);
+         errno = saved;
+         perror ("rename");
+         unlink (from);
+       }
     }
   else
     {
       ret = simple_copy (from, to);
-      if (ret == 0)
-       unlink (from);
+      if (ret != 0)
+       {
+         int saved = errno;
+         fprintf (stderr, "%s: %s: ", program_name, to);
+         errno = saved;
+         perror ("simple_copy");
+       }
+      unlink (from);
     }
   return ret;
 }
@@ -1171,8 +1389,9 @@ strip_main (argc, argv)
   char *input_target = NULL, *output_target = NULL;
   boolean show_version = false;
   int c, i;
+  struct section_list *p;
 
-  while ((c = getopt_long (argc, argv, "I:O:F:R:sSgxXVvN:",
+  while ((c = getopt_long (argc, argv, "I:O:F:K:N:R:sSgxXVv",
                           strip_options, (int *) 0)) != EOF)
     {
       switch (c)
@@ -1187,15 +1406,9 @@ strip_main (argc, argv)
          input_target = output_target = optarg;
          break;
        case 'R':
-         {
-           struct section_list *n;
-
-           n = (struct section_list *) xmalloc (sizeof (struct section_list));
-           n->name = optarg;
-           n->used = false;
-           n->next = remove_sections;
-           remove_sections = n;
-         }
+         p = find_section_list (optarg, true);
+         p->remove = true;
+         sections_removed = true;
          break;
        case 's':
          strip_symbols = strip_all;
@@ -1204,7 +1417,26 @@ strip_main (argc, argv)
        case 'g':
          strip_symbols = strip_debug;
          break;
+       case OPTION_STRIP_UNNEEDED:
+         strip_symbols = strip_unneeded;
+         break;
+       case 'K':
+         if (! keep_symbols && strip_specific_list != NULL)
+           {
+             fprintf (stderr, "%s: Can not specify both -K and -N\n",
+                      program_name);
+             strip_usage (stderr, 1);
+           }
+         keep_symbols = true;
+         add_strip_symbol (optarg);
+         break;
        case 'N':
+         if (keep_symbols)
+           {
+             fprintf (stderr, "%s: Can not specify both -K and -N\n",
+                      program_name);
+             strip_usage (stderr, 1);
+           }
          add_strip_symbol (optarg);
          break;
        case 'x':
@@ -1279,7 +1511,7 @@ copy_main (argc, argv)
   int c;
   struct section_list *p;
 
-  while ((c = getopt_long (argc, argv, "b:i:I:s:O:d:F:R:SgxXVvN:",
+  while ((c = getopt_long (argc, argv, "b:i:I:K:N:s:O:d:F:R:SgxXVv",
                           copy_options, (int *) 0)) != EOF)
     {
       switch (c)
@@ -1314,11 +1546,9 @@ copy_main (argc, argv)
          input_target = output_target = optarg;
          break;
        case 'R':
-         p = (struct section_list *) xmalloc (sizeof (struct section_list));
-         p->name = optarg;
-         p->used = false;
-         p->next = remove_sections;
-         remove_sections = p;
+         p = find_section_list (optarg, true);
+         p->remove = true;
+         sections_removed = true;
          break;
        case 'S':
          strip_symbols = strip_all;
@@ -1326,7 +1556,26 @@ copy_main (argc, argv)
        case 'g':
          strip_symbols = strip_debug;
          break;
+       case OPTION_STRIP_UNNEEDED:
+         strip_symbols = strip_unneeded;
+         break;
+       case 'K':
+         if (! keep_symbols && strip_specific_list != NULL)
+           {
+             fprintf (stderr, "%s: Can not specify both -K and -N\n",
+                      program_name);
+             strip_usage (stderr, 1);
+           }
+         keep_symbols = true;
+         add_strip_symbol (optarg);
+         break;
        case 'N':
+         if (keep_symbols)
+           {
+             fprintf (stderr, "%s: Can not specify both -K and -N\n",
+                      program_name);
+             strip_usage (stderr, 1);
+           }
          add_strip_symbol (optarg);
          break;
        case 'x':
@@ -1341,6 +1590,64 @@ copy_main (argc, argv)
        case 'V':
          show_version = true;
          break;
+       case OPTION_ADD_SECTION:
+         {
+           const char *s;
+           struct stat st;
+           struct section_add *pa;
+           int len;
+           char *name;
+           FILE *f;
+
+           s = strchr (optarg, '=');
+           if (s == NULL)
+             {
+               fprintf (stderr,
+                        "%s: bad format for --add-section NAME=FILENAME\n",
+                        program_name);
+               exit (1);
+             }
+
+           if (stat (s + 1, &st) < 0)
+             {
+               fprintf (stderr, "%s: ", program_name);
+               perror (s + 1);
+               exit (1);
+             }
+
+           pa = (struct section_add *) xmalloc (sizeof (struct section_add));
+
+           len = s - optarg;
+           name = (char *) xmalloc (len + 1);
+           strncpy (name, optarg, len);
+           name[len] = '\0';
+           pa->name = name;
+
+           pa->filename = s + 1;
+
+           pa->size = st.st_size;
+
+           pa->contents = (bfd_byte *) xmalloc (pa->size);
+           f = fopen (pa->filename, FOPEN_RB);
+           if (f == NULL)
+             {
+               fprintf (stderr, "%s: ", program_name);
+               perror (pa->filename);
+               exit (1);
+             }
+           if (fread (pa->contents, 1, pa->size, f) == 0
+               || ferror (f))
+             {
+               fprintf (stderr, "%s: %s: fread failed\n",
+                        program_name, pa->filename);
+               exit (1);
+             }
+           fclose (f);
+
+           pa->next = add_sections;
+           add_sections = pa;
+         }
+         break;
        case OPTION_ADJUST_START:
          adjust_start = parse_vma (optarg, "--adjust-start");
          break;
@@ -1350,14 +1657,8 @@ copy_main (argc, argv)
            int len;
            char *name;
 
-           p = (struct section_list *) xmalloc (sizeof (struct section_list));
            s = strchr (optarg, '=');
-           if (s != NULL)
-             {
-               p->adjust = false;
-               p->val = parse_vma (s + 1, "--adjust-section-vma");
-             }
-           else
+           if (s == NULL)
              {
                s = strchr (optarg, '+');
                if (s == NULL)
@@ -1371,22 +1672,25 @@ copy_main (argc, argv)
                        exit (1);
                      }
                  }
-               p->adjust = true;
-               p->val = parse_vma (s + 1, "--adjust-section-vma");
-               if (*s == '-')
-                 p->val = - p->val;
              }
 
            len = s - optarg;
            name = (char *) xmalloc (len + 1);
            strncpy (name, optarg, len);
            name[len] = '\0';
-           p->name = name;
 
-           p->used = false;
+           p = find_section_list (name, true);
+
+           p->val = parse_vma (s + 1, "--adjust-section-vma");
 
-           p->next = adjust_sections;
-           adjust_sections = p;
+           if (*s == '=')
+             p->adjust = set_vma;
+           else
+             {
+               p->adjust = adjust_vma;
+               if (*s == '-')
+                 p->val = - p->val;
+             }
          }
          break;
        case OPTION_ADJUST_VMA:
@@ -1419,6 +1723,31 @@ copy_main (argc, argv)
          pad_to = parse_vma (optarg, "--pad-to");
          pad_to_set = true;
          break;
+       case OPTION_SET_SECTION_FLAGS:
+         {
+           const char *s;
+           int len;
+           char *name;
+
+           s = strchr (optarg, '=');
+           if (s == NULL)
+             {
+               fprintf (stderr, "%s: bad format for --set-section-flags\n",
+                        program_name);
+               exit (1);
+             }
+
+           len = s - optarg;
+           name = (char *) xmalloc (len + 1);
+           strncpy (name, optarg, len);
+           name[len] = '\0';
+
+           p = find_section_list (name, true);
+
+           p->set_flags = true;
+           p->flags = parse_flags (s + 1);
+         }
+         break;
        case OPTION_SET_START:
          set_start = parse_vma (optarg, "--set-start");
          set_start_set = true;
@@ -1480,20 +1809,17 @@ copy_main (argc, argv)
     {
       for (p = adjust_sections; p != NULL; p = p->next)
        {
-         if (! p->used)
+         if (! p->used && p->adjust != ignore_vma)
            {
              fprintf (stderr, "%s: warning: --adjust-section-vma %s%c0x",
                       program_name, p->name,
-                      p->adjust ? '=' : '+');
+                      p->adjust == set_vma ? '=' : '+');
              fprintf_vma (stderr, p->val);
              fprintf (stderr, " never used\n");
            }
        }
     }
 
-  /* We could issue similar warnings for remove_sections, but I don't
-     think that would be as useful.  */
-
   return 0;
 }
 
@@ -1504,6 +1830,9 @@ main (argc, argv)
 {
   program_name = argv[0];
   xmalloc_set_program_name (program_name);
+
+  START_PROGRESS (program_name, 0);
+
   strip_symbols = strip_undef;
   discard_locals = locals_undef;
 
@@ -1520,5 +1849,7 @@ main (argc, argv)
   else
     copy_main (argc, argv);
 
+  END_PROGRESS (program_name);
+
   return status;
 }
This page took 0.034838 seconds and 4 git commands to generate.