FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

gdb: make so_list::lm_info a unique_ptr

Browse source 
Make it a unique_ptr, so it gets automatically deleted when the so_list
is deleted.

Change-Id: Ib62d60ae2a80656239860b80e4359121c93da13d
Approved-By: Pedro Alves <pedro@palves.net>
Reviewed-By: Reviewed-By: Lancelot Six <lancelot.six@amd.com>
This commit is contained in:
Simon Marchi 2023-10-10 14:47:13 -04:00
parent 1c0dfccba7
commit 7ad0a42e88
10 changed files with 40 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
gdb/nto-tdep.c
View file

@ -247,7 +247,7 @@ nto_parse_redirection (char *pargv[], const char **pin, const char **pout,
static CORE_ADDR
lm_addr (const so_list &so)
{
auto *li = gdb::checked_static_cast<const lm_info_svr4 *> (so.lm_info);
auto *li = gdb::checked_static_cast<const lm_info_svr4 *> (so.lm_info.get ());
return li->l_addr;
}

4
gdb/solib-aix.c
View file

@ -316,7 +316,7 @@ solib_aix_relocate_section_addresses (so_list &so, target_section *sec)
struct bfd_section *bfd_sect = sec->the_bfd_section;
bfd *abfd = bfd_sect->owner;
const char *section_name = bfd_section_name (bfd_sect);
auto *info = gdb::checked_static_cast<lm_info_aix *> (so.lm_info);
auto *info = gdb::checked_static_cast<lm_info_aix *> (so.lm_info.get ());
if (strcmp (section_name, ".text") == 0)
{
@ -487,7 +487,7 @@ solib_aix_current_sos (void)
new_solib->so_name[SO_NAME_MAX_PATH_SIZE - 1] = '\0';
memcpy (new_solib->so_name, new_solib->so_original_name,
SO_NAME_MAX_PATH_SIZE);
new_solib->lm_info = new lm_info_aix (info);
new_solib->lm_info = gdb::make_unique<lm_info_aix> (info);
/* Add it to the list. */
if (!start)

7
gdb/solib-darwin.c
View file

@ -275,14 +275,15 @@ darwin_current_sos (void)
/* Create and fill the new so_list element. */
so_list *newobj = new so_list;
lm_info_darwin *li = new lm_info_darwin;
newobj->lm_info = li;
auto li = gdb::make_unique<lm_info_darwin> ();
strncpy (newobj->so_name, file_path.get (), SO_NAME_MAX_PATH_SIZE - 1);
newobj->so_name[SO_NAME_MAX_PATH_SIZE - 1] = '\0';
strcpy (newobj->so_original_name, newobj->so_name);
li->lm_addr = load_addr;
newobj->lm_info = std::move (li);
if (head == NULL)
head = newobj;
else
@ -614,7 +615,7 @@ darwin_clear_solib (program_space *pspace)
static void
darwin_relocate_section_addresses (so_list &so, target_section *sec)
{
auto *li = gdb::checked_static_cast<lm_info_darwin *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_darwin *> (so.lm_info.get ());
sec->addr += li->lm_addr;
sec->endaddr += li->lm_addr;

7
gdb/solib-dsbt.c
View file

@ -595,8 +595,7 @@ dsbt_current_sos (void)
}
so_list *sop = new so_list;
lm_info_dsbt *li = new lm_info_dsbt;
sop->lm_info = li;
auto li = gdb::make_unique<lm_info_dsbt> ();
li->map = loadmap;
/* Fetch the name. */
addr = extract_unsigned_integer (lm_buf.l_name,
@ -618,6 +617,8 @@ dsbt_current_sos (void)
strcpy (sop->so_original_name, sop->so_name);
}
sop->lm_info = std::move (li);
*sos_next_ptr = sop;
sos_next_ptr = &sop->next;
}
@ -883,7 +884,7 @@ static void
dsbt_relocate_section_addresses (so_list &so, target_section *sec)
{
int seg;
auto *li = gdb::checked_static_cast<lm_info_dsbt *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_dsbt *> (so.lm_info.get ());
int_elf32_dsbt_loadmap *map = li->map;
for (seg = 0; seg < map->nsegs; seg++)

13
gdb/solib-frv.c
View file

@ -378,8 +378,7 @@ frv_current_sos (void)
}
so_list *sop = new so_list;
lm_info_frv *li = new lm_info_frv;
sop->lm_info = li;
auto li = gdb::make_unique<lm_info_frv> ();
li->map = loadmap;
li->got_value = got_addr;
li->lm_addr = lm_addr;
@ -402,6 +401,8 @@ frv_current_sos (void)
strcpy (sop->so_original_name, sop->so_name);
}
sop->lm_info = std::move (li);
*sos_next_ptr = sop;
sos_next_ptr = &sop->next;
}
@ -819,7 +820,7 @@ static void
frv_relocate_section_addresses (so_list &so, target_section *sec)
{
int seg;
auto *li = gdb::checked_static_cast<lm_info_frv *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_frv *> (so.lm_info.get ());
int_elf32_fdpic_loadmap *map = li->map;
for (seg = 0; seg < map->nsegs; seg++)
@ -860,7 +861,7 @@ frv_fdpic_find_global_pointer (CORE_ADDR addr)
for (struct so_list *so : current_program_space->solibs ())
{
int seg;
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info);
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info.get ());
int_elf32_fdpic_loadmap *map = li->map;
for (seg = 0; seg < map->nsegs; seg++)
@ -916,7 +917,7 @@ frv_fdpic_find_canonical_descriptor (CORE_ADDR entry_point)
{
for (struct so_list *so : current_program_space->solibs ())
{
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info);
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info.get ());
addr = find_canonical_descriptor_in_load_object
(entry_point, got_value, name, so->abfd, li);
@ -1068,7 +1069,7 @@ frv_fetch_objfile_link_map (struct objfile *objfile)
of shared libraries. */
for (struct so_list *so : current_program_space->solibs ())
{
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info);
auto *li = gdb::checked_static_cast<lm_info_frv *> (so->lm_info.get ());
if (so->objfile == objfile)
return li->lm_addr;

4
gdb/solib-rocm.c
View file

@ -181,7 +181,7 @@ rocm_solib_relocate_section_addresses (so_list &so,
return;
}
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info.get ());
sec->addr = sec->addr + li->l_addr;
sec->endaddr = sec->endaddr + li->l_addr;
}
@ -213,7 +213,7 @@ so_list_from_rocm_sos (const std::vector<rocm_so> &sos)
for (const rocm_so &so : sos)
{
so_list *newobj = new so_list;
newobj->lm_info = new lm_info_svr4 (*so.lm_info);
newobj->lm_info = gdb::make_unique<lm_info_svr4> (*so.lm_info);
strncpy (newobj->so_name, so.name.c_str (), sizeof (newobj->so_name));
newobj->so_name[sizeof (newobj->so_name) - 1] = '\0';

25
gdb/solib-svr4.c
View file

@ -188,8 +188,10 @@ svr4_same (const char *gdb_name, const char *inferior_name,
static int
svr4_same (const so_list &gdb, const so_list &inferior)
{
auto *lmg = gdb::checked_static_cast<const lm_info_svr4 *> (gdb.lm_info);
auto *lmi = gdb::checked_static_cast<const lm_info_svr4 *> (inferior.lm_info);
auto *lmg
= gdb::checked_static_cast<const lm_info_svr4 *> (gdb.lm_info.get ());
auto *lmi
= gdb::checked_static_cast<const lm_info_svr4 *> (inferior.lm_info.get ());
return svr4_same (gdb.so_original_name, inferior.so_original_name,
*lmg, *lmi);
@ -239,7 +241,7 @@ has_lm_dynamic_from_link_map (void)
static CORE_ADDR
lm_addr_check (const so_list &so, bfd *abfd)
{
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info.get ());
if (!li->l_addr_p)
{
@ -979,7 +981,7 @@ svr4_free_objfile_observer (struct objfile *objfile)
static void
svr4_clear_so (const so_list &so)
{
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so.lm_info.get ());
if (li != NULL)
li->l_addr_p = 0;
@ -1001,7 +1003,7 @@ so_list_from_svr4_sos (const std::vector<svr4_so> &sos)
sizeof (newobj->so_name) - 1);
newobj->so_name[sizeof (newobj->so_name) - 1] = 0;
strcpy (newobj->so_original_name, newobj->so_name);
newobj->lm_info = new lm_info_svr4 (*so.lm_info);
newobj->lm_info = gdb::make_unique<lm_info_svr4> (*so.lm_info);
newobj->next = NULL;
*link = newobj;
@ -1192,13 +1194,14 @@ svr4_default_sos (svr4_info *info)
return NULL;
so_list *newobj = new so_list;
lm_info_svr4 *li = new lm_info_svr4;
newobj->lm_info = li;
auto li = gdb::make_unique<lm_info_svr4> ();
/* Nothing will ever check the other fields if we set l_addr_p. */
li->l_addr = li->l_addr_inferior = info->debug_loader_offset;
li->l_addr_p = 1;
newobj->lm_info = std::move (li);
strncpy (newobj->so_name, info->debug_loader_name, SO_NAME_MAX_PATH_SIZE - 1);
newobj->so_name[SO_NAME_MAX_PATH_SIZE - 1] = '\0';
strcpy (newobj->so_original_name, newobj->so_name);
@ -1495,7 +1498,7 @@ svr4_current_sos (void)
[ 9] .dynamic DYNAMIC ffffffffff700580 000580 0000f0
*/
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so->lm_info);
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so->lm_info.get ());
if (address_in_mem_range (li->l_ld, &vsyscall_range))
{
@ -1531,7 +1534,8 @@ svr4_fetch_objfile_link_map (struct objfile *objfile)
for (struct so_list *so : current_program_space->solibs ())
if (so->objfile == objfile)
{
auto *li = gdb::checked_static_cast<lm_info_svr4 *> (so->lm_info);
auto *li
= gdb::checked_static_cast<lm_info_svr4 *> (so->lm_info.get ());
return li->lm_addr;
}
@ -3292,7 +3296,8 @@ find_debug_base_for_solib (so_list *solib)
svr4_info *info = get_svr4_info (current_program_space);
gdb_assert (info != nullptr);
const lm_info_svr4 *lm_info = (const lm_info_svr4 *) solib->lm_info;
auto *lm_info
= gdb::checked_static_cast<const lm_info_svr4 *> (solib->lm_info.get ());
for (const auto &tuple : info->solib_lists)
{

4
gdb/solib-target.c
View file

@ -260,7 +260,7 @@ solib_target_current_sos (void)
/* We no longer need this copy of the name. */
info->name.clear ();
new_solib->lm_info = info.release ();
new_solib->lm_info = std::move (info);
/* Add it to the list. */
if (!start)
@ -285,7 +285,7 @@ static void
solib_target_relocate_section_addresses (so_list &so, target_section *sec)
{
CORE_ADDR offset;
auto *li = gdb::checked_static_cast<lm_info_target *> (so.lm_info);
auto *li = gdb::checked_static_cast<lm_info_target *> (so.lm_info.get ());
/* Build the offset table only once per object file. We can not do
it any earlier, since we need to open the file first. */

1
gdb/solib.c
View file

@ -675,7 +675,6 @@ void
free_so (so_list &so)
{
clear_so (so);
delete so.lm_info;
delete &so;
}

4
gdb/solist.h
View file

@ -34,6 +34,8 @@ struct lm_info
virtual ~lm_info () = 0;
};
using lm_info_up = std::unique_ptr<lm_info>;
struct so_list
{
/* The following fields of the structure come directly from the
@ -46,7 +48,7 @@ struct so_list
will be a copy of struct link_map from the user process, but
it need not be; it can be any collection of data needed to
traverse the dynamic linker's data structures. */
struct lm_info *lm_info = nullptr;
lm_info_up lm_info;
/* Shared object file name, exactly as it appears in the
inferior's link map. This may be a relative path, or something