2007-08-10 Michael Snyder <msnyder@access-company.com>

	* solib-svr4.c (enable_break): Don't free tmp_pathname until
	after closing bfd.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index b3894a4..2057dcc 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,8 @@
 2007-08-10  Michael Snyder  <msnyder@access-company.com>
 
+       * solib-svr4.c (enable_break): Don't free tmp_pathname until
+       after closing bfd.
+
        * completer.c: Comment/whitespace cleanup.
 
 2007-08-10  Joel Brobecker  <brobecker@adacore.com>

diff --git a/gdb/solib-svr4.c b/gdb/solib-svr4.c
index 9b72a57..6391827 100644 (file)
--- a/gdb/solib-svr4.c
+++ b/gdb/solib-svr4.c
@@ -1008,7 +1008,6 @@ enable_break (void)
       tmp_fd = solib_open (buf, &tmp_pathname);
       if (tmp_fd >= 0)
        tmp_bfd = bfd_fopen (tmp_pathname, gnutarget, FOPEN_RB, tmp_fd);
-      xfree (tmp_pathname);
 
       if (tmp_bfd == NULL)
        goto bkpt_at_symbol;
@@ -1102,6 +1101,7 @@ enable_break (void)
       /* For whatever reason we couldn't set a breakpoint in the dynamic
          linker.  Warn and drop into the old code.  */
     bkpt_at_symbol:
+      xfree (tmp_pathname);
       warning (_("Unable to find dynamic linker breakpoint function.\n"
                "GDB will be unable to debug shared library initializers\n"
                "and track explicitly loaded dynamic code."));