From: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date: Mon, 27 Jul 2015 10:45:27 +0000 (+0100)
Subject: [ARM] Fix extern protected data handling
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=b68a20d6675f1360ea4db50a9835c073675b9889;p=deliverable%2Fbinutils-gdb.git

[ARM] Fix extern protected data handling

Emit *GLOB_DAT instead of *RELATIVE relocs for protected data
in shared objects.

This is needed for the fix of glibc bug 17711:
https://sourceware.org/bugzilla/show_bug.cgi?id=17711

bfd:
	PR ld/18705
	* elf32-arm.c (elf_backend_extern_protected_data): Define.

ld/testsuite:
	PR ld/18705
	* ld-arm/protected-data.d: New.
	* ld-arm/protected-data.s: New.
	* ld-arm/arm-elf.exp: Add new test.
---

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 1d1a1d28c1..553b8c92d6 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2015-07-27  Szabolcs Nagy  <szabolcs.nagy@arm.com>
+
+	PR ld/18705
+	* elf32-arm.c (elf_backend_extern_protected_data): Define.
+
 2015-07-25  H.J. Lu  <hongjiu.lu@intel.com>
 
 	PR ld/18718
diff --git a/bfd/elf32-arm.c b/bfd/elf32-arm.c
index 367dff38c0..ff69728729 100644
--- a/bfd/elf32-arm.c
+++ b/bfd/elf32-arm.c
@@ -16236,6 +16236,7 @@ elf32_arm_get_synthetic_symtab (bfd *abfd,
 #define elf_backend_default_use_rela_p 0
 
 #define elf_backend_got_header_size	12
+#define elf_backend_extern_protected_data 1
 
 #undef  elf_backend_obj_attrs_vendor
 #define elf_backend_obj_attrs_vendor		"aeabi"
diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog
index de0ad1b3f5..734fe69cdd 100644
--- a/ld/testsuite/ChangeLog
+++ b/ld/testsuite/ChangeLog
@@ -1,3 +1,10 @@
+2015-07-27  Szabolcs Nagy  <szabolcs.nagy@arm.com>
+
+	PR ld/18705
+	* ld-arm/protected-data.d: New.
+	* ld-arm/protected-data.s: New.
+	* ld-arm/arm-elf.exp: Add new test.
+
 2015-07-26  H.J. Lu  <hongjiu.lu@intel.com>
 
 	PR ld/18718
diff --git a/ld/testsuite/ld-arm/arm-elf.exp b/ld/testsuite/ld-arm/arm-elf.exp
index 28b41ca4a6..882f5cb99e 100644
--- a/ld/testsuite/ld-arm/arm-elf.exp
+++ b/ld/testsuite/ld-arm/arm-elf.exp
@@ -904,3 +904,4 @@ if { ![istarget "arm*-*-nacl*"] } {
 }
 run_dump_test "unresolved-2"
 run_dump_test "gc-hidden-1"
+run_dump_test "protected-data"
diff --git a/ld/testsuite/ld-arm/protected-data.d b/ld/testsuite/ld-arm/protected-data.d
new file mode 100644
index 0000000000..51c51c6b95
--- /dev/null
+++ b/ld/testsuite/ld-arm/protected-data.d
@@ -0,0 +1,4 @@
+#ld: -shared
+#readelf: -r -W
+#...
+.* R_ARM_GLOB_DAT .* var.*
diff --git a/ld/testsuite/ld-arm/protected-data.s b/ld/testsuite/ld-arm/protected-data.s
new file mode 100644
index 0000000000..8c8072d334
--- /dev/null
+++ b/ld/testsuite/ld-arm/protected-data.s
@@ -0,0 +1,29 @@
+	.syntax unified
+
+	.data
+	.protected	var
+	.global	var
+	.align	2
+	.type	var, %object
+	.size	var, 4
+var:
+	.word	1
+
+	.text
+	.align	2
+	.global	getaddr
+	.thumb
+	.thumb_func
+	.type	getaddr, %function
+getaddr:
+	ldr	r3, 2f
+	ldr	r2, 2f+4
+1:
+	add	r3, pc
+	ldr	r0, [r3, r2]
+	bx	lr
+	.align	2
+2:
+	.word	_GLOBAL_OFFSET_TABLE_-(1b+4)
+	.word	var(GOT)
+	.size	getaddr, .-getaddr