From ed38cbc331efbfe65def811773e68c431bfbcc75 Mon Sep 17 00:00:00 2001 From: Indu Bhagat Date: Fri, 9 Dec 2022 10:25:46 -0800 Subject: [PATCH] objdump: sframe: fix memory leaks ChangeLog: * binutils/objdump.c (dump_section_sframe): free up contents and SFrame decoder context on exit. --- binutils/objdump.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/binutils/objdump.c b/binutils/objdump.c index d95c8b68bf0..6695c5d343f 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -4871,12 +4871,18 @@ dump_section_sframe (bfd *abfd ATTRIBUTE_UNUSED, /* Decode the contents of the section. */ sfd_ctx = sframe_decode ((const char*)sframe_data, sf_size, &err); if (!sfd_ctx) - bfd_fatal (bfd_get_filename (abfd)); + { + free (sframe_data); + bfd_fatal (bfd_get_filename (abfd)); + } printf (_("Contents of the SFrame section %s:"), sanitize_string (sect_name)); /* Dump the contents as text. */ dump_sframe (sfd_ctx, sf_vma); + + free (sframe_data); + sframe_decoder_free (&sfd_ctx); } -- 2.30.2