diff options
author | Andreas Jaeger <aj@suse.de> | 2001-02-20 08:02:07 +0000 |
---|---|---|
committer | Andreas Jaeger <aj@suse.de> | 2001-02-20 08:02:07 +0000 |
commit | 8b27f3d56aea27263a35e0b54de27f4f464548f5 (patch) | |
tree | 4bb7e2c02b30d453b1d318b923f75772b789cd19 | |
parent | 73c5ee7ee36d536a0e1daeda3343943f8e71a4d8 (diff) | |
download | gdb-8b27f3d56aea27263a35e0b54de27f4f464548f5.tar.gz |
2001-02-20 Andreas Jaeger <aj@suse.de>
* elf64-x86-64.c (elf64_x86_64_finish_dynamic_symbol): Don't make
PLT entries that could serve as a definition for a weak symbol.
-rw-r--r-- | bfd/ChangeLog | 7 | ||||
-rw-r--r-- | bfd/elf64-x86-64.c | 7 |
2 files changed, 13 insertions, 1 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 74b92a105a9..b45c1ec5429 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2001-02-20 Andreas Jaeger <aj@suse.de> + + * elf64-x86-64.c (elf64_x86_64_finish_dynamic_symbol): Don't make + PLT entries that could serve as a definition for a weak symbol. + 2001-02-18 David O'Brien <obrien@FreeBSD> * configure.in: Recognize FreeBSD/arm, FreeBSD/PowerPC, and treat @@ -14,7 +19,7 @@ checking. (pe_bfd_object_p): Rewrite with external_PEI_DOS_hdr and external_PEI_IMAGE_hdr. - + 2001-02-14 Bo Thorsen <bo@suse.de> * elf64-x86-64.c: Small formatting fixes and rearrangements of code. diff --git a/bfd/elf64-x86-64.c b/bfd/elf64-x86-64.c index 9e10c917f0d..3845cc183ab 100644 --- a/bfd/elf64-x86-64.c +++ b/bfd/elf64-x86-64.c @@ -1656,6 +1656,13 @@ elf64_x86_64_finish_dynamic_symbol (output_bfd, info, h, sym) /* Mark the symbol as undefined, rather than as defined in the .plt section. Leave the value alone. */ sym->st_shndx = SHN_UNDEF; + /* If the symbol is weak, we do need to clear the value. + Otherwise, the PLT entry would provide a definition for + the symbol even if the symbol wasn't defined anywhere, + and so the symbol would never be NULL. */ + if ((h->elf_link_hash_flags & ELF_LINK_HASH_REF_REGULAR_NONWEAK) + == 0) + sym->st_value = 0; } } |