[Bug ld/18720] No symbol version section for versioned symbol address@hidden'

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=18720

--- Comment #7 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> ---
The binutils-2_28-branch branch has been updated by Alan Modra
<address@hidden>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=de1f27ad53e4bcc3aa47209cdb5f32eb1fc41a76

commit de1f27ad53e4bcc3aa47209cdb5f32eb1fc41a76
Author: Alan Modra <address@hidden>
Date:   Thu Dec 29 22:49:41 2016 +1030

    link_hash_copy_indirect and symbol flags

    A while ago HJ fixed PR ld/18720 with commit 6e33951ed, which, among
    other things, modified _bfd_elf_link_hash_copy_indirect to not copy
    ref_dynamic, ref_regular, ref_regular_nonweak, non_got_ref, needs_plt
    and pointer_equality_needed when setting up an indirect non-versioned
    symbol pointing to a non-default versioned symbol.  I didn't notice at
    the time, but the pr18720 testcase fails on hppa-linux with
    "internal error, aborting at binutils-gdb-2.28/bfd/elf32-hppa.c:3933
    in elf32_hppa_relocate_section".

    Now hppa-linux creates entries in the plt even for local functions, if
    they are referenced using plabel (function pointer) relocations.   So
    needs_plt is set for foo when processing pr18720a.o.  When the aliases
    in pr28720b.o are processed, we get an indirection from foo to
    address@hidden, but don't copy needs_plt.  Since address@hidden is the 
"real" symbol
    that is used after that point, no plt entry is made for foo and we
    bomb when relocating the plabel.

    As shown by the hppa-linux scenario, needs_plt should be copied even
    for non-default versioned symbols.  I believe all of the others ought
    to be copied too, with the exception of ref_dynamic.  Not copying
    ref_dynamic is right because if a shared lib references "foo" it
    should not be satisfied by any non-default version "address@hidden".

        * elflink.c (_bfd_elf_link_hash_copy_indirect): Only omit
        copying one flag, ref_dynamic, when versioned_hidden.
        * elf64-ppc.c (ppc64_elf_copy_indirect_symbol): Likewise.
        * elf32-hppa.c (elf32_hppa_copy_indirect_symbol): Use same
        logic for copying weakdef flags.  Copy plabel flag and merge
        tls_type.
        * elf32-i386.c (elf_i386_copy_indirect_symbol): Use same logic
        for copying weakdef flags.
        * elf32-ppc.c (ppc_elf_copy_indirect_symbol): Likewise.
        * elf32-s390.c (elf_s390_copy_indirect_symbol): Likewise.
        * elf32-sh.c (sh_elf_copy_indirect_symbol): Likewise.
        * elf64-s390.c (elf_s390_copy_indirect_symbol): Likewise.
        * elfnn-ia64.c (elfNN_ia64_hash_copy_indirect): Likewise.
        * elf64-x86-64.c (elf_x86_64_copy_indirect_symbol): Likewise.
        Simplify.

-- 
You are receiving this mail because:
You are on the CC list for the bug.