summaryrefslogtreecommitdiff
path: root/binutils/coffdump.c
diff options
context:
space:
mode:
authorAlan Modra <amodra@gmail.com>2022-06-16 14:48:13 +0930
committerAlan Modra <amodra@gmail.com>2022-06-16 15:01:14 +0930
commit370426d0da768345fb53683c803d6d5a20558065 (patch)
treedc3e9cab9672e0a09b4113ecf0b7365780d07fbc /binutils/coffdump.c
parent8ad7c8be4b5705a1138f7b2922ab7611e1d61e06 (diff)
Always free matching vector from bfd_check_format_matches
At least one place calling list_matching_formats failed to free the "matching" vector from bfd_check_format_matches afterwards. Fix that by calling free inside list_matching_formats. binutils/ * bucomm.c (list_matching_formats): Free arg. * addr2line.c (process_file): Adjust to suit. * ar.c (open_inarch, ranlib_touch): Likewise. * coffdump.c (main): Likewise. * nm.c (display_archive, display_file): Likewise. * objcopy.c (copy_file): Likewise. * objdump.c (display_object_bfd): Likewise. * size.c (display_bfd): Likewise. * srconv.c (main): Likewise. ld/ * ldlang.c (load_symbols): Free "matching".
Diffstat (limited to 'binutils/coffdump.c')
-rw-r--r--binutils/coffdump.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/binutils/coffdump.c b/binutils/coffdump.c
index d760a6ccb3..bccc395e5f 100644
--- a/binutils/coffdump.c
+++ b/binutils/coffdump.c
@@ -541,10 +541,7 @@ main (int ac, char **av)
bfd_nonfatal (input_file);
if (bfd_get_error () == bfd_error_file_ambiguously_recognized)
- {
- list_matching_formats (matching);
- free (matching);
- }
+ list_matching_formats (matching);
exit (1);
}