Skip missing symbol version section check for executable
authorH.J. Lu <hjl.tools@gmail.com>
Sat, 25 Jul 2015 14:56:18 +0000 (07:56 -0700)
committerH.J. Lu <hjl.tools@gmail.com>
Sat, 25 Jul 2015 15:07:33 +0000 (08:07 -0700)
Missing symbol version section is a run-time problem only if it will
be referenced dynamically at run-time.  We should skip the check for
locally defined symbol, which isn't referenced by shared library, when
linking executable.

bfd/

PR ld/18718
* elflink.c (elf_link_output_extsym): Check symbol version
section check only if not linking executable, the symbol is
referenced by shared library or not locally defined.

ld/testsuite/

PR ld/18718
* ld-elf/pr18718.c: New file.
* ld-elf/shared.exp: Run tests for PR ld/18718.

bfd/elflink.c
ld/testsuite/ld-elf/pr18718.c [new file with mode: 0644]
ld/testsuite/ld-elf/shared.exp

index b741e7e..ccb7ba2 100644 (file)
@@ -9217,8 +9217,12 @@ elf_link_output_extsym (struct bfd_hash_entry *bh, void *data)
 
       /* Since there is no version information in the dynamic string,
         if there is no version info in symbol version section, we will
-        have a run-time problem.  */
-      if (h->verinfo.verdef == NULL)
+        have a run-time problem if not linking executable, referenced
+        by shared library, or not locally defined.  */
+      if (h->verinfo.verdef == NULL
+         && (!flinfo->info->executable
+             || h->ref_dynamic
+             || !h->def_regular))
        {
          char *p = strrchr (h->root.root.string, ELF_VER_CHR);
 
diff --git a/ld/testsuite/ld-elf/pr18718.c b/ld/testsuite/ld-elf/pr18718.c
new file mode 100644 (file)
index 0000000..de628ef
--- /dev/null
@@ -0,0 +1,26 @@
+#include <stdio.h>
+#include <bfd_stdint.h>
+
+extern void foo (void);
+
+void
+new_foo (void)
+{
+}
+
+__asm__(".symver new_foo, foo@@VERS_2.0");
+
+__attribute__ ((noinline, noclone))
+int
+bar (void)
+{
+  return (intptr_t) &foo == 0x12345678 ? 1 : 0;
+}
+
+int
+main(void)
+{
+  bar ();
+  printf("PASS\n");
+  return 0;
+}
index bcbe8e2..92188f9 100644 (file)
@@ -529,5 +529,50 @@ if { [istarget *-*-linux*]
            "pr2404.out" \
            "-fPIE" \
        ] \
+       [list \
+           "Run pr18718" \
+           "" \
+           "" \
+           { pr18718.c } \
+           "pr18718" \
+           "pass.out" \
+           "-I../bfd" \
+       ] \
+       [list \
+           "Run pr18718 with PIE (1)" \
+           "-pie" \
+           "" \
+           { pr18718.c } \
+           "pr18718pie1" \
+           "pass.out" \
+           "-fPIE -I../bfd" \
+       ] \
+       [list \
+           "Run pr18718 with PIE (2)" \
+           "" \
+           "" \
+           { pr18718.c } \
+           "pr18718pie2" \
+           "pass.out" \
+           "-fPIE -I../bfd" \
+       ] \
+       [list \
+           "Run pr18718 with PIC (1)" \
+           "" \
+           "" \
+           { pr18718.c } \
+           "pr18718pic1" \
+           "pass.out" \
+           "-fPIC -I../bfd" \
+       ] \
+       [list \
+           "Run pr18718 with PIC (2)" \
+           "-pie" \
+           "" \
+           { pr18718.c } \
+           "pr18718pic2" \
+           "pass.out" \
+           "-fPIC -I../bfd" \
+       ] \
     ]
 }