Re: [Elftoolchain-developers] small memory leak in libdwarf.

[Joseph K. <jk...@us...>]

2017-08-31 4:47 GMT+01:00 John Kearney <det...@gm...>:
> Not sure if this is the correct plae to post this?
>
> This is just a small memory leak.fix.
>
> svn diff libdwarf/libdwarf_elf_init.c
> Index: libdwarf/libdwarf_elf_init.c
> ===================================================================
> --- libdwarf/libdwarf_elf_init.c        (revision 3564)
> +++ libdwarf/libdwarf_elf_init.c        (working copy)
> @@ -383,6 +383,7 @@
>         if (e->eo_shdr)
>                 free(e->eo_shdr);
>
> +       elf_end(e->eo_elf);
>         free(e);
>         free(iface);

Thanks for your email.

Unfortunately the e->eo_elf need not be owned by libdwarf; it could have been
used as a parameter to the dwarf_elf_init() API.  We would need to track
ownership using an internal flag of some kind.

I have filed ticket #553 to track this bug.

Regards,
Joseph Koshy