From 1c16dfa5fdc5576b3129d29775da52302b72c500 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Wed, 19 Jan 2005 16:15:11 +0000 Subject: [PATCH] 2005-01-19 H.J. Lu PR 679 * elflink.c (_bfd_elf_dynamic_symbol_p): Only protected non-function symbols are local. (_bfd_elf_symbol_refs_local_p): Likewise. --- bfd/ChangeLog | 7 +++++++ bfd/elflink.c | 6 +++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 97a61bdfdf..e2abbe8a97 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2005-01-19 H.J. Lu + + PR 679 + * elflink.c (_bfd_elf_dynamic_symbol_p): Only protected + non-function symbols are local. + (_bfd_elf_symbol_refs_local_p): Likewise. + 2005-01-18 Alan Modra * bfd.c (bfd_link_just_syms): Add abfd param. diff --git a/bfd/elflink.c b/bfd/elflink.c index a973cd59a3..7a02d3415f 100644 --- a/bfd/elflink.c +++ b/bfd/elflink.c @@ -2479,7 +2479,7 @@ _bfd_elf_dynamic_symbol_p (struct elf_link_hash_entry *h, /* Proper resolution for function pointer equality may require that these symbols perhaps be resolved dynamically, even though we should be resolving them to the current module. */ - if (!ignore_protected) + if (!ignore_protected || h->type != STT_FUNC) binding_stays_local_p = TRUE; break; @@ -2542,6 +2542,10 @@ _bfd_elf_symbol_refs_local_p (struct elf_link_hash_entry *h, if (ELF_ST_VISIBILITY (h->other) != STV_PROTECTED) return TRUE; + /* STV_PROTECTED non-function symbols are local. */ + if (h->type != STT_FUNC) + return TRUE; + /* Function pointer equality tests may require that STV_PROTECTED symbols be treated as dynamic symbols, even when we know that the dynamic linker will resolve them locally. */