From cc86f28f31e2122fb7429cdd19936846e0821606 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Mon, 14 Jan 2008 14:53:47 +0000 Subject: [PATCH] (DEBUG_INFO_UNAVAILABLE): Value stored in num_debug_info_entries when the .debug_info section could not be loaded/parsed. (process_debug_info): Display the length of the compilation unit in hex, so that it corresponds with the offsets that will follow. Tell the user if the length was 32-bit or 64-bit. If a DIE abbreviation could not be found, tell the user the offset of the DIE. (free_debug_memory): Do not attempt to free any entries in the debug_information array if num_debug_info_entries is set to DEBUG_INFO_UNAVAILABLE. --- binutils/ChangeLog | 10 ++++++++++ binutils/dwarf.c | 39 +++++++++++++++++++++++---------------- 2 files changed, 33 insertions(+), 16 deletions(-) diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 94330de..3435a48 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -8,6 +8,16 @@ and return whilst displaying a warning message if the load failed. (display_debug_loc): Likewise. (display_debug_ranges): Likewise. + (DEBUG_INFO_UNAVAILABLE): Value stored in num_debug_info_entries + when the .debug_info section could not be loaded/parsed. + (process_debug_info): Display the length of the compilation unit + in hex, so that it corresponds with the offsets that will follow. + Tell the user if the length was 32-bit or 64-bit. + If a DIE abbreviation could not be found, tell the user the offset + of the DIE. + (free_debug_memory): Do not attempt to free any entries in the + debug_information array if num_debug_info_entries is set to + DEBUG_INFO_UNAVAILABLE. 2008-01-10 Andreas Schwab diff --git a/binutils/dwarf.c b/binutils/dwarf.c index bac80bc..f211237 100644 --- a/binutils/dwarf.c +++ b/binutils/dwarf.c @@ -34,6 +34,9 @@ static int warned_about_missing_comp_units = FALSE; static unsigned int num_debug_info_entries = 0; static debug_info *debug_information = NULL; +/* Special value for num_debug_info_entries to indicate + that the .debug_info section could not be loaded/parsed. */ +#define DEBUG_INFO_UNAVAILABLE (unsigned int) -1 dwarf_vma eh_addr_size; @@ -1823,7 +1826,8 @@ process_debug_info (struct dwarf_section *section, void *file, if (!do_loc) { printf (_(" Compilation Unit @ offset 0x%lx:\n"), cu_offset); - printf (_(" Length: %ld\n"), compunit.cu_length); + printf (_(" Length: 0x%lx (%s)\n"), compunit.cu_length, + initial_length_size == 8 ? "64-bit" : "32-bit"); printf (_(" Version: %d\n"), compunit.cu_version); printf (_(" Abbrev Offset: %ld\n"), compunit.cu_abbrev_offset); printf (_(" Pointer Size: %d\n"), compunit.cu_pointer_size); @@ -1913,8 +1917,8 @@ process_debug_info (struct dwarf_section *section, void *file, printf ("\n"); fflush (stdout); } - warn (_("Unable to locate entry %lu in the abbreviation table\n"), - abbrev_number); + warn (_("DIE at offset %lx refers to abbreviation number %lu which does not exist\n"), + die_offset, abbrev_number); return 0; } @@ -1988,7 +1992,7 @@ load_debug_info (void * file) /* If we have already tried and failed to load the .debug_info section then do not bother to repear the task. */ - if (num_debug_info_entries == (unsigned) -1) + if (num_debug_info_entries == DEBUG_INFO_UNAVAILABLE) return 0; /* If we already have the information there is nothing else to do. */ @@ -1999,7 +2003,7 @@ load_debug_info (void * file) && process_debug_info (&debug_displays [info].section, file, 1)) return num_debug_info_entries; - num_debug_info_entries = (unsigned) -1; + num_debug_info_entries = DEBUG_INFO_UNAVAILABLE; return 0; } @@ -3244,7 +3248,7 @@ display_debug_frames (struct dwarf_section *section, fc->ncols = 0; fc->col_type = xmalloc (sizeof (short int)); fc->col_offset = xmalloc (sizeof (int)); - frame_need_space (fc, max_regs-1); + frame_need_space (fc, max_regs - 1); version = *start++; @@ -3386,7 +3390,7 @@ display_debug_frames (struct dwarf_section *section, fc->cfa_reg = cie->cfa_reg; fc->cfa_offset = cie->cfa_offset; fc->ra = cie->ra; - frame_need_space (fc, max_regs-1); + frame_need_space (fc, max_regs - 1); fc->fde_encoding = cie->fde_encoding; } @@ -3713,7 +3717,7 @@ display_debug_frames (struct dwarf_section *section, if (rs) { remembered_state = rs->next; - frame_need_space (fc, rs->ncols-1); + frame_need_space (fc, rs->ncols - 1); memcpy (fc->col_type, rs->col_type, rs->ncols); memcpy (fc->col_offset, rs->col_offset, rs->ncols * sizeof (int)); @@ -3958,23 +3962,26 @@ free_debug_memory (void) for (i = 0; i < max; i++) free_debug_section (i); - if (debug_information) + if (debug_information != NULL) { - for (i = 0; i < num_debug_info_entries; i++) + if (num_debug_info_entries != DEBUG_INFO_UNAVAILABLE) { - if (!debug_information [i].max_loc_offsets) + for (i = 0; i < num_debug_info_entries; i++) { - free (debug_information [i].loc_offsets); - free (debug_information [i].have_frame_base); + if (!debug_information [i].max_loc_offsets) + { + free (debug_information [i].loc_offsets); + free (debug_information [i].have_frame_base); + } + if (!debug_information [i].max_range_lists) + free (debug_information [i].range_lists); } - if (!debug_information [i].max_range_lists) - free (debug_information [i].range_lists); } + free (debug_information); debug_information = NULL; num_debug_info_entries = 0; } - } struct dwarf_section_display debug_displays[] = -- 2.7.4