Add a testcase for PR ld/13839
[deliverable/binutils-gdb.git] / gdb / elfread.c
index 2d589a42905b7a9702792ac43c1af675a4ebed84..9f8a7e8bf0f1b81d973cce5458c3f2faaf6d5548 100644 (file)
@@ -1,8 +1,6 @@
 /* Read ELF (Executable and Linking Format) object files for GDB.
 
-   Copyright (C) 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
-   2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011
-   Free Software Foundation, Inc.
+   Copyright (C) 1991-2012 Free Software Foundation, Inc.
 
    Written by Fred Fish at Cygnus Support.
 
@@ -43,6 +41,7 @@
 #include "infcall.h"
 #include "gdbthread.h"
 #include "regcache.h"
+#include "bcache.h"
 
 extern void _initialize_elfread (void);
 
@@ -143,7 +142,7 @@ elf_symfile_segments (bfd *abfd)
         binaries are not relocatable.  */
       if (bfd_get_section_size (sect) > 0 && j == num_segments
          && (bfd_get_section_flags (abfd, sect) & SEC_LOAD) != 0)
-       warning (_("Loadable segment \"%s\" outside of ELF segments"),
+       warning (_("Loadable section \"%s\" outside of ELF segments"),
                 bfd_section_name (abfd, sect));
     }
 
@@ -206,18 +205,7 @@ record_minimal_symbol (const char *name, int name_len, int copy_name,
                                          bfd_section, objfile);
 }
 
-/*
-
-   LOCAL FUNCTION
-
-   elf_symtab_read -- read the symbol table of an ELF file
-
-   SYNOPSIS
-
-   void elf_symtab_read (struct objfile *objfile, int type,
-                        long number_of_symbols, asymbol **symbol_table)
-
-   DESCRIPTION
+/* Read the symbol table of an ELF file.
 
    Given an objfile, a symbol table, and a flag indicating whether the
    symbol table contains regular, dynamic, or synthetic symbols, add all
@@ -227,9 +215,7 @@ record_minimal_symbol (const char *name, int name_len, int copy_name,
    defined in the ELF symbol table, which can be used to locate
    the beginnings of sections from each ".o" file that was linked to
    form the executable objfile.  We gather any such info and record it
-   in data structures hung off the objfile's private data.
-
- */
+   in data structures hung off the objfile's private data.  */
 
 #define ST_REGULAR 0
 #define ST_DYNAMIC 1
@@ -253,11 +239,10 @@ elf_symtab_read (struct objfile *objfile, int type,
      seen any section info for it yet.  */
   asymbol *filesym = 0;
   /* Name of filesym.  This is either a constant string or is saved on
-     the objfile's obstack.  */
-  char *filesymname = "";
+     the objfile's filename cache.  */
+  const char *filesymname = "";
   struct dbx_symfile_info *dbx = objfile->deprecated_sym_stab_info;
   int stripped = (bfd_get_symcount (objfile->obfd) == 0);
-  struct cleanup *back_to = make_cleanup (null_cleanup, NULL);
 
   for (i = 0; i < number_of_symbols; i++)
     {
@@ -286,7 +271,7 @@ elf_symtab_read (struct objfile *objfile, int type,
        {
          struct minimal_symbol *msym;
          bfd *abfd = objfile->obfd;
-         asection *sect; 
+         asection *sect;
 
          /* Symbol is a reference to a function defined in
             a shared library.
@@ -317,6 +302,23 @@ elf_symtab_read (struct objfile *objfile, int type,
          if (!sect)
            continue;
 
+         /* On ia64-hpux, we have discovered that the system linker
+            adds undefined symbols with nonzero addresses that cannot
+            be right (their address points inside the code of another
+            function in the .text section).  This creates problems
+            when trying to determine which symbol corresponds to
+            a given address.
+
+            We try to detect those buggy symbols by checking which
+            section we think they correspond to.  Normally, PLT symbols
+            are stored inside their own section, and the typical name
+            for that section is ".plt".  So, if there is a ".plt"
+            section, and yet the section name of our symbol does not
+            start with ".plt", we ignore that symbol.  */
+         if (strncmp (sect->name, ".plt", 4) != 0
+             && bfd_get_section_by_name (abfd, ".plt") != NULL)
+           continue;
+
          symaddr += ANOFFSET (objfile->section_offsets, sect->index);
 
          msym = record_minimal_symbol
@@ -343,9 +345,8 @@ elf_symtab_read (struct objfile *objfile, int type,
              sectinfo = NULL;
            }
          filesym = sym;
-         filesymname =
-           obsavestring ((char *) filesym->name, strlen (filesym->name),
-                         &objfile->objfile_obstack);
+         filesymname = bcache (filesym->name, strlen (filesym->name) + 1,
+                               objfile->filename_cache);
        }
       else if (sym->flags & BSF_SECTION_SYM)
        continue;
@@ -376,7 +377,7 @@ elf_symtab_read (struct objfile *objfile, int type,
 
                 NOTE: uweigand-20071112: Synthetic symbols do not
                 have an ELF-private part, so do not touch those.  */
-             unsigned int shndx = type == ST_SYNTHETIC ? 0 : 
+             unsigned int shndx = type == ST_SYNTHETIC ? 0 :
                ((elf_symbol_type *) sym)->internal_elf_sym.st_shndx;
 
              switch (shndx)
@@ -482,11 +483,10 @@ elf_symtab_read (struct objfile *objfile, int type,
                             already includes one element, so we
                             need to allocate max_index aadditional
                             elements.  */
-                         size = (sizeof (struct stab_section_info) 
+                         size = (sizeof (struct stab_section_info)
                                  + (sizeof (CORE_ADDR) * max_index));
                          sectinfo = (struct stab_section_info *)
                            xmalloc (size);
-                         make_cleanup (xfree, sectinfo);
                          memset (sectinfo, 0, size);
                          sectinfo->num_sections = max_index;
                          if (filesym == NULL)
@@ -537,7 +537,7 @@ elf_symtab_read (struct objfile *objfile, int type,
          else
            {
              /* FIXME:  Solaris2 shared libraries include lots of
-                odd "absolute" and "undefined" symbols, that play 
+                odd "absolute" and "undefined" symbols, that play
                 hob with actions like finding what function the PC
                 is in.  Ignore them if they aren't text, data, or bss.  */
              /* ms_type = mst_unknown; */
@@ -556,7 +556,7 @@ elf_symtab_read (struct objfile *objfile, int type,
                 ELF-private part.  However, in some cases (e.g. synthetic
                 'dot' symbols on ppc64) the udata.p entry is set to point back
                 to the original ELF symbol it was derived from.  Get the size
-                from that symbol.  */ 
+                from that symbol.  */
              if (type != ST_SYNTHETIC)
                elf_sym = (elf_symbol_type *) sym;
              else
@@ -564,7 +564,7 @@ elf_symtab_read (struct objfile *objfile, int type,
 
              if (elf_sym)
                MSYMBOL_SIZE(msym) = elf_sym->internal_elf_sym.st_size;
-         
+
              msym->filename = filesymname;
              gdbarch_elf_make_msymbol_special (gdbarch, sym, msym);
            }
@@ -595,7 +595,6 @@ elf_symtab_read (struct objfile *objfile, int type,
            }
        }
     }
-  do_cleanups (back_to);
 }
 
 /* Build minimal symbols named `function@got.plt' (see SYMBOL_GOT_PLT_SUFFIX)
@@ -670,14 +669,14 @@ elf_rel_plt_read (struct objfile *objfile, asymbol **dyn_symbol_table)
         OBJFILE the symbol is undefined and the objfile having NAME defined
         may not yet have been loaded.  */
 
-      if (string_buffer_size < name_len + got_suffix_len)
+      if (string_buffer_size < name_len + got_suffix_len + 1)
        {
          string_buffer_size = 2 * (name_len + got_suffix_len);
          string_buffer = xrealloc (string_buffer, string_buffer_size);
        }
       memcpy (string_buffer, name, name_len);
       memcpy (&string_buffer[name_len], SYMBOL_GOT_PLT_SUFFIX,
-             got_suffix_len);
+             got_suffix_len + 1);
 
       msym = record_minimal_symbol (string_buffer, name_len + got_suffix_len,
                                     1, address, mst_slot_got_plt, got_plt,
@@ -906,7 +905,7 @@ elf_gnu_ifunc_resolve_name (const char *name, CORE_ADDR *addr_p)
 {
   if (elf_gnu_ifunc_resolve_by_cache (name, addr_p))
     return 1;
-  
+
   if (elf_gnu_ifunc_resolve_by_got (name, addr_p))
     return 1;
 
@@ -921,7 +920,7 @@ elf_gnu_ifunc_resolve_name (const char *name, CORE_ADDR *addr_p)
 static CORE_ADDR
 elf_gnu_ifunc_resolve_addr (struct gdbarch *gdbarch, CORE_ADDR pc)
 {
-  char *name_at_pc;
+  const char *name_at_pc;
   CORE_ADDR start_at_pc, address;
   struct type *func_func_type = builtin_type (gdbarch)->builtin_func_func;
   struct value *function, *address_val;
@@ -996,6 +995,9 @@ elf_gnu_ifunc_resolver_stop (struct breakpoint *b)
                                           prev_frame_id,
                                           bp_gnu_ifunc_resolver_return);
 
+      /* set_momentary_breakpoint invalidates PREV_FRAME.  */
+      prev_frame = NULL;
+
       /* Add new b_return to the ring list b->related_breakpoint.  */
       gdb_assert (b_return->related_breakpoint == b_return);
       b_return->related_breakpoint = b->related_breakpoint;
@@ -1048,7 +1050,7 @@ elf_gnu_ifunc_resolver_return_stop (struct breakpoint *b)
     }
   gdb_assert (b->type == bp_gnu_ifunc_resolver);
 
-  gdb_assert (current_program_space == b->pspace);
+  gdb_assert (current_program_space == b->pspace || b->pspace == NULL);
   elf_gnu_ifunc_record_cache (b->addr_string, resolved_pc);
 
   sal = find_pc_line (resolved_pc, 0);
@@ -1204,7 +1206,7 @@ find_separate_debug_file_by_buildid (struct objfile *objfile)
 }
 
 /* Scan and build partial symbols for a symbol file.
-   We have been initialized by a call to elf_symfile_init, which 
+   We have been initialized by a call to elf_symfile_init, which
    currently does nothing.
 
    SECTION_OFFSETS is a set of offsets to apply to relocate the symbols
@@ -1234,7 +1236,7 @@ find_separate_debug_file_by_buildid (struct objfile *objfile)
 static void
 elf_symfile_read (struct objfile *objfile, int symfile_flags)
 {
-  bfd *abfd = objfile->obfd;
+  bfd *synth_abfd, *abfd = objfile->obfd;
   struct elfinfo ei;
   struct cleanup *back_to;
   long symcount = 0, dynsymcount = 0, synthcount, storage_needed;
@@ -1305,9 +1307,26 @@ elf_symfile_read (struct objfile *objfile, int symfile_flags)
       elf_rel_plt_read (objfile, dyn_symbol_table);
     }
 
+  /* Contrary to binutils --strip-debug/--only-keep-debug the strip command from
+     elfutils (eu-strip) moves even the .symtab section into the .debug file.
+
+     bfd_get_synthetic_symtab on ppc64 for each function descriptor ELF symbol
+     'name' creates a new BSF_SYNTHETIC ELF symbol '.name' with its code
+     address.  But with eu-strip files bfd_get_synthetic_symtab would fail to
+     read the code address from .opd while it reads the .symtab section from
+     a separate debug info file as the .opd section is SHT_NOBITS there.
+
+     With SYNTH_ABFD the .opd section will be read from the original
+     backlinked binary where it is valid.  */
+
+  if (objfile->separate_debug_objfile_backlink)
+    synth_abfd = objfile->separate_debug_objfile_backlink->obfd;
+  else
+    synth_abfd = abfd;
+
   /* Add synthetic symbols - for instance, names for any PLT entries.  */
 
-  synthcount = bfd_get_synthetic_symtab (abfd, symcount, symbol_table,
+  synthcount = bfd_get_synthetic_symtab (synth_abfd, symcount, symbol_table,
                                         dynsymcount, dyn_symbol_table,
                                         &synthsyms);
   if (synthcount > 0)
@@ -1377,9 +1396,14 @@ elf_symfile_read (struct objfile *objfile, int symfile_flags)
                                bfd_section_size (abfd, str_sect));
     }
 
-  if (dwarf2_has_info (objfile))
+  if (dwarf2_has_info (objfile, NULL))
     {
-      if (dwarf2_initialize_objfile (objfile))
+      /* elf_sym_fns_gdb_index cannot handle simultaneous non-DWARF debug
+        information present in OBJFILE.  If there is such debug info present
+        never use .gdb_index.  */
+
+      if (!objfile_has_partial_symbols (objfile)
+         && dwarf2_initialize_objfile (objfile))
        objfile->sf = &elf_sym_fns_gdb_index;
       else
        {
@@ -1418,7 +1442,7 @@ elf_symfile_read (struct objfile *objfile, int symfile_flags)
 static void
 read_psyms (struct objfile *objfile)
 {
-  if (dwarf2_has_info (objfile))
+  if (dwarf2_has_info (objfile, NULL))
     dwarf2_build_psymtabs (objfile);
 }
 
@@ -1543,7 +1567,7 @@ elfstab_offset_sections (struct objfile *objfile, struct partial_symtab *pst)
       /* Found it!  Allocate a new psymtab struct, and fill it in.  */
       maybe->found++;
       pst->section_offsets = (struct section_offsets *)
-       obstack_alloc (&objfile->objfile_obstack, 
+       obstack_alloc (&objfile->objfile_obstack,
                       SIZEOF_N_SECTION_OFFSETS (objfile->num_sections));
       for (i = 0; i < maybe->num_sections; i++)
        (pst->section_offsets)->offsets[i] = maybe->sections[i];
This page took 0.028129 seconds and 4 git commands to generate.