From: Alan Modra <amodra@gmail.com>
Date: Tue, 23 Oct 2018 07:59:24 +0000 (+1030)
Subject: PR23804, buffer overflow in sec_merge_hash_lookup
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=ab419ddbb2cdd17ca83618990f2cacf904ce1d61;p=binutils-gdb.git

PR23804, buffer overflow in sec_merge_hash_lookup

	PR 23804
	* merge.c (_bfd_add_merge_section): Don't attempt to merge
	sections where size is not a multiple of entsize.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 31ff3d6db43..da423b14763 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2018-10-23  Alan Modra  <amodra@gmail.com>
+
+	PR 23804
+	* merge.c (_bfd_add_merge_section): Don't attempt to merge
+	sections where size is not a multiple of entsize.
+
 2018-10-20  Alan Modra  <amodra@gmail.com>
 
 	PR 23788
diff --git a/bfd/merge.c b/bfd/merge.c
index 79045529427..5e3bba09825 100644
--- a/bfd/merge.c
+++ b/bfd/merge.c
@@ -376,6 +376,9 @@ _bfd_add_merge_section (bfd *abfd, void **psinfo, asection *sec,
       || sec->entsize == 0)
     return TRUE;
 
+  if (sec->size % sec->entsize != 0)
+    return TRUE;
+
   if ((sec->flags & SEC_RELOC) != 0)
     {
       /* We aren't prepared to handle relocations in merged sections.  */