X-Git-Url: http://drtracing.org/?a=blobdiff_plain;f=gdb%2Fsolib-target.c;h=ba056478c061f22deb9dc692c2f324ce430c8a2f;hb=423054beadf317757d43aefb9eddfe711a42e3e1;hp=01171f85dd2eeaa341297207a1d0e268ba480f4a;hpb=b7b030adc405017f01e996a90f85e40730ef8397;p=deliverable%2Fbinutils-gdb.git diff --git a/gdb/solib-target.c b/gdb/solib-target.c index 01171f85dd..ba056478c0 100644 --- a/gdb/solib-target.c +++ b/gdb/solib-target.c @@ -1,6 +1,6 @@ /* Definitions for targets which report shared library events. - Copyright (C) 2007-2017 Free Software Foundation, Inc. + Copyright (C) 2007-2020 Free Software Foundation, Inc. This file is part of GDB. @@ -23,7 +23,6 @@ #include "symtab.h" #include "symfile.h" #include "target.h" -#include "vec.h" #include "solib-target.h" #include @@ -47,15 +46,14 @@ struct lm_info_target : public lm_info_base /* The cached offsets for each section of this shared library, determined from SEGMENT_BASES, or SECTION_BASES. */ - section_offsets *offsets = NULL; + section_offsets offsets; }; -typedef lm_info_target *lm_info_target_p; -DEF_VEC_P(lm_info_target_p); +typedef std::vector> lm_info_vector; #if !defined(HAVE_LIBEXPAT) -static VEC(lm_info_target_p) * +static lm_info_vector solib_target_parse_libraries (const char *library) { static int have_warned; @@ -67,7 +65,7 @@ solib_target_parse_libraries (const char *library) "at compile time")); } - return NULL; + return lm_info_vector (); } #else /* HAVE_LIBEXPAT */ @@ -79,12 +77,13 @@ solib_target_parse_libraries (const char *library) static void library_list_start_segment (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, - void *user_data, VEC(gdb_xml_value_s) *attributes) + void *user_data, + std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; - lm_info_target *last = VEC_last (lm_info_target_p, *list); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *last = list->back ().get (); ULONGEST *address_p - = (ULONGEST *) xml_find_attribute (attributes, "address")->value; + = (ULONGEST *) xml_find_attribute (attributes, "address")->value.get (); CORE_ADDR address = (CORE_ADDR) *address_p; if (!last->section_bases.empty ()) @@ -97,12 +96,13 @@ library_list_start_segment (struct gdb_xml_parser *parser, static void library_list_start_section (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, - void *user_data, VEC(gdb_xml_value_s) *attributes) + void *user_data, + std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; - lm_info_target *last = VEC_last (lm_info_target_p, *list); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *last = list->back ().get (); ULONGEST *address_p - = (ULONGEST *) xml_find_attribute (attributes, "address")->value; + = (ULONGEST *) xml_find_attribute (attributes, "address")->value.get (); CORE_ADDR address = (CORE_ADDR) *address_p; if (!last->segment_bases.empty ()) @@ -117,15 +117,15 @@ library_list_start_section (struct gdb_xml_parser *parser, static void library_list_start_library (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, - void *user_data, VEC(gdb_xml_value_s) *attributes) + void *user_data, + std::vector &attributes) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; + lm_info_vector *list = (lm_info_vector *) user_data; lm_info_target *item = new lm_info_target; - const char *name - = (const char *) xml_find_attribute (attributes, "name")->value; + item->name + = (const char *) xml_find_attribute (attributes, "name")->value.get (); - item->name = xstrdup (name); - VEC_safe_push (lm_info_target_p, *list, item); + list->emplace_back (item); } static void @@ -133,8 +133,8 @@ library_list_end_library (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, void *user_data, const char *body_text) { - VEC(lm_info_target_p) **list = (VEC(lm_info_target_p) **) user_data; - lm_info_target *lm_info = VEC_last (lm_info_target_p, *list); + lm_info_vector *list = (lm_info_vector *) user_data; + lm_info_target *lm_info = list->back ().get (); if (lm_info->segment_bases.empty () && lm_info->section_bases.empty ()) gdb_xml_error (parser, _("No segment or section bases defined")); @@ -146,14 +146,15 @@ library_list_end_library (struct gdb_xml_parser *parser, static void library_list_start_list (struct gdb_xml_parser *parser, const struct gdb_xml_element *element, - void *user_data, VEC(gdb_xml_value_s) *attributes) + void *user_data, + std::vector &attributes) { struct gdb_xml_value *version = xml_find_attribute (attributes, "version"); /* #FIXED attribute may be omitted, Expat returns NULL in such case. */ if (version != NULL) { - const char *string = (const char *) version->value; + const char *string = (const char *) version->value.get (); if (strcmp (string, "1.0") != 0) gdb_xml_error (parser, @@ -162,22 +163,6 @@ library_list_start_list (struct gdb_xml_parser *parser, } } -/* Discard the constructed library list. */ - -static void -solib_target_free_library_list (void *p) -{ - VEC(lm_info_target_p) **result = (VEC(lm_info_target_p) **) p; - lm_info_target *info; - int ix; - - for (ix = 0; VEC_iterate (lm_info_target_p, *result, ix, info); ix++) - delete info; - - VEC_free (lm_info_target_p, *result); - *result = NULL; -} - /* The allowed elements and attributes for an XML library list. The root element is a . */ @@ -224,23 +209,20 @@ static const struct gdb_xml_element library_list_elements[] = { { NULL, NULL, NULL, GDB_XML_EF_NONE, NULL, NULL } }; -static VEC(lm_info_target_p) * +static lm_info_vector solib_target_parse_libraries (const char *library) { - VEC(lm_info_target_p) *result = NULL; - struct cleanup *back_to = make_cleanup (solib_target_free_library_list, - &result); + lm_info_vector result; if (gdb_xml_parse_quick (_("target library list"), "library-list.dtd", library_list_elements, library, &result) == 0) { - /* Parsed successfully, keep the result. */ - discard_cleanups (back_to); + /* Parsed successfully. */ return result; } - do_cleanups (back_to); - return NULL; + result.clear (); + return result; } #endif @@ -248,24 +230,23 @@ static struct so_list * solib_target_current_sos (void) { struct so_list *new_solib, *start = NULL, *last = NULL; - VEC(lm_info_target_p) *library_list; - lm_info_target *info; - int ix; /* Fetch the list of shared libraries. */ - gdb::unique_xmalloc_ptr library_document - = target_read_stralloc (¤t_target, TARGET_OBJECT_LIBRARIES, NULL); - if (library_document == NULL) + gdb::optional library_document + = target_read_stralloc (current_top_target (), TARGET_OBJECT_LIBRARIES, + NULL); + if (!library_document) return NULL; /* Parse the list. */ - library_list = solib_target_parse_libraries (library_document.get ()); + lm_info_vector library_list + = solib_target_parse_libraries (library_document->data ()); - if (library_list == NULL) + if (library_list.empty ()) return NULL; /* Build a struct so_list for each entry on the list. */ - for (ix = 0; VEC_iterate (lm_info_target_p, library_list, ix, info); ix++) + for (auto &&info : library_list) { new_solib = XCNEW (struct so_list); strncpy (new_solib->so_name, info->name.c_str (), @@ -274,11 +255,12 @@ solib_target_current_sos (void) strncpy (new_solib->so_original_name, info->name.c_str (), SO_NAME_MAX_PATH_SIZE - 1); new_solib->so_original_name[SO_NAME_MAX_PATH_SIZE - 1] = '\0'; - new_solib->lm_info = info; /* We no longer need this copy of the name. */ info->name.clear (); + new_solib->lm_info = info.release (); + /* Add it to the list. */ if (!start) last = start = new_solib; @@ -289,9 +271,6 @@ solib_target_current_sos (void) } } - /* Free the library list, but not its members. */ - VEC_free (lm_info_target_p, library_list); - return start; } @@ -326,13 +305,11 @@ solib_target_relocate_section_addresses (struct so_list *so, /* Build the offset table only once per object file. We can not do it any earlier, since we need to open the file first. */ - if (li->offsets == NULL) + if (li->offsets.empty ()) { int num_sections = gdb_bfd_count_sections (so->abfd); - li->offsets - = ((struct section_offsets *) - xzalloc (SIZEOF_N_SECTION_OFFSETS (num_sections))); + li->offsets.assign (num_sections, 0); if (!li->section_bases.empty ()) { @@ -343,7 +320,7 @@ solib_target_relocate_section_addresses (struct so_list *so, for (i = 0, sect = so->abfd->sections; sect != NULL; i++, sect = sect->next) - if ((bfd_get_section_flags (so->abfd, sect) & SEC_ALLOC)) + if ((bfd_section_flags (sect) & SEC_ALLOC)) num_alloc_sections++; if (num_alloc_sections != li->section_bases.size ()) @@ -361,14 +338,14 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), sect != NULL; i++, sect = sect->next) { - if (!(bfd_get_section_flags (so->abfd, sect) & SEC_ALLOC)) + if (!(bfd_section_flags (sect) & SEC_ALLOC)) continue; - if (bfd_section_size (so->abfd, sect) > 0) + if (bfd_section_size (sect) > 0) { CORE_ADDR low, high; low = li->section_bases[i]; - high = low + bfd_section_size (so->abfd, sect) - 1; + high = low + bfd_section_size (sect) - 1; if (low < so->addr_low) so->addr_low = low; @@ -377,7 +354,7 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), gdb_assert (so->addr_low <= so->addr_high); found_range = 1; } - li->offsets->offsets[i] = li->section_bases[bases_index]; + li->offsets[i] = li->section_bases[bases_index]; bases_index++; } if (!found_range) @@ -387,9 +364,9 @@ Could not relocate shared library \"%s\": wrong number of ALLOC sections"), } else if (!li->segment_bases.empty ()) { - struct symfile_segment_data *data; + symfile_segment_data_up data + = get_symfile_segment_data (so->abfd); - data = get_symfile_segment_data (so->abfd); if (data == NULL) warning (_("\ Could not relocate shared library \"%s\": no segments"), so->so_name); @@ -398,7 +375,8 @@ Could not relocate shared library \"%s\": no segments"), so->so_name); ULONGEST orig_delta; int i; - if (!symfile_map_offsets_to_segments (so->abfd, data, li->offsets, + if (!symfile_map_offsets_to_segments (so->abfd, data.get (), + li->offsets, li->segment_bases.size (), li->segment_bases.data ())) warning (_("\ @@ -408,9 +386,9 @@ Could not relocate shared library \"%s\": bad offsets"), so->so_name); "info sharedlibrary". Report any consecutive segments which were relocated as a single unit. */ gdb_assert (li->segment_bases.size () > 0); - orig_delta = li->segment_bases[0] - data->segment_bases[0]; + orig_delta = li->segment_bases[0] - data->segments[0].base; - for (i = 1; i < data->num_segments; i++) + for (i = 1; i < data->segments.size (); i++) { /* If we have run out of offsets, assume all remaining segments have the same offset. */ @@ -419,25 +397,22 @@ Could not relocate shared library \"%s\": bad offsets"), so->so_name); /* If this segment does not have the same offset, do not include it in the library's range. */ - if (li->segment_bases[i] - data->segment_bases[i] + if (li->segment_bases[i] - data->segments[i].base != orig_delta) break; } so->addr_low = li->segment_bases[0]; - so->addr_high = (data->segment_bases[i - 1] - + data->segment_sizes[i - 1] + so->addr_high = (data->segments[i - 1].base + + data->segments[i - 1].size + orig_delta); gdb_assert (so->addr_low <= so->addr_high); - - free_symfile_segment_data (data); } } } - offset = li->offsets->offsets[gdb_bfd_section_index - (sec->the_bfd_section->owner, - sec->the_bfd_section)]; + offset = li->offsets[gdb_bfd_section_index (sec->the_bfd_section->owner, + sec->the_bfd_section)]; sec->addr += offset; sec->endaddr += offset; } @@ -461,8 +436,9 @@ solib_target_in_dynsym_resolve_code (CORE_ADDR pc) struct target_so_ops solib_target_so_ops; +void _initialize_solib_target (); void -_initialize_solib_target (void) +_initialize_solib_target () { solib_target_so_ops.relocate_section_addresses = solib_target_relocate_section_addresses;