From: Michael Snyder <msnyder@vmware.com>
Date: Wed, 1 Aug 2007 19:58:40 +0000 (+0000)
Subject: 2007-08-01  Michael Snyder  <msnyder@access-company.com>
X-Git-Tag: binutils-2_18-branchpoint~35
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=20c6e70e8d514109c569b17a279c2dcd70b229fc;p=external%2Fbinutils.git

2007-08-01  Michael Snyder  <msnyder@access-company.com>

	* aoutx.h (aout_link_add_symbols): Return if count is zero.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index ea5ece5..f8c3343 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,7 @@
 2007-08-01  Michael Snyder  <msnyder@access-company.com>
 
+	* aoutx.h (aout_link_add_symbols): Return if count is zero.
+
 	* elf.c (bfd_elf_print_symbol): Macro dereferences pointer, so
 	pointer must be non-null.
 
diff --git a/bfd/aoutx.h b/bfd/aoutx.h
index 78b0594..1244510 100644
--- a/bfd/aoutx.h
+++ b/bfd/aoutx.h
@@ -2959,13 +2959,16 @@ aout_link_add_symbols (bfd *abfd, struct bfd_link_info *info)
 	return FALSE;
     }
 
+  if (sym_count == 0)
+    return TRUE;		/* Nothing to do.  */
+
   /* We keep a list of the linker hash table entries that correspond
      to particular symbols.  We could just look them up in the hash
      table, but keeping the list is more efficient.  Perhaps this
      should be conditional on info->keep_memory.  */
   amt = sym_count * sizeof (struct aout_link_hash_entry *);
   sym_hash = bfd_alloc (abfd, amt);
-  if (sym_hash == NULL && sym_count != 0)
+  if (sym_hash == NULL)
     return FALSE;
   obj_aout_sym_hashes (abfd) = sym_hash;