This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] Mach-O: fix two memory leaks


This patch fixes the memory leaks I've noticed while writing my
previous patch. The patch which can be applied to the CVS head:

I'd like to add a few descriptions about this although they might be
too obvious for binutils gurus...

For bfd_mach_o_canonicalize_reloc, this leak can be reproduced by something like

% valgrind --leak-check=full objdump -b mach-o-x86-64 -x hello.o

As for bfd_mach_o_canonicalize_dynamic_reloc, I confirmed this patch
fixes the leaks with a dynamic library created by

% gcc -dynamiclib mach/hello.c -mmacosx-version-min=10.4 -o hello.dylib

This patch compiled with --enable-targets=all and "make check" passed.

2011-12-13  Shinichiro Hamaji  <>

	* mach-o.c (bfd_mach_o_canonicalize_reloc): Use bfd_alloc instead
	of bfd_malloc to get rid of memory leaks.
	(bfd_mach_o_canonicalize_dynamic_reloc): Free the malloced

diff --git a/bfd/mach-o.c b/bfd/mach-o.c
index 54edd07..32d6d6c 100644
--- a/bfd/mach-o.c
+++ b/bfd/mach-o.c
@@ -810,7 +810,7 @@ bfd_mach_o_canonicalize_reloc (bfd *abfd, asection *asect,
   if (bed->_bfd_mach_o_swap_reloc_in == NULL)
     return 0;

-  res = bfd_malloc (asect->reloc_count * sizeof (arelent));
+  res = bfd_alloc (abfd, asect->reloc_count * sizeof (arelent));
   if (res == NULL)
     return -1;

@@ -881,6 +881,7 @@ bfd_mach_o_canonicalize_dynamic_reloc (bfd *abfd,
arelent **rels,
   for (i = 0; i < dysymtab->nextrel + dysymtab->nlocrel; i++)
     rels[i] = &res[i];
   rels[i] = NULL;
+  free (res);
   return i;

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]