diff options
author | Ian Coolidge <icoolidge@google.com> | 2015-04-20 18:04:07 -0700 |
---|---|---|
committer | Cary Coutant <ccoutant@gmail.com> | 2015-04-20 18:07:49 -0700 |
commit | 4b889c3013d7d45e2775781c3904b657419db2fa (patch) | |
tree | 1b525037f020ab96f55b8a376f4c88bb208fe9bc | |
parent | efdacec57ccde20bb53045e2930ef02e7e49416b (diff) | |
download | gdb-4b889c3013d7d45e2775781c3904b657419db2fa.zip gdb-4b889c3013d7d45e2775781c3904b657419db2fa.tar.gz gdb-4b889c3013d7d45e2775781c3904b657419db2fa.tar.bz2 |
Ensure that dynamically loaded libraries won't use separate copies of GNU_UNIQUE symbols.
gold/
* symtab.cc (Symbol::should_add_dynsym_entry): Return true for
GNU_UNIQUE.
-rw-r--r-- | gold/ChangeLog | 5 | ||||
-rw-r--r-- | gold/symtab.cc | 6 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index 789ba66..e736d33 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,8 @@ +2015-04-20 Ian Coolidge <icoolidge@google.com> + + * symtab.cc (Symbol::should_add_dynsym_entry): Return true for + GNU_UNIQUE. + 2015-04-17 Rafael Ávila de Espíndola <rafael.espindola@gmail.com> * gc.cc (Garbage_collection::do_transitive_closure): Use back and diff --git a/gold/symtab.cc b/gold/symtab.cc index d4f40c8..c197221 100644 --- a/gold/symtab.cc +++ b/gold/symtab.cc @@ -419,9 +419,13 @@ Symbol::should_add_dynsym_entry(Symbol_table* symtab) const } // If exporting all symbols or building a shared library, + // or the symbol should be globally unique (GNU_UNIQUE), // and the symbol is defined in a regular object and is // externally visible, we need to add it. - if ((parameters->options().export_dynamic() || parameters->options().shared()) + if ((parameters->options().export_dynamic() + || parameters->options().shared() + || (parameters->options().gnu_unique() + && this->binding() == elfcpp::STB_GNU_UNIQUE)) && !this->is_from_dynobj() && !this->is_undefined() && this->is_externally_visible()) |