binutils/dwarf.c

Jan Beulich jbeulich@suse.com
Sun Jun 29 13:25:01 GMT 2025


On 29.06.2025 06:50, Vladimir Mezentsev via Binutils wrote:
> 
> % cat -n show binutils/dwarf.c
> ...
>  12328      filename = xmalloc (strlen (".build-id/")
>  12329                  + build_id_size * 2 + 2
>  12330                  + strlen (".debug")
>  12331                  /* The next string should be the same as the longest
>  12332                 name found in the prefixes[] array below. */
>  12333                  + strlen ("/usrlib64/debug/usr")
> 
> Should be:              + strlen ("/usr/lib64/debug/usr")

It really should be /usr/lib64/debug/usr/, and ...

>  12334                  + 1);
>  12335      void * handle;
>  12336
>  12337      static const char * prefixes[] =
>  12338        {
>  12339          "",
>  12340          ".debug/",
>  12341          "/usr/lib/debug/",
>  12342          "/usr/lib/debug/usr/",
>  12343          "/usr/lib64/debug/",
>  12344          "/usr/lib64/debug/usr"

... this should have a slash appended as well. What I wonder is why you
didn't send the correction in patch form right away.

Jan


More information about the Binutils mailing list