[Bug binutils/27371] [readelf] .debug_rnglists section header not parsed

[vries at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=27371

--- Comment #4 from Tom de Vries <vries at gcc dot gnu.org> ---
(In reply to Nick Clifton from comment #3)
> (In reply to Tom de Vries from comment #1)
> Hi Tom,
> 
> I do not quite get this:
> 
>   > -  int is_rnglists = strstr (section->name, "debug_rnglists") != NULL;
>   > +  int is_rnglists = strstr (section->name, ".debug_rnglists") != NULL;
> 
> The strstr() function locates a sub-string within a a longer string,
> so won't both versions of the code above produce the same results ?
> 
> I think that the reason that the test did not include the "." prefix
> was that it is meant to work with compressed range list sections as
> well, ie .zdebug_rnglists.

Hi Nick,

you're right, this is incorrect.  Alan had the same comment.  I've resubmitted
here: https://sourceware.org/pipermail/binutils/2021-February/115309.html

Thanks,
- Tom

-- 
You are receiving this mail because:
You are on the CC list for the bug.