Check if __start/__stop symbols are referenced by shared objects

Define  __start/__stop symbols if they are referenced by shared objects,
not if they are also defined in shared objects.

bfd/

	PR ld/21964
	* elflink.c (bfd_elf_define_start_stop): Check if __start and
	__stop symbols are referenced by shared objects.

ld/

	PR ld/21964
	* testsuite/ld-elf/pr21964-4.c: New file.
	* testsuite/ld-elf/shared.exp: Run pr21964-4 test on Linux.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index fb93985b71d82778de5074c6716de5b6aeaa92b4..2a0330c8a8f7abc7310a4cddcbcde0b02f777941 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,9 @@
+2018-01-31  H.J. Lu  <hongjiu.lu@intel.com>
+
+       PR ld/21964
+       * elflink.c (bfd_elf_define_start_stop): Check if __start and
+       __stop symbols are referenced by shared objects.
+
 2018-01-30  Alan Modra  <amodra@gmail.com>
 
        PR 22758

diff --git a/bfd/elflink.c b/bfd/elflink.c
index f1ec880f987b634934b81a8bef57e46767b631d8..3787c85e49d74073ad3adf67b0cacbbabb516e9c 100644 (file)
--- a/bfd/elflink.c
+++ b/bfd/elflink.c
@@ -14338,7 +14338,7 @@ bfd_elf_define_start_stop (struct bfd_link_info *info,
   if (h != NULL
       && (h->root.type == bfd_link_hash_undefined
          || h->root.type == bfd_link_hash_undefweak
-         || ((h->ref_regular || h->def_dynamic) && !h->def_regular)))
+         || ((h->ref_regular || h->ref_dynamic) && !h->def_regular)))
     {
       h->root.type = bfd_link_hash_defined;
       h->root.u.def.section = sec;
@@ -14358,7 +14358,7 @@ bfd_elf_define_start_stop (struct bfd_link_info *info,
        {
          if (ELF_ST_VISIBILITY (h->other) == STV_DEFAULT)
            h->other = (h->other & ~ELF_ST_VISIBILITY (-1)) | STV_PROTECTED;
-         if (h->ref_dynamic || h->def_dynamic)
+         if (h->ref_dynamic)
            bfd_elf_link_record_dynamic_symbol (info, h);
        }
       return &h->root;

diff --git a/ld/ChangeLog b/ld/ChangeLog
index f29d51ada00e5b3e47e6a51b9eb763324b48b00c..67e29b9653a6686af6262fe3aa4046580a249dbe 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,9 @@
+2018-01-31  H.J. Lu  <hongjiu.lu@intel.com>
+
+       PR ld/21964
+       * testsuite/ld-elf/pr21964-4.c: New file.
+       * testsuite/ld-elf/shared.exp: Run pr21964-4 test on Linux.
+
 2018-01-30  Alan Modra  <amodra@gmail.com>
 
        * testsuite/ld-gc/stop.d,

diff --git a/ld/testsuite/ld-elf/pr21964-4.c b/ld/testsuite/ld-elf/pr21964-4.c
new file mode 100644 (file)
index 0000000..d50c148
--- /dev/null
+++ b/ld/testsuite/ld-elf/pr21964-4.c
@@ -0,0 +1,22 @@
+#define _GNU_SOURCE
+#include <dlfcn.h>
+#include <stdio.h>
+
+extern int __start___verbose[];
+int bar (void)
+{
+  static int my_var __attribute__ ((section("__verbose"), used)) = 6;
+  int *ptr;
+  ptr = (int*) dlsym (RTLD_DEFAULT, "__start___verbose");
+  if (ptr != NULL || __start___verbose[0] != 6)
+    return -1;
+  return 0;
+}
+
+int
+main ()
+{
+  if (bar () == 0)
+    printf ("PASS\n");
+  return 0;
+}

diff --git a/ld/testsuite/ld-elf/shared.exp b/ld/testsuite/ld-elf/shared.exp
index 8223d3df014710a073cbed356c43020ec19014a2..833a4280551c29706c53f68f113452553bf4f878 100644 (file)
--- a/ld/testsuite/ld-elf/shared.exp
+++ b/ld/testsuite/ld-elf/shared.exp
@@ -1099,6 +1099,18 @@ if { [istarget *-*-linux*]
            "pr22393-2-static" \
            "pass.out" \
        ] \
+       [list \
+           "Run pr21964-4" \
+           "" \
+           "" \
+           {pr21964-4.c} \
+           "pr21964-4" \
+           "pass.out" \
+           "" \
+           "" \
+           "" \
+           "-ldl" \
+       ] \
     ]
 }