[binutils-gdb] bfd_check_format memory leak

[gdb-buildbot@sergiodj.net]

*** TEST RESULTS FOR COMMIT 9d78076ef8ef07890ad89c1122bdf49932a979a5 ***

commit 9d78076ef8ef07890ad89c1122bdf49932a979a5
Author:     Alan Modra <amodra@gmail.com>
AuthorDate: Fri Dec 20 10:03:30 2019 +1030
Commit:     Alan Modra <amodra@gmail.com>
CommitDate: Fri Dec 20 13:36:06 2019 +1030

    bfd_check_format memory leak
    
            * format.c (bfd_check_format_matches): Free matching_vector when
            not returning matching target strings.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index c399eac35a..61d1991159 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-20  Alan Modra  <amodra@gmail.com>
+
+	* format.c (bfd_check_format_matches): Free matching_vector when
+	not returning matching target strings.
+
 2019-12-20  Alan Modra  <amodra@gmail.com>
 
 	* coff-alpha.c (alpha_ecoff_read_ar_hdr): Free ar_hdr on error return.
diff --git a/bfd/format.c b/bfd/format.c
index 1d1363d184..17b48556a9 100644
--- a/bfd/format.c
+++ b/bfd/format.c
@@ -513,6 +513,8 @@ bfd_check_format_matches (bfd *abfd, bfd_format format, char ***matching)
 	  *(const char **) &matching_vector[match_count] = name;
 	}
     }
+  else if (matching_vector)
+    free (matching_vector);
   return FALSE;
 }