From ade72a3453670d518ba14fe47bc7a64df81ce766 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Fri, 4 May 2018 10:17:52 -0600 Subject: Use gdb_bfd_ref_ptr in target_bfd I noticed that target_bfd was using manual reference counting for the BFD it held. This patch changes it to use gdb_bfd_ref_ptr instead. Tested by the buildbot. ChangeLog 2018-05-04 Tom Tromey * bfd-target.c (target_bfd::m_bfd): Now a gdb_bfd_ref_ptr. (target_bfd::target_bfd, target_bfd::~target_bfd): Update. --- gdb/bfd-target.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'gdb/bfd-target.c') diff --git a/gdb/bfd-target.c b/gdb/bfd-target.c index ba194d7b04b..6138d450d96 100644 --- a/gdb/bfd-target.c +++ b/gdb/bfd-target.c @@ -53,7 +53,7 @@ public: private: /* The BFD we're wrapping. */ - struct bfd *m_bfd; + gdb_bfd_ref_ptr m_bfd; /* The section table build from the ALLOC sections in BFD. Note that we can't rely on extracting the BFD from a random section in @@ -90,10 +90,9 @@ target_bfd::get_section_table () } target_bfd::target_bfd (struct bfd *abfd) + : m_bfd (gdb_bfd_ref_ptr::new_reference (abfd)) { this->to_stratum = file_stratum; - m_bfd = abfd; - gdb_bfd_ref (abfd); m_table.sections = NULL; m_table.sections_end = NULL; build_section_table (abfd, &m_table.sections, &m_table.sections_end); @@ -101,7 +100,6 @@ target_bfd::target_bfd (struct bfd *abfd) target_bfd::~target_bfd () { - gdb_bfd_unref (m_bfd); xfree (m_table.sections); } -- cgit v1.2.1