[ARC] Fix parsing leave_s and enter_s mnemonics.
[deliverable/binutils-gdb.git] / binutils / objcopy.c
index 8185f2830c89e207c771029756a3a7555f6f5a95..1821c0ce1be5c1cd39106b9b9fe67fa22c8a6c21 100644 (file)
@@ -28,7 +28,6 @@
 #include "filenames.h"
 #include "fnmatch.h"
 #include "elf-bfd.h"
-#include "libbfd.h"
 #include "coff/internal.h"
 #include "libcoff.h"
 
@@ -139,6 +138,7 @@ struct section_list
 #define SECTION_CONTEXT_SET_LMA   (1 << 4) /* Set the sections' LMA address.  */
 #define SECTION_CONTEXT_ALTER_LMA (1 << 5) /* Increment or decrement the section's LMA address.  */
 #define SECTION_CONTEXT_SET_FLAGS (1 << 6) /* Set the section's flags.  */
+#define SECTION_CONTEXT_REMOVE_RELOCS (1 << 7) /* Remove relocations for this section.  */
 
   bfd_vma              vma_val;   /* Amount to change by or set to.  */
   bfd_vma              lma_val;   /* Amount to change by or set to.  */
@@ -220,6 +220,9 @@ static enum
   decompress = 1 << 4
 } do_debug_sections = nothing;
 
+/* Whether to generate ELF common symbols with the STT_COMMON type.  */
+static enum bfd_link_elf_stt_common do_elf_stt_common = unchanged;
+
 /* Whether to change the leading character in symbol names.  */
 static bfd_boolean change_leading_char = FALSE;
 
@@ -294,6 +297,7 @@ enum command_line_switch
   OPTION_DEBUGGING,
   OPTION_DECOMPRESS_DEBUG_SECTIONS,
   OPTION_DUMP_SECTION,
+  OPTION_ELF_STT_COMMON,
   OPTION_EXTRACT_DWO,
   OPTION_EXTRACT_SYMBOL,
   OPTION_FILE_ALIGNMENT,
@@ -322,6 +326,7 @@ enum command_line_switch
   OPTION_REDEFINE_SYM,
   OPTION_REDEFINE_SYMS,
   OPTION_REMOVE_LEADING_CHAR,
+  OPTION_REMOVE_RELOCS,
   OPTION_RENAME_SECTION,
   OPTION_REVERSE_BYTES,
   OPTION_SECTION_ALIGNMENT,
@@ -363,6 +368,7 @@ static struct option strip_options[] =
   {"output-target", required_argument, 0, 'O'},
   {"preserve-dates", no_argument, 0, 'p'},
   {"remove-section", required_argument, 0, 'R'},
+  {"remove-relocations", required_argument, 0, OPTION_REMOVE_RELOCS},
   {"strip-all", no_argument, 0, 's'},
   {"strip-debug", no_argument, 0, 'S'},
   {"strip-dwo", no_argument, 0, OPTION_STRIP_DWO},
@@ -403,6 +409,7 @@ static struct option copy_options[] =
   {"discard-all", no_argument, 0, 'x'},
   {"discard-locals", no_argument, 0, 'X'},
   {"dump-section", required_argument, 0, OPTION_DUMP_SECTION},
+  {"elf-stt-common", required_argument, 0, OPTION_ELF_STT_COMMON},
   {"enable-deterministic-archives", no_argument, 0, 'D'},
   {"extract-dwo", no_argument, 0, OPTION_EXTRACT_DWO},
   {"extract-symbol", no_argument, 0, OPTION_EXTRACT_SYMBOL},
@@ -446,6 +453,7 @@ static struct option copy_options[] =
   {"redefine-syms", required_argument, 0, OPTION_REDEFINE_SYMS},
   {"remove-leading-char", no_argument, 0, OPTION_REMOVE_LEADING_CHAR},
   {"remove-section", required_argument, 0, 'R'},
+  {"remove-relocations", required_argument, 0, OPTION_REMOVE_RELOCS},
   {"rename-section", required_argument, 0, OPTION_RENAME_SECTION},
   {"reverse-bytes", required_argument, 0, OPTION_REVERSE_BYTES},
   {"section-alignment", required_argument, 0, OPTION_SECTION_ALIGNMENT},
@@ -502,8 +510,9 @@ static int compare_section_lma (const void *, const void *);
 static void mark_symbols_used_in_relocations (bfd *, asection *, void *);
 static bfd_boolean write_debugging_info (bfd *, void *, long *, asymbol ***);
 static const char *lookup_sym_redefinition (const char *);
+static const char *find_section_rename (const char *, flagword *);
 \f
-static void
+ATTRIBUTE_NORETURN static void
 copy_usage (FILE *stream, int exit_status)
 {
   fprintf (stream, _("Usage: %s [option(s)] in-file [out-file]\n"), program_name);
@@ -532,6 +541,7 @@ copy_usage (FILE *stream, int exit_status)
   -j --only-section <name>         Only copy section <name> into the output\n\
      --add-gnu-debuglink=<file>    Add section .gnu_debuglink linking to <file>\n\
   -R --remove-section <name>       Remove section <name> from the output\n\
+     --remove-relocations <name>   Remove relocations from section <name>\n\
   -S --strip-all                   Remove all symbol and relocation information\n\
   -g --strip-debug                 Remove all debugging symbols & sections\n\
      --strip-dwo                   Remove all DWO sections\n\
@@ -622,6 +632,8 @@ copy_usage (FILE *stream, int exit_status)
      --compress-debug-sections[={none|zlib|zlib-gnu|zlib-gabi}]\n\
                                    Compress DWARF debug sections using zlib\n\
      --decompress-debug-sections   Decompress DWARF debug sections using zlib\n\
+     --elf-stt-common=[yes|no]     Generate ELF common symbols with STT_COMMON\n\
+                                     type\n\
   -v --verbose                     List all object files modified\n\
   @<file>                          Read options from <file>\n\
   -V --version                     Display this program's version number\n\
@@ -634,7 +646,7 @@ copy_usage (FILE *stream, int exit_status)
   exit (exit_status);
 }
 
-static void
+ATTRIBUTE_NORETURN static void
 strip_usage (FILE *stream, int exit_status)
 {
   fprintf (stream, _("Usage: %s <option(s)> in-file(s)\n"), program_name);
@@ -660,6 +672,7 @@ strip_usage (FILE *stream, int exit_status)
                                    Disable -D behavior (default)\n"));
   fprintf (stream, _("\
   -R --remove-section=<name>       Also remove section <name> from the output\n\
+     --remove-relocations <name>   Remove relocations from section <name>\n\
   -s --strip-all                   Remove all symbol and relocation information\n\
   -g -S -d --strip-debug           Remove all debugging symbols & sections\n\
      --strip-dwo                   Remove all DWO sections\n\
@@ -708,8 +721,8 @@ parse_flags (const char *s)
        }
 
       if (0) ;
-#define PARSE_FLAG(fname,fval) \
-  else if (strncasecmp (fname, s, len) == 0) ret |= fval
+#define PARSE_FLAG(fname,fval)                                 \
+      else if (strncasecmp (fname, s, len) == 0) ret |= fval
       PARSE_FLAG ("alloc", SEC_ALLOC);
       PARSE_FLAG ("load", SEC_LOAD);
       PARSE_FLAG ("noload", SEC_NEVER_LOAD);
@@ -750,7 +763,7 @@ parse_symflags (const char *s, char **other)
 {
   flagword ret;
   const char *snext;
-  int len;
+  size_t len;
 
   ret = BSF_NO_FLAGS;
 
@@ -758,21 +771,23 @@ parse_symflags (const char *s, char **other)
     {
       snext = strchr (s, ',');
       if (snext == NULL)
-         len = strlen (s);
+       len = strlen (s);
       else
        {
          len = snext - s;
          ++snext;
        }
 
-#define PARSE_FLAG(fname,fval)                                                   \
-      else if (len == (int) sizeof fname - 1 && strncasecmp (fname, s, len) == 0) \
+#define PARSE_FLAG(fname, fval)                                                \
+      else if (len == sizeof fname - 1                                 \
+              && strncasecmp (fname, s, len) == 0)                     \
        ret |= fval
 
-#define PARSE_OTHER(fname,fval)                                                                   \
-      else if (len >= (int) sizeof fname && strncasecmp (fname, s, sizeof fname - 1) == 0) \
+#define PARSE_OTHER(fname, fval)                                       \
+      else if (len >= sizeof fname                                     \
+              && strncasecmp (fname, s, sizeof fname - 1) == 0)        \
        fval = xstrndup (s + sizeof fname - 1, len - sizeof fname + 1)
-      
+
       if (0) ;
       PARSE_FLAG ("local", BSF_LOCAL);
       PARSE_FLAG ("global", BSF_GLOBAL);
@@ -802,9 +817,9 @@ parse_symflags (const char *s, char **other)
          copy[len] = '\0';
          non_fatal (_("unrecognized symbol flag `%s'"), copy);
          fatal (_("supported flags: %s"),
-                 "local, global, export, debug, function, weak, section, "
-                 "constructor, warning, indirect, file, object, synthetic, "
-                 "indirect-function, unique-object, before=<othersym>");
+                "local, global, export, debug, function, weak, section, "
+                "constructor, warning, indirect, file, object, synthetic, "
+                "indirect-function, unique-object, before=<othersym>");
        }
 
       s = snext;
@@ -844,7 +859,7 @@ parse_symflags (const char *s, char **other)
 static struct section_list *
 find_section_list (const char *name, bfd_boolean add, unsigned int context)
 {
-  struct section_list *p;
+  struct section_list *p, *match = NULL;
 
   /* assert ((context & ((1 << 7) - 1)) != 0); */
 
@@ -880,19 +895,36 @@ find_section_list (const char *name, bfd_boolean add, unsigned int context)
        }
       /* If we are not adding a new name/pattern then
         only check for a match if the context applies.  */
-      else if ((p->context & context)
-              /* We could check for the presence of wildchar characters
-                 first and choose between calling strcmp and fnmatch,
-                 but is that really worth it ?  */
-              && fnmatch (p->pattern, name, 0) == 0)
-       {
-         p->used = TRUE;
-         return p;
-       }
+      else if (p->context & context)
+        {
+          /* We could check for the presence of wildchar characters
+             first and choose between calling strcmp and fnmatch,
+             but is that really worth it ?  */
+          if (p->pattern [0] == '!')
+            {
+              if (fnmatch (p->pattern + 1, name, 0) == 0)
+                {
+                  p->used = TRUE;
+                  return NULL;
+                }
+            }
+          else
+            {
+              if (fnmatch (p->pattern, name, 0) == 0)
+                {
+                  if (match == NULL)
+                    match = p;
+                }
+            }
+        }
     }
 
   if (! add)
-    return NULL;
+    {
+      if (match != NULL)
+        match->used = TRUE;
+      return match;
+    }
 
   p = (struct section_list *) xmalloc (sizeof (struct section_list));
   p->pattern = name;
@@ -1116,6 +1148,10 @@ group_signature (asection *group)
   bfd *abfd = group->owner;
   Elf_Internal_Shdr *ghdr;
 
+  /* PR 20089: An earlier error may have prevented us from loading the symbol table.  */
+  if (isympp == NULL)
+    return NULL;
+
   if (bfd_get_flavour (abfd) != bfd_target_elf_flavour)
     return NULL;
 
@@ -1126,7 +1162,8 @@ group_signature (asection *group)
       Elf_Internal_Shdr *symhdr = elf_elfsections (abfd) [ghdr->sh_link];
 
       if (symhdr->sh_type == SHT_SYMTAB
-         && ghdr->sh_info < symhdr->sh_size / bed->s->sizeof_sym)
+         && ghdr->sh_info > 0
+         && ghdr->sh_info < (symhdr->sh_size / bed->s->sizeof_sym))
        return isympp[ghdr->sh_info - 1];
     }
   return NULL;
@@ -1153,12 +1190,12 @@ is_update_section (bfd *abfd ATTRIBUTE_UNUSED, asection *sec)
       struct section_add *pupdate;
 
       for (pupdate = update_sections;
-           pupdate != NULL;
-           pupdate = pupdate->next)
+          pupdate != NULL;
+          pupdate = pupdate->next)
        {
-          if (strcmp (sec->name, pupdate->name) == 0)
-            return TRUE;
-        }
+         if (strcmp (sec->name, pupdate->name) == 0)
+           return TRUE;
+       }
     }
 
   return FALSE;
@@ -1183,8 +1220,8 @@ is_strip_section_1 (bfd *abfd ATTRIBUTE_UNUSED, asection *sec)
        fatal (_("error: section %s matches both remove and copy options"),
               bfd_get_section_name (abfd, sec));
       if (p && is_update_section (abfd, sec))
-        fatal (_("error: section %s matches both update and remove options"),
-               bfd_get_section_name (abfd, sec));
+       fatal (_("error: section %s matches both update and remove options"),
+              bfd_get_section_name (abfd, sec));
 
       if (p != NULL)
        return TRUE;
@@ -1326,9 +1363,9 @@ need_sym_before (struct addsym_node **node, const char *sym)
 static asymbol *
 create_new_symbol (struct addsym_node *ptr, bfd *obfd)
 {
-  asymbol *sym = bfd_make_empty_symbol(obfd);
+  asymbol *sym = bfd_make_empty_symbol (obfd);
 
-  bfd_asymbol_name(sym) = ptr->symdef;
+  bfd_asymbol_name (sym) = ptr->symdef;
   sym->value = ptr->symval;
   sym->flags = ptr->flags;
   if (ptr->section)
@@ -1338,8 +1375,8 @@ create_new_symbol (struct addsym_node *ptr, bfd *obfd)
        fatal (_("Section %s not found"), ptr->section);
       sym->section = sec;
     }
-  else 
-      sym->section = bfd_abs_section_ptr;
+  else
+    sym->section = bfd_abs_section_ptr;
   return sym;
 }
 
@@ -1376,12 +1413,14 @@ filter_symbols (bfd *abfd, bfd *obfd, asymbol **osyms,
            to[dst_count++] = create_new_symbol (ptr, obfd);
        }
 
-      if (redefine_sym_list)
+      if (redefine_sym_list || section_rename_list)
        {
-         char *old_name, *new_name;
+         char *new_name;
 
-         old_name = (char *) bfd_asymbol_name (sym);
-         new_name = (char *) lookup_sym_redefinition (old_name);
+         new_name = (char *) lookup_sym_redefinition (name);
+         if (new_name == name
+             && (flags & BSF_SECTION_SYM) != 0)
+           new_name = (char *) find_section_rename (name, NULL);
          bfd_asymbol_name (sym) = new_name;
          name = new_name;
        }
@@ -1404,12 +1443,12 @@ filter_symbols (bfd *abfd, bfd *obfd, asymbol **osyms,
 
       /* Short circuit for change_leading_char if we can do it in-place.  */
       if (rem_leading_char && add_leading_char && !prefix_symbols_string)
-        {
+       {
          name[0] = bfd_get_symbol_leading_char (obfd);
          bfd_asymbol_name (sym) = name;
          rem_leading_char = FALSE;
          add_leading_char = FALSE;
-        }
+       }
 
       /* Remove leading char.  */
       if (rem_leading_char)
@@ -1417,23 +1456,23 @@ filter_symbols (bfd *abfd, bfd *obfd, asymbol **osyms,
 
       /* Add new leading char and/or prefix.  */
       if (add_leading_char || prefix_symbols_string)
-        {
-          char *n, *ptr;
+       {
+         char *n, *ptr;
 
-          ptr = n = (char *) xmalloc (1 + strlen (prefix_symbols_string)
-                                      + strlen (name) + 1);
-          if (add_leading_char)
+         ptr = n = (char *) xmalloc (1 + strlen (prefix_symbols_string)
+                                     + strlen (name) + 1);
+         if (add_leading_char)
            *ptr++ = bfd_get_symbol_leading_char (obfd);
 
-          if (prefix_symbols_string)
-            {
-              strcpy (ptr, prefix_symbols_string);
-              ptr += strlen (prefix_symbols_string);
-           }
+         if (prefix_symbols_string)
+           {
+             strcpy (ptr, prefix_symbols_string);
+             ptr += strlen (prefix_symbols_string);
+           }
 
-          strcpy (ptr, name);
-          bfd_asymbol_name (sym) = n;
-          name = n;
+         strcpy (ptr, name);
+         bfd_asymbol_name (sym) = n;
+         name = n;
        }
 
       if (strip_symbols == STRIP_ALL)
@@ -1677,7 +1716,7 @@ add_redefine_syms_file (const char *filename)
       if ((c == '\r' && (c = getc (file)) == '\n')
          || c == '\n' || c == EOF)
        {
- end_of_line:
      end_of_line:
          /* Append the redefinition to the list.  */
          if (buf[0] != '\0')
            redefine_list_append (filename, &buf[0], &buf[outsym_off]);
@@ -1692,7 +1731,7 @@ add_redefine_syms_file (const char *filename)
        }
       else
        fatal (_("%s:%d: garbage found at end of line"), filename, lineno);
- comment:
   comment:
       if (len != 0 && (outsym_off == 0 || outsym_off == len))
        fatal (_("%s:%d: missing new symbol name"), filename, lineno);
       buf[len++] = '\0';
@@ -1820,13 +1859,22 @@ copy_object (bfd *ibfd, bfd *obfd, const bfd_arch_info_type *input_arch)
       return FALSE;
     }
 
-  if ((do_debug_sections & compress) != 0
-      && do_debug_sections != compress
-      && ibfd->xvec->flavour != bfd_target_elf_flavour)
+  if (ibfd->xvec->flavour != bfd_target_elf_flavour)
     {
-      non_fatal (_("--compress-debug-sections=[zlib|zlib-gnu|zlib-gabi] is unsupported on `%s'"),
-                bfd_get_archive_filename (ibfd));
-      return FALSE;
+      if ((do_debug_sections & compress) != 0
+         && do_debug_sections != compress)
+       {
+         non_fatal (_("--compress-debug-sections=[zlib|zlib-gnu|zlib-gabi] is unsupported on `%s'"),
+                    bfd_get_archive_filename (ibfd));
+         return FALSE;
+       }
+
+      if (do_elf_stt_common)
+       {
+         non_fatal (_("--elf-stt-common=[yes|no] is unsupported on `%s'"),
+                    bfd_get_archive_filename (ibfd));
+         return FALSE;
+       }
     }
 
   if (verbose)
@@ -2019,15 +2067,15 @@ copy_object (bfd *ibfd, bfd *obfd, const bfd_arch_info_type *input_arch)
          if (bfd_get_section_by_name (obfd, padd->name))
            {
              bfd_nonfatal_message (NULL, obfd, NULL,
-                                _("can't add section '%s'"), padd->name);
+                                   _("can't add section '%s'"), padd->name);
              return FALSE;
            }
          else
            {
              /* We use LINKER_CREATED here so that the backend hooks
-                will create any special section type information,
-                instead of presuming we know what we're doing merely
-                because we set the flags.  */
+                will create any special section type information,
+                instead of presuming we know what we're doing merely
+                because we set the flags.  */
              padd->section = bfd_make_section_with_flags
                (obfd, padd->name, flags | SEC_LINKER_CREATED);
              if (padd->section == NULL)
@@ -2334,6 +2382,7 @@ copy_object (bfd *ibfd, bfd *obfd, const bfd_arch_info_type *input_arch)
       || change_leading_char
       || remove_leading_char
       || redefine_sym_list
+      || section_rename_list
       || weaken
       || add_symbols)
     {
@@ -2390,14 +2439,14 @@ copy_object (bfd *ibfd, bfd *obfd, const bfd_arch_info_type *input_arch)
       struct section_add *pupdate;
 
       for (pupdate = update_sections;
-           pupdate != NULL;
-           pupdate = pupdate->next)
+          pupdate != NULL;
+          pupdate = pupdate->next)
        {
          asection *osec;
 
          osec = pupdate->section->output_section;
          if (! bfd_set_section_contents (obfd, osec, pupdate->contents,
-                                         0, pupdate->size))
+                                         0, pupdate->size))
            {
              bfd_nonfatal_message (NULL, obfd, osec, NULL);
              return FALSE;
@@ -2519,7 +2568,7 @@ copy_archive (bfd *ibfd, bfd *obfd, const char *output_target,
   /* Make a temp directory to hold the contents.  */
   dir = make_tempdir (bfd_get_filename (obfd));
   if (dir == NULL)
-      fatal (_("cannot create tempdir for archive copying (error: %s)"),
+    fatal (_("cannot create tempdir for archive copying (error: %s)"),
           strerror (errno));
 
   if (strip_symbols == STRIP_ALL)
@@ -2767,13 +2816,26 @@ copy_file (const char *input_filename, const char *output_filename,
       break;
     }
 
+  switch (do_elf_stt_common)
+    {
+    case elf_stt_common:
+      ibfd->flags |= BFD_CONVERT_ELF_COMMON | BFD_USE_ELF_STT_COMMON;
+      break;
+      break;
+    case no_elf_stt_common:
+      ibfd->flags |= BFD_CONVERT_ELF_COMMON;
+      break;
+    default:
+      break;
+    }
+
   if (bfd_check_format (ibfd, bfd_archive))
     {
       bfd_boolean force_output_target;
       bfd *obfd;
 
       /* bfd_get_target does not return the correct value until
-         bfd_check_format succeeds.  */
+        bfd_check_format succeeds.  */
       if (output_target == NULL)
        {
          output_target = bfd_get_target (ibfd);
@@ -2800,7 +2862,7 @@ copy_file (const char *input_filename, const char *output_filename,
     do_copy:
 
       /* bfd_get_target does not return the correct value until
-         bfd_check_format succeeds.  */
+        bfd_check_format succeeds.  */
       if (output_target == NULL)
        output_target = bfd_get_target (ibfd);
 
@@ -2901,24 +2963,19 @@ add_section_rename (const char * old_name, const char * new_name,
 }
 
 /* Check the section rename list for a new name of the input section
-   ISECTION.  Return the new name if one is found.
-   Also set RETURNED_FLAGS to the flags to be used for this section.  */
+   called OLD_NAME.  Returns the new name if one is found and sets
+   RETURNED_FLAGS if non-NULL to the flags to be used for this section.  */
 
 static const char *
-find_section_rename (bfd * ibfd ATTRIBUTE_UNUSED, sec_ptr isection,
-                    flagword * returned_flags)
+find_section_rename (const char *old_name, flagword *returned_flags)
 {
-  const char * old_name = bfd_section_name (ibfd, isection);
-  section_rename * srename;
-
-  /* Default to using the flags of the input section.  */
-  * returned_flags = bfd_get_section_flags (ibfd, isection);
+  const section_rename *srename;
 
   for (srename = section_rename_list; srename != NULL; srename = srename->next)
     if (strcmp (srename->old_name, old_name) == 0)
       {
-       if (srename->flags != (flagword) -1)
-         * returned_flags = srename->flags;
+       if (returned_flags != NULL && srename->flags != (flagword) -1)
+         *returned_flags = srename->flags;
 
        return srename->new_name;
       }
@@ -2968,7 +3025,9 @@ setup_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
     return;
 
   /* Get the, possibly new, name of the output section.  */
-  name = find_section_rename (ibfd, isection, & flags);
+  name = bfd_section_name (ibfd, isection);
+  flags = bfd_get_section_flags (ibfd, isection);
+  name = find_section_rename (name, &flags);
 
   /* Prefix sections.  */
   if ((prefix_alloc_sections_string)
@@ -2995,7 +3054,7 @@ setup_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
     flags = p->flags | (flags & (SEC_HAS_CONTENTS | SEC_RELOC));
   else if (strip_symbols == STRIP_NONDEBUG
           && (flags & (SEC_ALLOC | SEC_GROUP)) != 0
-           && !is_nondebug_keep_contents_section (ibfd, isection))
+          && !is_nondebug_keep_contents_section (ibfd, isection))
     {
       flags &= ~(SEC_HAS_CONTENTS | SEC_LOAD | SEC_GROUP);
       if (obfd->xvec->flavour == bfd_target_elf_flavour)
@@ -3112,7 +3171,7 @@ setup_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
   /* All went well.  */
   return;
 
-loser:
+ loser:
   status = 1;
   bfd_nonfatal_message (NULL, obfd, osection, err);
 }
@@ -3153,6 +3212,46 @@ skip_section (bfd *ibfd, sec_ptr isection)
   return FALSE;
 }
 
+/* Add section SECTION_PATTERN to the list of sections that will have their
+   relocations removed.  */
+
+static void
+handle_remove_relocations_option (const char *section_pattern)
+{
+  find_section_list (section_pattern, TRUE, SECTION_CONTEXT_REMOVE_RELOCS);
+}
+
+/* Return TRUE if ISECTION from IBFD should have its relocations removed,
+   otherwise return FALSE.  If the user has requested that relocations be
+   removed from a section that does not have relocations then this
+   function will still return TRUE.  */
+
+static bfd_boolean
+discard_relocations (bfd *ibfd ATTRIBUTE_UNUSED, asection *isection)
+{
+  return (find_section_list (bfd_section_name (ibfd, isection), FALSE,
+                            SECTION_CONTEXT_REMOVE_RELOCS) != NULL);
+}
+
+/* Wrapper for dealing with --remove-section (-R) command line arguments.
+   A special case is detected here, if the user asks to remove a relocation
+   section (one starting with ".rela." or ".rel.") then this removal must
+   be done using a different technique.  */
+
+static void
+handle_remove_section_option (const char *section_pattern)
+{
+  if (strncmp (section_pattern, ".rela.", 6) == 0)
+    handle_remove_relocations_option (section_pattern + 5);
+  else if (strncmp (section_pattern, ".rel.", 5) == 0)
+    handle_remove_relocations_option (section_pattern + 4);
+  else
+    {
+      find_section_list (section_pattern, TRUE, SECTION_CONTEXT_REMOVE);
+      sections_removed = TRUE;
+    }
+}
+
 /* Copy relocations in input section ISECTION of IBFD to an output
    section with the same name in OBFDARG.  If stripping then don't
    copy any relocation info.  */
@@ -3172,7 +3271,9 @@ copy_relocations_in_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
   osection = isection->output_section;
 
   /* Core files and DWO files do not need to be relocated.  */
-  if (bfd_get_format (obfd) == bfd_core || strip_symbols == STRIP_NONDWO)
+  if (bfd_get_format (obfd) == bfd_core
+      || strip_symbols == STRIP_NONDWO
+      || discard_relocations (ibfd, isection))
     relsize = 0;
   else
     {
@@ -3221,7 +3322,9 @@ copy_relocations_in_section (bfd *ibfd, sec_ptr isection, void *obfdarg)
          for (i = 0; i < relcount; i++)
            {
              /* PR 17512: file: 9e907e0c.  */
-             if (relpp[i]->sym_ptr_ptr)
+             if (relpp[i]->sym_ptr_ptr
+                 /* PR 20096 */
+                 && * relpp[i]->sym_ptr_ptr)
                if (is_specified_symbol (bfd_asymbol_name (*relpp[i]->sym_ptr_ptr),
                                         keep_specific_htab))
                  temp_relpp [temp_relcount++] = relpp [i];
@@ -3494,9 +3597,9 @@ write_debugging_info (bfd *obfd, void *dhandle,
        }
 
       /* We can get away with setting the section contents now because
-         the next thing the caller is going to do is copy over the
-         real sections.  We may someday have to split the contents
-         setting out of this function.  */
+        the next thing the caller is going to do is copy over the
+        real sections.  We may someday have to split the contents
+        setting out of this function.  */
       if (! bfd_set_section_contents (obfd, stabsec, syms, 0, symsize)
          || ! bfd_set_section_contents (obfd, stabstrsec, strings, 0,
                                         stringsize))
@@ -3511,7 +3614,7 @@ write_debugging_info (bfd *obfd, void *dhandle,
 
   bfd_nonfatal_message (NULL, obfd, NULL,
                        _("don't know how to write debugging information for %s"),
-            bfd_get_target (obfd));
+                       bfd_get_target (obfd));
   return FALSE;
 }
 
@@ -3550,8 +3653,10 @@ strip_main (int argc, char *argv[])
          input_target = output_target = optarg;
          break;
        case 'R':
-         find_section_list (optarg, TRUE, SECTION_CONTEXT_REMOVE);
-         sections_removed = TRUE;
+         handle_remove_section_option (optarg);
+         break;
+       case OPTION_REMOVE_RELOCS:
+         handle_remove_relocations_option (optarg);
          break;
        case 's':
          strip_symbols = STRIP_ALL;
@@ -3819,8 +3924,8 @@ convert_efi_target (char *efi)
 
 static struct section_add *
 init_section_add (const char *arg,
-                  struct section_add *next,
-                  const char *option)
+                 struct section_add *next,
+                 const char *option)
 {
   struct section_add *pa;
   const char *s;
@@ -3857,7 +3962,7 @@ section_add_load_file (struct section_add *pa)
   f = fopen (pa->filename, FOPEN_RB);
   if (f == NULL)
     fatal (_("cannot open: %s: %s"),
-           pa->filename, strerror (errno));
+          pa->filename, strerror (errno));
 
   off = 0;
   alloc = 4096;
@@ -3867,14 +3972,14 @@ section_add_load_file (struct section_add *pa)
       off_t got;
 
       if (off == alloc)
-        {
-          alloc <<= 1;
-          pa->contents = (bfd_byte *) xrealloc (pa->contents, alloc);
-        }
+       {
+         alloc <<= 1;
+         pa->contents = (bfd_byte *) xrealloc (pa->contents, alloc);
+       }
 
       got = fread (pa->contents + off, 1, alloc - off, f);
       if (ferror (f))
-        fatal (_("%s: fread failed"), pa->filename);
+       fatal (_("%s: fread failed"), pa->filename);
 
       off += got;
     }
@@ -3953,8 +4058,11 @@ copy_main (int argc, char *argv[])
          break;
 
        case 'R':
-         find_section_list (optarg, TRUE, SECTION_CONTEXT_REMOVE);
-         sections_removed = TRUE;
+         handle_remove_section_option (optarg);
+         break;
+
+        case OPTION_REMOVE_RELOCS:
+         handle_remove_relocations_option (optarg);
          break;
 
        case 'S':
@@ -4055,20 +4163,20 @@ copy_main (int argc, char *argv[])
          break;
 
        case OPTION_ADD_SECTION:
-          add_sections = init_section_add (optarg, add_sections,
-                                           "--add-section");
-          section_add_load_file (add_sections);
+         add_sections = init_section_add (optarg, add_sections,
+                                          "--add-section");
+         section_add_load_file (add_sections);
          break;
 
        case OPTION_UPDATE_SECTION:
          update_sections = init_section_add (optarg, update_sections,
-                                              "--update-section");
+                                             "--update-section");
          section_add_load_file (update_sections);
          break;
 
        case OPTION_DUMP_SECTION:
-          dump_sections = init_section_add (optarg, dump_sections,
-                                            "--dump-section");
+         dump_sections = init_section_add (optarg, dump_sections,
+                                           "--dump-section");
          break;
 
        case OPTION_ADD_SYMBOL:
@@ -4096,6 +4204,7 @@ copy_main (int argc, char *argv[])
              }
 
            t = strchr (t + 1, ',');
+           newsym->othersym = NULL;
            if (t)
              newsym->flags = parse_symflags (t+1, &newsym->othersym);
            else
@@ -4193,7 +4302,7 @@ copy_main (int argc, char *argv[])
              {
              case OPTION_CHANGE_SECTION_ADDRESS:
                p->vma_val = val;
-               /* Drop through.  */
+               /* Fall through.  */
 
              case OPTION_CHANGE_SECTION_LMA:
                p->lma_val = val;
@@ -4246,6 +4355,16 @@ copy_main (int argc, char *argv[])
          do_debug_sections = decompress;
          break;
 
+       case OPTION_ELF_STT_COMMON:
+         if (strcasecmp (optarg, "yes") == 0)
+           do_elf_stt_common = elf_stt_common;
+         else if (strcasecmp (optarg, "no") == 0)
+           do_elf_stt_common = no_elf_stt_common;
+         else
+           fatal (_("unrecognized --elf-stt-common= option `%s'"),
+                  optarg);
+         break;
+
        case OPTION_GAP_FILL:
          {
            bfd_vma gap_fill_vma;
@@ -4481,39 +4600,39 @@ copy_main (int argc, char *argv[])
          break;
 
        case OPTION_REVERSE_BYTES:
-          {
-            int prev = reverse_bytes;
+         {
+           int prev = reverse_bytes;
 
-            reverse_bytes = atoi (optarg);
-            if ((reverse_bytes <= 0) || ((reverse_bytes % 2) != 0))
-              fatal (_("number of bytes to reverse must be positive and even"));
+           reverse_bytes = atoi (optarg);
+           if ((reverse_bytes <= 0) || ((reverse_bytes % 2) != 0))
+             fatal (_("number of bytes to reverse must be positive and even"));
 
-            if (prev && prev != reverse_bytes)
-              non_fatal (_("Warning: ignoring previous --reverse-bytes value of %d"),
-                         prev);
-            break;
-          }
+           if (prev && prev != reverse_bytes)
+             non_fatal (_("Warning: ignoring previous --reverse-bytes value of %d"),
+                        prev);
+           break;
+         }
 
        case OPTION_FILE_ALIGNMENT:
          pe_file_alignment = parse_vma (optarg, "--file-alignment");
          break;
 
        case OPTION_HEAP:
-           {
-             char *end;
-             pe_heap_reserve = strtoul (optarg, &end, 0);
-             if (end == optarg
-                 || (*end != '.' && *end != '\0'))
-               non_fatal (_("%s: invalid reserve value for --heap"),
-                          optarg);
-             else if (*end != '\0')
-               {
-                 pe_heap_commit = strtoul (end + 1, &end, 0);
-                 if (*end != '\0')
-                   non_fatal (_("%s: invalid commit value for --heap"),
-                              optarg);
-               }
-           }
+         {
+           char *end;
+           pe_heap_reserve = strtoul (optarg, &end, 0);
+           if (end == optarg
+               || (*end != '.' && *end != '\0'))
+             non_fatal (_("%s: invalid reserve value for --heap"),
+                        optarg);
+           else if (*end != '\0')
+             {
+               pe_heap_commit = strtoul (end + 1, &end, 0);
+               if (*end != '\0')
+                 non_fatal (_("%s: invalid commit value for --heap"),
+                            optarg);
+             }
+         }
          break;
 
        case OPTION_IMAGE_BASE:
@@ -4530,21 +4649,21 @@ copy_main (int argc, char *argv[])
          break;
 
        case OPTION_STACK:
-           {
-             char *end;
-             pe_stack_reserve = strtoul (optarg, &end, 0);
-             if (end == optarg
-                 || (*end != '.' && *end != '\0'))
-               non_fatal (_("%s: invalid reserve value for --stack"),
-                          optarg);
-             else if (*end != '\0')
-               {
-                 pe_stack_commit = strtoul (end + 1, &end, 0);
-                 if (*end != '\0')
-                   non_fatal (_("%s: invalid commit value for --stack"),
-                              optarg);
-               }
-           }
+         {
+           char *end;
+           pe_stack_reserve = strtoul (optarg, &end, 0);
+           if (end == optarg
+               || (*end != '.' && *end != '\0'))
+             non_fatal (_("%s: invalid reserve value for --stack"),
+                        optarg);
+           else if (*end != '\0')
+             {
+               pe_stack_commit = strtoul (end + 1, &end, 0);
+               if (*end != '\0')
+                 non_fatal (_("%s: invalid commit value for --stack"),
+                            optarg);
+             }
+         }
          break;
 
        case 0:
This page took 0.0377 seconds and 4 git commands to generate.