aboutsummaryrefslogtreecommitdiff
path: root/gdb/dtrace-probe.c
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2022-08-02 09:55:32 -0600
committerTom Tromey <tom@tromey.com>2022-08-03 13:26:58 -0600
commit98badbfdc222d1d7f346046f23a64522b88d22a0 (patch)
treea57a95b75f64a0e318e0b1bd31dd6b246e4797d8 /gdb/dtrace-probe.c
parent4d44946794e68cf79cfba467fa414a958dba2185 (diff)
downloadgdb-98badbfdc222d1d7f346046f23a64522b88d22a0.zip
gdb-98badbfdc222d1d7f346046f23a64522b88d22a0.tar.gz
gdb-98badbfdc222d1d7f346046f23a64522b88d22a0.tar.bz2
Use gdb_bfd_ref_ptr in objfile
This changes struct objfile to use a gdb_bfd_ref_ptr. In addition to removing some manual memory management, this fixes a use-after-free that was introduced by the registry rewrite series. The issue there was that, in some cases, registry shutdown could refer to memory that had already been freed. This help fix the bug by delaying the destruction of the BFD reference (and thus the per-bfd object) until after the registry has been shut down.
Diffstat (limited to 'gdb/dtrace-probe.c')
-rw-r--r--gdb/dtrace-probe.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/dtrace-probe.c b/gdb/dtrace-probe.c
index 6f01edf..b1f6ce5 100644
--- a/gdb/dtrace-probe.c
+++ b/gdb/dtrace-probe.c
@@ -830,7 +830,7 @@ dtrace_static_probe_ops::get_probes
(std::vector<std::unique_ptr<probe>> *probesp,
struct objfile *objfile) const
{
- bfd *abfd = objfile->obfd;
+ bfd *abfd = objfile->obfd.get ();
asection *sect = NULL;
/* Do nothing in case this is a .debug file, instead of the objfile