From: Pedro Alves <palves@redhat.com>
Date: Tue, 28 Jul 2009 15:22:20 +0000 (+0000)
Subject: Revert previous commit.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d551e57bf793cc95d833626ff467e84f0bfab146;p=binutils-gdb.git

Revert previous commit.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 857456fc0eb..48743fd4802 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,10 +1,3 @@
-2009-07-28  Pedro Alves  <pedro@codesourcery.com>
-
-	* bfd-target.c (target_bfd_xclose): Only close the bfd if the
-	section tarble is not empty.
-	(target_bfd_reopen): If the section table ends up empty, close the
-	bfd here.
-
 2009-07-28  Aleksandar Ristovski  <aristovski@qnx.com>
 
 	* nto-tdep.c (nto_thread_state_str): New array.
diff --git a/gdb/bfd-target.c b/gdb/bfd-target.c
index 2b7894e28e1..ffaa4ffe440 100644
--- a/gdb/bfd-target.c
+++ b/gdb/bfd-target.c
@@ -54,10 +54,7 @@ static void
 target_bfd_xclose (struct target_ops *t, int quitting)
 {
   struct target_section_table *table = t->to_data;
-
-  /* If the target sections table is empty, the bfd had already been
-     closed.  */
-  if (table->sections != table->sections_end)
+  if (table->sections)
     bfd_close (table->sections->bfd);
   xfree (table->sections);
   xfree (table);
@@ -73,12 +70,6 @@ target_bfd_reopen (struct bfd *bfd)
   table = XZALLOC (struct target_section_table);
   build_section_table (bfd, &table->sections, &table->sections_end);
 
-  /* No use keeping the bfd open if there are no target sections we
-     care about.  This way, we avoid keeping the bfd pointer stored
-     somewhere so that target_bfd_xclose could use it.  */
-  if (table->sections == table->sections_end)
-    bfd_close (bfd);
-
   t = XZALLOC (struct target_ops);
   t->to_shortname = "bfd";
   t->to_longname = _("BFD backed target");