gold/

	* layout.cc (Layout::make_output_section): Mark .tdata section
	as RELRO.
	* testsuite/relro_test.cc: Add a TLS variable.

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 7fffe40..80ce49e 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,9 @@
+2012-05-14  Cary Coutant  <ccoutant@google.com>
+
+       * layout.cc (Layout::make_output_section): Mark .tdata section
+       as RELRO.
+       * testsuite/relro_test.cc: Add a TLS variable.
+
 2012-05-10  H.J. Lu  <hongjiu.lu@intel.com>
 
        PR gold/14091

diff --git a/gold/layout.cc b/gold/layout.cc
index b58f9d2..0ac0fbf 100644 (file)
--- a/gold/layout.cc
+++ b/gold/layout.cc
@@ -1430,7 +1430,9 @@ Layout::make_output_section(const char* name, elfcpp::Elf_Word type,
     {
       if (type == elfcpp::SHT_PROGBITS)
        {
-         if (strcmp(name, ".data.rel.ro") == 0)
+         if ((flags & elfcpp::SHF_TLS) != 0)
+           is_relro = true;
+         else if (strcmp(name, ".data.rel.ro") == 0)
            is_relro = true;
          else if (strcmp(name, ".data.rel.ro.local") == 0)
            {

diff --git a/gold/testsuite/relro_test.cc b/gold/testsuite/relro_test.cc
index d741022..795ad39 100644 (file)
--- a/gold/testsuite/relro_test.cc
+++ b/gold/testsuite/relro_test.cc
@@ -45,6 +45,9 @@ int* const p1 __attribute__ ((aligned(64))) = &i1;
 // P2 is a local relro variable.
 int* const p2 __attribute__ ((aligned(64))) = &i2;
 
+// Add a TLS variable to make sure -z relro works correctly with TLS.
+__thread int i3 = 1;
+
 // Test symbol addresses.
 
 bool
@@ -76,6 +79,7 @@ t1()
   assert(i1page != p2page);
   assert(i2page != p1page);
   assert(i2page != p2page);
+  assert(i3 == 1);
 
   return true;
 }