From: Alan Modra <amodra@gmail.com>
Date: Wed, 1 Jan 2025 12:22:41 +0000 (+1030)
Subject: gnu_debuglink related memory leak
X-Git-Tag: binutils-2_44~287
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=822bda23f618286724e449815c8da8cde03e32e0;p=thirdparty%2Fbinutils-gdb.git

gnu_debuglink related memory leak

	* opncls.c (bfd_fill_in_gnu_debuglink_section): Free section
	contents on success too.
---

diff --git a/bfd/opncls.c b/bfd/opncls.c
index fb0bca7689a..a0b30f568d7 100644
--- a/bfd/opncls.c
+++ b/bfd/opncls.c
@@ -1768,7 +1768,7 @@ bfd_fill_in_gnu_debuglink_section (bfd *abfd,
   debuglink_size &= ~3;
   debuglink_size += 4;
 
-  contents = (char *) bfd_malloc (debuglink_size);
+  contents = bfd_malloc (debuglink_size);
   if (contents == NULL)
     {
       /* XXX Should we delete the section from the bfd ?  */
@@ -1781,14 +1781,9 @@ bfd_fill_in_gnu_debuglink_section (bfd *abfd,
 
   bfd_put_32 (abfd, crc32, contents + crc_offset);
 
-  if (! bfd_set_section_contents (abfd, sect, contents, 0, debuglink_size))
-    {
-      /* XXX Should we delete the section from the bfd ?  */
-      free (contents);
-      return false;
-    }
-
-  return true;
+  bool ret = bfd_set_section_contents (abfd, sect, contents, 0, debuglink_size);
+  free (contents);
+  return ret;
 }
 
 /* Finds the build-id associated with @var{abfd}.  If the build-id is