[PATCH][binutils] Fix debug_rnglists test in display_debug_ranges

Alan Modra amodra@gmail.com
Wed Feb 10 03:54:49 GMT 2021


On Tue, Feb 09, 2021 at 05:56:54PM +0100, Tom de Vries wrote:
> --- a/binutils/dwarf.c
> +++ b/binutils/dwarf.c
> @@ -7581,7 +7581,7 @@ display_debug_ranges (struct dwarf_section *section,
>    unsigned char *finish = start + bytes;
>    unsigned int num_range_list, i;
>    struct range_entry *range_entries, *range_entry_fill;
> -  int is_rnglists = strstr (section->name, "debug_rnglists") != NULL;
> +  int is_rnglists = strstr (section->name, ".debug_rnglists") != NULL;
>    /* Initialize it due to a false compiler warning.  */
>    unsigned char address_size = 0;
>    dwarf_vma last_offset = 0;

I must be having a slow brain day, but I don't see what the above is
doing besides making .zdebug_rnglists not match.  Isn't the whole
point of using strstr here to allow the compressed section prefix?

-- 
Alan Modra
Australia Development Lab, IBM


More information about the Binutils mailing list