[PATCH 2/4] objdump: Add variable types and structure for DWARF info parsing
Alan Modra
amodra@gmail.com
Fri Aug 22 11:44:25 GMT 2025
On Fri, Aug 22, 2025 at 09:48:26AM +0200, Jan Beulich wrote:
> On 19.08.2025 04:12, Guillaume VACHERIAS wrote:
> > + free_mapping_info_struct ();
> > + xexit (1);
>
> Alan - what's our general position towards such freeing before exiting
> a tool? Overally that's quite a bit of code (most sitting further down),
> which for all practical purposes (outside of running analysis tools) is
> useless.
As you say, it's a complete waste of time except when compiling with
-fsanitize=address or using other memory analysis tools. However,
enough people do that, and report bugs, that I've added quite a bit of
code to free memory. The justification for freeing is to make more
important memory leaks, eg. a leak per symbol, stand out.
--
Alan Modra
More information about the Binutils
mailing list