* aout64.h (BMAGIC): Define.
[deliverable/binutils-gdb.git] / gdb / solib.c
index f2f646d17f00e4fab48852a5eed1b2330f663a49..a328c3110d3d84793c7966a2d16eaeed300c8faa 100644 (file)
@@ -1,5 +1,5 @@
 /* Handle SunOS and SVR4 shared libraries for GDB, the GNU Debugger.
-   Copyright 1990, 1991, 1992 Free Software Foundation, Inc.
+   Copyright 1990, 1991, 1992, 1993, 1994 Free Software Foundation, Inc.
    
 This file is part of GDB.
 
@@ -26,40 +26,72 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #include <link.h>
 #include <sys/param.h>
 #include <fcntl.h>
-#include <a.out.h>
+
+#ifndef SVR4_SHARED_LIBS
+ /* SunOS shared libs need the nlist structure.  */
+#include <a.out.h> 
+#else
+#include "libelf.h"
+#ifndef DT_MIPS_RLD_MAP
+#include "elf/mips.h"
+#endif
+#endif
 
 #include "symtab.h"
 #include "bfd.h"
 #include "symfile.h"
+#include "objfiles.h"
 #include "gdbcore.h"
 #include "command.h"
 #include "target.h"
 #include "frame.h"
 #include "regex.h"
 #include "inferior.h"
+#include "language.h"
 
 #define MAX_PATH_SIZE 256              /* FIXME: Should be dynamic */
 
-/* On SVR4 systems, for the initial implementation, use main() as the
-   "startup mapping complete" breakpoint address.  The models for SunOS
-   and SVR4 dynamic linking debugger support are different in that SunOS
-   hits one breakpoint when all mapping is complete while using the SVR4
+/* On SVR4 systems, for the initial implementation, use some runtime startup
+   symbol as the "startup mapping complete" breakpoint address.  The models
+   for SunOS and SVR4 dynamic linking debugger support are different in that
+   SunOS hits one breakpoint when all mapping is complete while using the SVR4
    debugger support takes two breakpoint hits for each file mapped, and
    there is no way to know when the "last" one is hit.  Both these
    mechanisms should be tied to a "breakpoint service routine" that
    gets automatically executed whenever one of the breakpoints indicating
    a change in mapping is hit.  This is a future enhancement.  (FIXME) */
 
-#define BKPT_AT_MAIN 1
+#define BKPT_AT_SYMBOL 1
+
+#if defined (BKPT_AT_SYMBOL) && defined (SVR4_SHARED_LIBS)
+static char *bkpt_names[] = {
+#ifdef SOLIB_BKPT_NAME
+  SOLIB_BKPT_NAME,             /* Prefer configured name if it exists. */
+#endif
+  "_start",
+  "main",
+  NULL
+};
+#endif
+
+/* Symbols which are used to locate the base of the link map structures. */
+
+#ifndef SVR4_SHARED_LIBS
+static char *debug_base_symbols[] = {
+  "_DYNAMIC",
+  NULL
+};
+#endif
 
 /* local data declarations */
 
 #ifndef SVR4_SHARED_LIBS
 
-#define DEBUG_BASE "_DYNAMIC"
 #define LM_ADDR(so) ((so) -> lm.lm_addr)
 #define LM_NEXT(so) ((so) -> lm.lm_next)
 #define LM_NAME(so) ((so) -> lm.lm_name)
+/* Test for first link map entry; first entry is a shared library. */
+#define IGNORE_FIRST_LINK_MAP_ENTRY(x) (0)
 static struct link_dynamic dynamic_copy;
 static struct link_dynamic_2 ld_2_copy;
 static struct ld_debug debug_copy;
@@ -68,10 +100,11 @@ static CORE_ADDR flag_addr;
 
 #else  /* SVR4_SHARED_LIBS */
 
-#define DEBUG_BASE "_r_debug"
 #define LM_ADDR(so) ((so) -> lm.l_addr)
 #define LM_NEXT(so) ((so) -> lm.l_next)
 #define LM_NAME(so) ((so) -> lm.l_name)
+/* Test for first link map entry; first entry is the exec-file. */
+#define IGNORE_FIRST_LINK_MAP_ENTRY(x) ((x).l_prev == NULL)
 static struct r_debug debug_copy;
 char shadow_contents[BREAKPOINT_MAX];  /* Stash old bkpt addr contents */
 
@@ -85,16 +118,20 @@ struct so_list {
   char so_name[MAX_PATH_SIZE];         /* shared object lib name (FIXME) */
   char symbols_loaded;                 /* flag: symbols read in yet? */
   char from_tty;                       /* flag: print msgs? */
-  bfd *so_bfd;                         /* bfd for so_name */
   struct objfile *objfile;             /* objfile for loaded lib */
   struct section_table *sections;
   struct section_table *sections_end;
+  struct section_table *textsection;
+  bfd *abfd;
 };
 
 static struct so_list *so_list_head;   /* List of known shared objects */
 static CORE_ADDR debug_base;           /* Base of dynamic linker structures */
 static CORE_ADDR breakpoint_addr;      /* Address where end bkpt is set */
 
+extern int
+fdmatch PARAMS ((int, int));           /* In libiberty */
+
 /* Local function prototypes */
 
 static void
@@ -110,7 +147,7 @@ static int
 disable_break PARAMS ((void));
 
 static void
-info_sharedlibrary_command PARAMS ((void));
+info_sharedlibrary_command PARAMS ((char *, int));
 
 static int
 symbol_add_stub PARAMS ((char *));
@@ -124,16 +161,15 @@ first_link_map_member PARAMS ((void));
 static CORE_ADDR
 locate_base PARAMS ((void));
 
-static int
-look_for_base PARAMS ((int, CORE_ADDR));
-
-static CORE_ADDR
-bfd_lookup_symbol PARAMS ((bfd *, char *));
-
 static void
 solib_map_sections PARAMS ((struct so_list *));
 
-#ifndef SVR4_SHARED_LIBS
+#ifdef SVR4_SHARED_LIBS
+
+static CORE_ADDR
+elf_locate_base PARAMS ((void));
+
+#else
 
 static void
 solib_add_common_symbols PARAMS ((struct rtc_symb *, struct objfile *));
@@ -175,9 +211,11 @@ solib_map_sections (so)
   char *scratch_pathname;
   int scratch_chan;
   struct section_table *p;
+  struct cleanup *old_chain;
+  bfd *abfd;
   
   filename = tilde_expand (so -> so_name);
-  make_cleanup (free, filename);
+  old_chain = make_cleanup (free, filename);
   
   scratch_chan = openp (getenv ("PATH"), 1, filename, O_RDONLY, 0,
                        &scratch_pathname);
@@ -189,23 +227,29 @@ solib_map_sections (so)
   if (scratch_chan < 0)
     {
       perror_with_name (filename);
-    }  
+    }
+  /* Leave scratch_pathname allocated.  abfd->name will point to it.  */
 
-  so -> so_bfd = bfd_fdopenr (scratch_pathname, NULL, scratch_chan);
-  if (!so -> so_bfd)
+  abfd = bfd_fdopenr (scratch_pathname, gnutarget, scratch_chan);
+  if (!abfd)
     {
+      close (scratch_chan);
       error ("Could not open `%s' as an executable file: %s",
-            scratch_pathname, bfd_errmsg (bfd_error));
+            scratch_pathname, bfd_errmsg (bfd_get_error ()));
     }
-  if (!bfd_check_format (so -> so_bfd, bfd_object))
+  /* Leave bfd open, core_xfer_memory and "info files" need it.  */
+  so -> abfd = abfd;
+  abfd -> cacheable = true;
+
+  if (!bfd_check_format (abfd, bfd_object))
     {
       error ("\"%s\": not in executable format: %s.",
-            scratch_pathname, bfd_errmsg (bfd_error));
+            scratch_pathname, bfd_errmsg (bfd_get_error ()));
     }
-  if (build_section_table (so -> so_bfd, &so -> sections, &so -> sections_end))
+  if (build_section_table (abfd, &so -> sections, &so -> sections_end))
     {
       error ("Can't find the file sections in `%s': %s", 
-            exec_bfd -> filename, bfd_errmsg (bfd_error));
+            bfd_get_filename (abfd), bfd_errmsg (bfd_get_error ()));
     }
 
   for (p = so -> sections; p < so -> sections_end; p++)
@@ -216,7 +260,14 @@ solib_map_sections (so)
       p -> addr += (CORE_ADDR) LM_ADDR (so);
       p -> endaddr += (CORE_ADDR) LM_ADDR (so);
       so -> lmend = (CORE_ADDR) max (p -> endaddr, so -> lmend);
+      if (STREQ (p -> the_bfd_section -> name, ".text"))
+       {
+         so -> textsection = p;
+       }
     }
+
+  /* Free the file names, close the file now.  */
+  do_cleanups (old_chain);
 }
 
 /* Read all dynamically loaded common symbol definitions from the inferior
@@ -224,6 +275,17 @@ solib_map_sections (so)
 
 #ifndef SVR4_SHARED_LIBS
 
+/* In GDB 4.9 this routine was a real performance hog.  According to
+   some gprof data which mtranle@paris.IntelliCorp.COM (Minh Tran-Le)
+   sent, almost all the time spend in solib_add (up to 20 minutes with
+   35 shared libraries) was spent here, with 5/6 in
+   lookup_minimal_symbol and 1/6 in read_memory.
+
+   To fix this, we moved the call to special_symbol_handling out of the
+   loop in solib_add, so this only gets called once, rather than once
+   for every shared library, and also removed the call to lookup_minimal_symbol
+   in this routine.  */
+
 static void
 solib_add_common_symbols (rtc_symp, objfile)
     struct rtc_symb *rtc_symp;
@@ -258,21 +320,26 @@ solib_add_common_symbols (rtc_symp, objfile)
 
          /* Don't enter the symbol twice if the target is re-run. */
 
-#ifdef NAMES_HAVE_UNDERSCORE
-         if (*name == '_')
+         if (name[0] == bfd_get_symbol_leading_char (objfile->obfd))
            {
              name++;
            }
-#endif
+
+#if 0
+         /* I think this is unnecessary, GDB can probably deal with
+            duplicate minimal symbols, more or less.  And the duplication
+            which used to happen because this was called for each shared
+            library is gone now that we are just called once.  */
          /* FIXME:  Do we really want to exclude symbols which happen
             to match symbols for other locations in the inferior's
             address space, even when they are in different linkage units? */
          if (lookup_minimal_symbol (name, (struct objfile *) NULL) == NULL)
+#endif
            {
              name = obsavestring (name, strlen (name),
                                   &objfile -> symbol_obstack);
              prim_record_minimal_symbol (name, inferior_rtc_nlist.n_value,
-                                         mst_bss);
+                                         mst_bss, objfile);
            }
          free (origname);
        }
@@ -287,149 +354,96 @@ solib_add_common_symbols (rtc_symp, objfile)
 
 #endif /* SVR4_SHARED_LIBS */
 
+
+#ifdef SVR4_SHARED_LIBS
+
 /*
 
 LOCAL FUNCTION
 
-       bfd_lookup_symbol -- lookup the value for a specific symbol
+       elf_locate_base -- locate the base address of dynamic linker structs
+       for SVR4 elf targets.
 
 SYNOPSIS
 
-       CORE_ADDR bfd_lookup_symbol (bfd *abfd, char *symname)
+       CORE_ADDR elf_locate_base (void)
 
 DESCRIPTION
 
-       An expensive way to lookup the value of a single symbol for
-       bfd's that are only temporary anyway.  This is used by the
-       shared library support to find the address of the debugger
-       interface structures in the shared library.
+       For SVR4 elf targets the address of the dynamic linker's runtime
+       structure is contained within the dynamic info section in the
+       executable file.  The dynamic section is also mapped into the
+       inferior address space.  Because the runtime loader fills in the
+       real address before starting the inferior, we have to read in the
+       dynamic info section from the inferior address space.
+       If there are any errors while trying to find the address, we
+       silently return 0, otherwise the found address is returned.
 
-       Note that 0 is specifically allowed as an error return (no
-       such symbol).
-
-       FIXME:  See if there is a less "expensive" way of doing this.
-       Also see if there is already another bfd or gdb function
-       that specifically does this, and if so, use it.
-*/
+ */
 
 static CORE_ADDR
-bfd_lookup_symbol (abfd, symname)
-     bfd *abfd;
-     char *symname;
+elf_locate_base ()
 {
-  unsigned int storage_needed;
-  asymbol *sym;
-  asymbol **symbol_table;
-  unsigned int number_of_symbols;
-  unsigned int i;
-  struct cleanup *back_to;
-  CORE_ADDR symaddr = 0;
-  
-  storage_needed = get_symtab_upper_bound (abfd);
-
-  if (storage_needed > 0)
+  struct elf_internal_shdr *dyninfo_sect;
+  int dyninfo_sect_size;
+  CORE_ADDR dyninfo_addr;
+  char *buf;
+  char *bufend;
+
+  /* Find the start address of the .dynamic section.  */
+  if (exec_bfd == NULL || bfd_get_flavour (exec_bfd) != bfd_target_elf_flavour)
+    return 0;
+  dyninfo_sect = bfd_elf_find_section (exec_bfd, ".dynamic");
+  if (dyninfo_sect == NULL)
+    return 0;
+  dyninfo_addr = dyninfo_sect->sh_addr;
+
+  /* Read in .dynamic section, silently ignore errors.  */
+  dyninfo_sect_size = dyninfo_sect->sh_size;
+  buf = alloca (dyninfo_sect_size);
+  if (target_read_memory (dyninfo_addr, buf, dyninfo_sect_size))
+    return 0;
+
+  /* Find the DT_DEBUG entry in the the .dynamic section.
+     For mips elf we look for DT_MIPS_RLD_MAP, mips elf apparently has
+     no DT_DEBUG entries.  */
+  /* FIXME: In lack of a 64 bit ELF ABI the following code assumes
+     a 32 bit ELF ABI target.  */
+  for (bufend = buf + dyninfo_sect_size;
+       buf < bufend;
+       buf += sizeof (Elf32_External_Dyn))
     {
-      symbol_table = (asymbol **) bfd_xmalloc (storage_needed);
-      back_to = make_cleanup (free, symbol_table);
-      number_of_symbols = bfd_canonicalize_symtab (abfd, symbol_table); 
-  
-      for (i = 0; i < number_of_symbols; i++)
+      Elf32_External_Dyn *x_dynp = (Elf32_External_Dyn *)buf;
+      long dyn_tag;
+      CORE_ADDR dyn_ptr;
+
+      dyn_tag = bfd_h_get_32 (exec_bfd, (bfd_byte *) x_dynp->d_tag);
+      if (dyn_tag == DT_NULL)
+       break;
+      else if (dyn_tag == DT_DEBUG)
        {
-         sym = *symbol_table++;
-         if (strcmp (sym -> name, symname) == 0)
-           {
-             symaddr = sym -> value;
-             break;
-           }
+         dyn_ptr = bfd_h_get_32 (exec_bfd, (bfd_byte *) x_dynp->d_un.d_ptr);
+         return dyn_ptr;
+       }
+      else if (dyn_tag == DT_MIPS_RLD_MAP)
+       {
+         char pbuf[TARGET_PTR_BIT / HOST_CHAR_BIT];
+
+         /* DT_MIPS_RLD_MAP contains a pointer to the address
+            of the dynamic link structure.  */
+         dyn_ptr = bfd_h_get_32 (exec_bfd, (bfd_byte *) x_dynp->d_un.d_ptr);
+         if (target_read_memory (dyn_ptr, pbuf, sizeof (pbuf)))
+           return 0;
+         return extract_unsigned_integer (pbuf, sizeof (pbuf));
        }
-      do_cleanups (back_to);
-    }
-  return (symaddr);
-}
-
-/*
-
-LOCAL FUNCTION
-
-       look_for_base -- examine file for each mapped address segment
-
-SYNOPSYS
-
-       static int look_for_base (int fd, CORE_ADDR baseaddr)
-
-DESCRIPTION
-
-       This function is passed to proc_iterate_over_mappings, which
-       causes it to get called once for each mapped address space, with
-       an open file descriptor for the file mapped to that space, and the
-       base address of that mapped space.
-
-       Our job is to find the symbol DEBUG_BASE in the file that this
-       fd is open on, if it exists, and if so, initialize the dynamic
-       linker structure base address debug_base.
-
-       Note that this is a computationally expensive proposition, since
-       we basically have to open a bfd on every call, so we specifically
-       avoid opening the exec file.
- */
-
-static int
-look_for_base (fd, baseaddr)
-     int fd;
-     CORE_ADDR baseaddr;
-{
-  bfd *interp_bfd;
-  CORE_ADDR address;
-
-  /* If the fd is -1, then there is no file that corresponds to this
-     mapped memory segment, so skip it.  Also, if the fd corresponds
-     to the exec file, skip it as well. */
-
-  if ((fd == -1) || fdmatch (fileno ((FILE *)(exec_bfd -> iostream)), fd))
-    {
-      return (0);
-    }
-
-  /* Try to open whatever random file this fd corresponds to.  Note that
-     we have no way currently to find the filename.  Don't gripe about
-     any problems we might have, just fail. */
-
-  if ((interp_bfd = bfd_fdopenr ("unnamed", NULL, fd)) == NULL)
-    {
-      return (0);
-    }
-  if (!bfd_check_format (interp_bfd, bfd_object))
-    {
-      bfd_close (interp_bfd);
-      return (0);
-    }
-
-  /* Now try to find our DEBUG_BASE symbol in this file, which we at
-     least know to be a valid ELF executable or shared library. */
-
-  if ((address = bfd_lookup_symbol (interp_bfd, DEBUG_BASE)) == 0)
-    {
-      bfd_close (interp_bfd);
-      return (0);
     }
 
-  /* Eureka!  We found the symbol.  But now we may need to relocate it
-     by the base address.  If the symbol's value is less than the base
-     address of the shared library, then it hasn't yet been relocated
-     by the dynamic linker, and we have to do it ourself.  FIXME: Note
-     that we make the assumption that the first segment that corresponds
-     to the shared library has the base address to which the library
-     was relocated. */
-
-  if (address < baseaddr)
-    {
-      address += baseaddr;
-    }
-  debug_base = address;
-  bfd_close (interp_bfd);
-  return (1);
+  /* DT_DEBUG entry not found.  */
+  return 0;
 }
 
+#endif /* SVR4_SHARED_LIBS */
+
 /*
 
 LOCAL FUNCTION
@@ -446,10 +460,9 @@ DESCRIPTION
        inferior executable has been linked dynamically, there is a single
        address somewhere in the inferior's data space which is the key to
        locating all of the dynamic linker's runtime structures.  This
-       address is the value of the symbol defined by the macro DEBUG_BASE.
-       The job of this function is to find and return that address, or to
-       return 0 if there is no such address (the executable is statically
-       linked for example).
+       address is the value of the debug base symbol.  The job of this
+       function is to find and return that address, or to return 0 if there
+       is no such address (the executable is statically linked for example).
 
        For SunOS, the job is almost trivial, since the dynamic linker and
        all of it's structures are statically linked to the executable at
@@ -459,19 +472,12 @@ DESCRIPTION
        have to do is look it up there.  Note that we explicitly do NOT want
        to find the copies in the shared library.
 
-       The SVR4 version is much more complicated because the dynamic linker
-       and it's structures are located in the shared C library, which gets
-       run as the executable's "interpreter" by the kernel.  We have to go
-       to a lot more work to discover the address of DEBUG_BASE.  Because
-       of this complexity, we cache the value we find and return that value
-       on subsequent invocations.  Note there is no copy in the executable
-       symbol tables.
-
-       Note that we can assume nothing about the process state at the time
-       we need to find this address.  We may be stopped on the first instruc-
-       tion of the interpreter (C shared library), the first instruction of
-       the executable itself, or somewhere else entirely (if we attached
-       to the process for example).
+       The SVR4 version is a bit more complicated because the address
+       is contained somewhere in the dynamic info section.  We have to go
+       to a lot more work to discover the address of the debug base symbol.
+       Because of this complexity, we cache the value we find and return that
+       value on subsequent invocations.  Note there is no copy in the
+       executable symbol tables.
 
  */
 
@@ -483,30 +489,33 @@ locate_base ()
 
   struct minimal_symbol *msymbol;
   CORE_ADDR address = 0;
+  char **symbolp;
 
-  /* For SunOS, we want to limit the search for DEBUG_BASE to the executable
-     being debugged, since there is a duplicate named symbol in the shared
-     library.  We don't want the shared library versions. */
+  /* For SunOS, we want to limit the search for the debug base symbol to the
+     executable being debugged, since there is a duplicate named symbol in the
+     shared library.  We don't want the shared library versions. */
 
-  msymbol = lookup_minimal_symbol (DEBUG_BASE, symfile_objfile);
-  if ((msymbol != NULL) && (msymbol -> address != 0))
+  for (symbolp = debug_base_symbols; *symbolp != NULL; symbolp++)
     {
-      address = msymbol -> address;
+      msymbol = lookup_minimal_symbol (*symbolp, symfile_objfile);
+      if ((msymbol != NULL) && (SYMBOL_VALUE_ADDRESS (msymbol) != 0))
+       {
+         address = SYMBOL_VALUE_ADDRESS (msymbol);
+         return (address);
+       }
     }
-  return (address);
+  return (0);
 
 #else  /* SVR4_SHARED_LIBS */
 
   /* Check to see if we have a currently valid address, and if so, avoid
      doing all this work again and just return the cached address.  If
-     we have no cached address, ask the /proc support interface to iterate
-     over the list of mapped address segments, calling look_for_base() for
-     each segment.  When we are done, we will have either found the base
-     address or not. */
+     we have no cached address, try to locate it in the dynamic info
+     section.  */
 
   if (debug_base == 0)
     {
-      proc_iterate_over_mappings (look_for_base);
+      debug_base = elf_locate_base ();
     }
   return (debug_base);
 
@@ -514,6 +523,23 @@ locate_base ()
 
 }
 
+/*
+
+LOCAL FUNCTION
+
+       first_link_map_member -- locate first member in dynamic linker's map
+
+SYNOPSIS
+
+       static struct link_map *first_link_map_member (void)
+
+DESCRIPTION
+
+       Read in a copy of the first member in the inferior's dynamic
+       link map from the inferior's dynamic linker structures, and return
+       a pointer to the copy in our address space.
+*/
+
 static struct link_map *
 first_link_map_member ()
 {
@@ -534,6 +560,9 @@ first_link_map_member ()
 #else  /* SVR4_SHARED_LIBS */
 
   read_memory (debug_base, (char *) &debug_copy, sizeof (struct r_debug));
+  /* FIXME:  Perhaps we should validate the info somehow, perhaps by
+     checking r_version for a known version number, or r_state for
+     RT_CONSISTENT. */
   lm = debug_copy.r_map;
 
 #endif /* !SVR4_SHARED_LIBS */
@@ -580,7 +609,7 @@ find_solib (so_list_ptr)
          /* We have not already read in the dynamic linking structures
             from the inferior, lookup the address of the base structure. */
          debug_base = locate_base ();
-         if (debug_base > 0)
+         if (debug_base != 0)
            {
              /* Read the base structure in and find the address of the first
                 link map list member. */
@@ -615,7 +644,7 @@ find_solib (so_list_ptr)
       /* Get next link map structure from inferior image and build a local
         abbreviated load_map structure */
       new = (struct so_list *) xmalloc (sizeof (struct so_list));
-      (void) memset ((char *) new, 0, sizeof (struct so_list));
+      memset ((char *) new, 0, sizeof (struct so_list));
       new -> lmaddr = lm;
       /* Add the new node as the next node in the list, or as the root
         node if this is the first one. */
@@ -630,14 +659,23 @@ find_solib (so_list_ptr)
       so_list_next = new;
       read_memory ((CORE_ADDR) lm, (char *) &(new -> lm),
                   sizeof (struct link_map));
-      /* For the SVR4 version, there is one entry that has no name
-        (for the inferior executable) since it is not a shared object. */
-      if (LM_NAME (new) != 0)
+      /* For SVR4 versions, the first entry in the link map is for the
+        inferior executable, so we must ignore it.  For some versions of
+        SVR4, it has no name.  For others (Solaris 2.3 for example), it
+        does have a name, so we can no longer use a missing name to
+        decide when to ignore it. */
+      if (!IGNORE_FIRST_LINK_MAP_ENTRY (new -> lm))
        {
-         if (!target_read_string((CORE_ADDR) LM_NAME (new), new -> so_name,
-                     MAX_PATH_SIZE - 1))
-             error ("find_solib: Can't read pathname for load map\n");
-         new -> so_name[MAX_PATH_SIZE - 1] = 0;
+         int errcode;
+         char *buffer;
+         target_read_string ((CORE_ADDR) LM_NAME (new), &buffer,
+                             MAX_PATH_SIZE - 1, &errcode);
+         if (errcode != 0)
+           error ("find_solib: Can't read pathname for load map: %s\n",
+                  safe_strerror (errcode));
+         strncpy (new -> so_name, buffer, MAX_PATH_SIZE - 1);
+         new -> so_name[MAX_PATH_SIZE - 1] = '\0';
+         free (buffer);
          solib_map_sections (new);
        }      
     }
@@ -653,7 +691,8 @@ symbol_add_stub (arg)
   register struct so_list *so = (struct so_list *) arg;        /* catch_errs bogon */
   
   so -> objfile = symbol_file_add (so -> so_name, so -> from_tty,
-                                  (unsigned int) LM_ADDR (so), 0, 0, 0);
+                                  (unsigned int) so -> textsection -> addr,
+                                  0, 0, 0);
   return (1);
 }
 
@@ -679,6 +718,10 @@ solib_add (arg_string, from_tty, target)
      struct target_ops *target;
 {      
   register struct so_list *so = NULL;          /* link map state variable */
+
+  /* Last shared library that we read.  */
+  struct so_list *so_last = NULL;
+
   char *re_err;
   int count;
   int old;
@@ -688,35 +731,10 @@ solib_add (arg_string, from_tty, target)
       error ("Invalid regexp: %s", re_err);
     }
   
-  /* Getting new symbols may change our opinion about what is
-     frameless.  */
-  reinit_frame_cache ();
-  
-  while ((so = find_solib (so)) != NULL)
-    {
-      if (so -> so_name[0] && re_exec (so -> so_name))
-       {
-         if (so -> symbols_loaded)
-           {
-             if (from_tty)
-               {
-                 printf ("Symbols already loaded for %s\n", so -> so_name);
-               }
-           }
-         else
-           {
-             catch_errors (symbol_add_stub, (char *) so,
-                           "Error while reading shared library symbols:\n");
-             
-             special_symbol_handling (so);
-             so -> symbols_loaded = 1;
-             so -> from_tty = from_tty;
-           }
-       }
-    }
-  
-  /* Now add the shared library sections to the section table of the
-     specified target, if any.  */
+  /* Add the shared library sections to the section table of the
+     specified target, if any. We have to do this before reading the
+     symbol files as symbol_file_add calls reinit_frame_cache and
+     creating a new frame might access memory in the shared library.  */
   if (target)
     {
       /* Count how many new section_table entries there are.  */
@@ -737,14 +755,14 @@ solib_add (arg_string, from_tty, target)
            {
              old = target -> to_sections_end - target -> to_sections;
              target -> to_sections = (struct section_table *)
-               realloc ((char *)target -> to_sections,
+               xrealloc ((char *)target -> to_sections,
                         (sizeof (struct section_table)) * (count + old));
            }
          else
            {
              old = 0;
              target -> to_sections = (struct section_table *)
-               malloc ((sizeof (struct section_table)) * count);
+               xmalloc ((sizeof (struct section_table)) * count);
            }
          target -> to_sections_end = target -> to_sections + (count + old);
          
@@ -754,13 +772,51 @@ solib_add (arg_string, from_tty, target)
              if (so -> so_name[0])
                {
                  count = so -> sections_end - so -> sections;
-                 bcopy (so -> sections, (char *)(target -> to_sections + old), 
-                        (sizeof (struct section_table)) * count);
+                 memcpy ((char *) (target -> to_sections + old),
+                         so -> sections, 
+                         (sizeof (struct section_table)) * count);
                  old += count;
                }
            }
        }
     }
+  
+  /* Now add the symbol files.  */
+  while ((so = find_solib (so)) != NULL)
+    {
+      if (so -> so_name[0] && re_exec (so -> so_name))
+       {
+         so -> from_tty = from_tty;
+         if (so -> symbols_loaded)
+           {
+             if (from_tty)
+               {
+                 printf_unfiltered ("Symbols already loaded for %s\n", so -> so_name);
+               }
+           }
+         else if (catch_errors
+                  (symbol_add_stub, (char *) so,
+                   "Error while reading shared library symbols:\n",
+                   RETURN_MASK_ALL))
+           {
+             so_last = so;
+             so -> symbols_loaded = 1;
+           }
+       }
+    }
+
+  /* Calling this once at the end means that we put all the minimal
+     symbols for commons into the objfile for the last shared library.
+     Since they are in common, this should not be a problem.  If we
+     delete the objfile with the minimal symbols, we can put all the
+     symbols into a new objfile (and will on the next call to solib_add).
+
+     An alternate approach would be to create an objfile just for
+     common minsyms, thus not needing any objfile argument to
+     solib_add_common_symbols.  */
+
+  if (so_last)
+    special_symbol_handling (so_last);
 }
 
 /*
@@ -780,14 +836,16 @@ DESCRIPTION
 */
 
 static void
-info_sharedlibrary_command ()
+info_sharedlibrary_command (ignore, from_tty)
+     char *ignore;
+     int from_tty;
 {
   register struct so_list *so = NULL;          /* link map state variable */
   int header_done = 0;
   
   if (exec_bfd == NULL)
     {
-      printf ("No exec file.\n");
+      printf_unfiltered ("No exec file.\n");
       return;
     }
   while ((so = find_solib (so)) != NULL)
@@ -796,19 +854,25 @@ info_sharedlibrary_command ()
        {
          if (!header_done)
            {
-             printf("%-12s%-12s%-12s%s\n", "From", "To", "Syms Read",
+             printf_unfiltered("%-12s%-12s%-12s%s\n", "From", "To", "Syms Read",
                     "Shared Object Library");
              header_done++;
            }
-         printf ("%-12s", local_hex_string_custom ((int) LM_ADDR (so), "08"));
-         printf ("%-12s", local_hex_string_custom (so -> lmend, "08"));
-         printf ("%-12s", so -> symbols_loaded ? "Yes" : "No");
-         printf ("%s\n",  so -> so_name);
+         /* FIXME-32x64: need print_address_numeric with field width or
+            some such.  */
+         printf_unfiltered ("%-12s",
+                 local_hex_string_custom ((unsigned long) LM_ADDR (so),
+                                          "08l"));
+         printf_unfiltered ("%-12s",
+                 local_hex_string_custom ((unsigned long) so -> lmend,
+                                          "08l"));
+         printf_unfiltered ("%-12s", so -> symbols_loaded ? "Yes" : "No");
+         printf_unfiltered ("%s\n",  so -> so_name);
        }
     }
   if (so_list_head == NULL)
     {
-      printf ("No shared libraries loaded at this time.\n");   
+      printf_unfiltered ("No shared libraries loaded at this time.\n");        
     }
 }
 
@@ -862,19 +926,27 @@ void
 clear_solib()
 {
   struct so_list *next;
+  char *bfd_filename;
   
   while (so_list_head)
     {
       if (so_list_head -> sections)
        {
-         free (so_list_head -> sections);
+         free ((PTR)so_list_head -> sections);
        }
-      if (so_list_head -> so_bfd)
+      if (so_list_head -> abfd)
        {
-         bfd_close (so_list_head -> so_bfd);
+         bfd_filename = bfd_get_filename (so_list_head -> abfd);
+         bfd_close (so_list_head -> abfd);
        }
+      else
+       /* This happens for the executable on SVR4.  */
+       bfd_filename = NULL;
+      
       next = so_list_head -> next;
-      free(so_list_head);
+      if (bfd_filename)
+       free ((PTR)bfd_filename);
+      free ((PTR)so_list_head);
       so_list_head = next;
     }
   debug_base = 0;
@@ -991,13 +1063,13 @@ DESCRIPTION
 static int
 enable_break ()
 {
-
-  int j;
+  int success = 0;
 
 #ifndef SVR4_SHARED_LIBS
 
+  int j;
   int in_debugger;
-  
+
   /* Get link_dynamic structure */
 
   j = target_read_memory (debug_base, (char *) &dynamic_copy,
@@ -1020,31 +1092,37 @@ enable_break ()
   /* Write a value of 1 to this member.  */
 
   in_debugger = 1;
-
   write_memory (flag_addr, (char *) &in_debugger, sizeof (in_debugger));
+  success = 1;
 
 #else  /* SVR4_SHARED_LIBS */
 
-#ifdef BKPT_AT_MAIN
+#ifdef BKPT_AT_SYMBOL
 
   struct minimal_symbol *msymbol;
+  char **bkpt_namep;
+  CORE_ADDR bkpt_addr;
 
-  msymbol = lookup_minimal_symbol ("main", symfile_objfile);
-  if ((msymbol != NULL) && (msymbol -> address != 0))
-    {
-      breakpoint_addr = msymbol -> address;
-    }
-  else
-    {
-      return (0);
-    }
+  /* Scan through the list of symbols, trying to look up the symbol and
+     set a breakpoint there.  Terminate loop when we/if we succeed. */
 
-  if (target_insert_breakpoint (breakpoint_addr, shadow_contents) != 0)
+  breakpoint_addr = 0;
+  for (bkpt_namep = bkpt_names; *bkpt_namep != NULL; bkpt_namep++)
     {
-      return (0);
+      msymbol = lookup_minimal_symbol (*bkpt_namep, symfile_objfile);
+      if ((msymbol != NULL) && (SYMBOL_VALUE_ADDRESS (msymbol) != 0))
+       {
+         bkpt_addr = SYMBOL_VALUE_ADDRESS (msymbol);
+         if (target_insert_breakpoint (bkpt_addr, shadow_contents) == 0)
+           {
+             breakpoint_addr = bkpt_addr;
+             success = 1;
+             break;
+           }
+       }
     }
 
-#else  /* !BKPT_AT_MAIN */
+#else  /* !BKPT_AT_SYMBOL */
 
   struct symtab_and_line sal;
 
@@ -1058,12 +1136,13 @@ enable_break ()
      deal with hitting these breakpoints.  (FIXME). */
 
   warning ("'%s': line %d: missing SVR4 support code", __FILE__, __LINE__);
+  success = 1;
 
-#endif /* BKPT_AT_MAIN */
+#endif /* BKPT_AT_SYMBOL */
 
 #endif /* !SVR4_SHARED_LIBS */
 
-  return (1);
+  return (success);
 }
   
 /*
@@ -1083,6 +1162,21 @@ DESCRIPTION
        point, this function gets called via expansion of the macro
        SOLIB_CREATE_INFERIOR_HOOK.
 
+       For SunOS executables, this first instruction is typically the
+       one at "_start", or a similar text label, regardless of whether
+       the executable is statically or dynamically linked.  The runtime
+       startup code takes care of dynamically linking in any shared
+       libraries, once gdb allows the inferior to continue.
+
+       For SVR4 executables, this first instruction is either the first
+       instruction in the dynamic linker (for dynamically linked
+       executables) or the instruction at "start" for statically linked
+       executables.  For dynamically linked executables, the system
+       first exec's /lib/libc.so.N, which contains the dynamic linker,
+       and starts it running.  The dynamic linker maps in any needed
+       shared libraries, maps in the actual user executable, and then
+       jumps to "start" in the user executable.
+
        For both SunOS shared libraries, and SVR4 shared libraries, we
        can arrange to cooperate with the dynamic linker to discover the
        names of shared libraries that are dynamically linked, and the
@@ -1106,17 +1200,17 @@ FIXME
 void 
 solib_create_inferior_hook()
 {
-  CORE_ADDR debug_addr;
-  int in_debugger;
-  CORE_ADDR in_debugger_addr;
-  CORE_ADDR breakpoint_addr;
-  int i, j;
-  
+  /* If we are using the BKPT_AT_SYMBOL code, then we don't need the base
+     yet.  In fact, in the case of a SunOS4 executable being run on
+     Solaris, we can't get it yet.  find_solib will get it when it needs
+     it.  */
+#if !(defined (SVR4_SHARED_LIBS) && defined (BKPT_AT_SYMBOL))
   if ((debug_base = locate_base ()) == 0)
     {
       /* Can't find the symbol or the executable is statically linked. */
       return;
     }
+#endif
 
   if (!enable_break ())
     {
@@ -1131,13 +1225,13 @@ solib_create_inferior_hook()
 
   clear_proceed_status ();
   stop_soon_quietly = 1;
-  stop_signal = 0;
+  stop_signal = TARGET_SIGNAL_0;
   do
     {
-      target_resume (0, stop_signal);
+      target_resume (-1, 0, stop_signal);
       wait_for_inferior ();
     }
-  while (stop_signal != SIGTRAP);
+  while (stop_signal != TARGET_SIGNAL_TRAP);
   stop_soon_quietly = 0;
   
   /* We are now either at the "mapping complete" breakpoint (or somewhere
@@ -1187,11 +1281,34 @@ special_symbol_handling (so)
 struct so_list *so;
 {
 #ifndef SVR4_SHARED_LIBS
+  int j;
+
+  if (debug_addr == 0)
+    {
+      /* Get link_dynamic structure */
+
+      j = target_read_memory (debug_base, (char *) &dynamic_copy,
+                             sizeof (dynamic_copy));
+      if (j)
+       {
+         /* unreadable */
+         return;
+       }
+
+      /* Calc address of debugger interface structure */
+      /* FIXME, this needs work for cross-debugging of core files
+        (byteorder, size, alignment, etc).  */
+
+      debug_addr = (CORE_ADDR) dynamic_copy.ldd;
+    }
 
   /* Read the debugger structure from the inferior, just to make sure
      we have a current copy. */
 
-  read_memory (debug_addr, (char *) &debug_copy, sizeof (debug_copy));
+  j = target_read_memory (debug_addr, (char *) &debug_copy,
+                         sizeof (debug_copy));
+  if (j)
+    return;            /* unreadable */
 
   /* Get common symbol definitions for the loaded object. */
 
This page took 0.037755 seconds and 4 git commands to generate.