cmdline_add_object_only_section leak

Free ofilename on error path.  Don't bother testing "if (foo)" before
"free (foo)".

diff --git a/ld/ldlang.c b/ld/ldlang.c
index 2e82eef61feaeba347abfba349687853571e0dc8..74c0271973f643bb401a5e2941ae604df174e6ac 100644 (file)
--- a/ld/ldlang.c
+++ b/ld/ldlang.c
@@ -10778,8 +10778,7 @@ cmdline_add_object_only_section (bfd_byte *contents, size_t size)
 
   /* Must be freed after bfd_close ().  */
   free (isympp);
-  if (osympp)
-    free (osympp);
+  free (osympp);
 
   if (rename (ofilename, output_filename))
     {
@@ -10791,14 +10790,15 @@ cmdline_add_object_only_section (bfd_byte *contents, size_t size)
   return;
 
 loser:
-  if (isympp)
-    free (isympp);
-  if (osympp)
-    free (osympp);
+  free (isympp);
+  free (osympp);
   if (obfd)
     bfd_close (obfd);
   if (ofilename)
-    unlink (ofilename);
+    {
+      unlink (ofilename);
+      free (ofilename);
+    }
   einfo (_("%P%F: failed to add object-only section: %s\n"), err);
 }