From: Richard Henderson <rth@redhat.com>
Date: Mon, 13 Feb 2012 18:08:51 +0000 (+0000)
Subject: PR ld/13621
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e1741a1ed122cb64e0d4fb6aa936574763de0d4c;p=binutils-gdb.git

PR ld/13621

bfd/
	* linker.c (fix_syms): Force symbols outside any section into
	bfd_abs_section_ptr.
ld/testsuite/
	* ld-elf/warn2.d: Expect ABS section for Foo.
	* ld-elf/zerosize1.d, zerosize1.s: New test.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 490120baa54..0a53fad026b 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2012-02-13  Richard Henderson  <rth@redhat.com>
+
+	PR ld/13621
+	* linker.c (fix_syms): Force symbols outside any section into
+	bfd_abs_section_ptr.
+
 2012-02-13  Alan Modra  <amodra@gmail.com>
 
 	* elf32-m68hc1x.c (elf32_m68hc11_merge_symbol_attribute): New function.
diff --git a/bfd/linker.c b/bfd/linker.c
index c51c5df93b4..04044743f26 100644
--- a/bfd/linker.c
+++ b/bfd/linker.c
@@ -3208,6 +3208,12 @@ fix_syms (struct bfd_link_hash_entry *h, void *data)
 		op = op1;
 	    }
 
+	  /* Refuse to choose a section for which we are out of bounds.  */
+	  /* ??? This may make most of the above moot.  */
+	  if (h->u.def.value < op->vma
+	      || h->u.def.value > op->vma + op->size)
+	    op = bfd_abs_section_ptr;
+
 	  h->u.def.value -= op->vma;
 	  h->u.def.section = op;
 	}
diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index f1c82a066a6..f118153e3f3 100644
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2012-02-13  Richard Henderson  <rth@redhat.com>
+
+	PR ld/13621
+	* ld-elf/warn2.d: Expect ABS section for Foo.
+	* ld-elf/zerosize1.d, zerosize1.s: New test.
+
 2012-02-13  Alan Modra  <amodra@gmail.com>
 
 	* ld-m68hc11/m68hc11.exp: Run for m68hc11-*-* and m68hc12-*-*.
diff --git a/ld/testsuite/ld-elf/warn2.d b/ld/testsuite/ld-elf/warn2.d
index 95b7ef49998..a9c05f95411 100644
--- a/ld/testsuite/ld-elf/warn2.d
+++ b/ld/testsuite/ld-elf/warn2.d
@@ -13,5 +13,5 @@
 # construct and that the symbol still appears as expected.
 
 #...
- +[0-9]+: +[0-9a-f]+ +20 +OBJECT +GLOBAL +DEFAULT +[1-9] Foo
+ +[0-9]+: +[0-9a-f]+ +20 +OBJECT +GLOBAL +DEFAULT +ABS Foo
 #pass