Merge "Return has_DT_SYMBOLIC flag."

linker/linker.cpp

@@ -485,15 +485,35 @@ static ElfW(Sym)* soinfo_do_lookup(soinfo* si, const char* name, soinfo** lsi) {
   unsigned elf_hash = elfhash(name);
   ElfW(Sym)* s = nullptr;
 
-  if (somain != nullptr) {
+  /* "This element's presence in a shared object library alters the dynamic linker's
+   * symbol resolution algorithm for references within the library. Instead of starting
+   * a symbol search with the executable file, the dynamic linker starts from the shared
+   * object itself. If the shared object fails to supply the referenced symbol, the
+   * dynamic linker then searches the executable file and other shared objects as usual."
+   *
+   * http://www.sco.com/developers/gabi/2012-12-31/ch5.dynamic.html
+   *
+   * Note that this is unlikely since static linker avoids generating
+   * relocations for -Bsymbolic linked dynamic executables.
+   */
+  if (si->has_DT_SYMBOLIC) {
+    DEBUG("%s: looking up %s in local scope (DT_SYMBOLIC)", si->name, name);
+    s = soinfo_elf_lookup(si, elf_hash, name);
+    if (s != nullptr) {
+      *lsi = si;
+    }
+  }
+
+  if (s == nullptr && somain != nullptr) {
+    // 1. Look for it in the main executable unless we already did.
+    if (si != somain || !si->has_DT_SYMBOLIC) {
       DEBUG("%s: looking up %s in executable %s",
             si->name, name, somain->name);
-
-    // 1. Look for it in the main executable
       s = soinfo_elf_lookup(somain, elf_hash, name);
       if (s != nullptr) {
         *lsi = somain;
       }
+    }
 
     // 2. Look for it in the ld_preloads
     if (s == nullptr) {
@@ -505,6 +525,7 @@ static ElfW(Sym)* soinfo_do_lookup(soinfo* si, const char* name, soinfo** lsi) {
         }
       }
     }
+  }
 
   /* Look for symbols in the local scope (the object who is
    * searching). This happens with C++ templates on x86 for some
@@ -516,13 +537,13 @@ static ElfW(Sym)* soinfo_do_lookup(soinfo* si, const char* name, soinfo** lsi) {
    * and some the first non-weak definition.   This is system dependent.
    * Here we return the first definition found for simplicity.  */
 
-    if (s == nullptr) {
+  if (s == nullptr && !si->has_DT_SYMBOLIC) {
+    DEBUG("%s: looking up %s in local scope", si->name, name);
     s = soinfo_elf_lookup(si, elf_hash, name);
     if (s != nullptr) {
       *lsi = si;
     }
   }
-  }
 
   if (s == nullptr) {
     si->get_children().visit([&](soinfo* child) {
@@ -2009,7 +2030,7 @@ bool soinfo::PrelinkImage() {
         break;
 #endif
       case DT_SYMBOLIC:
-        // ignored
+        has_DT_SYMBOLIC = true;
         break;
       case DT_NEEDED:
         ++needed_count;
@@ -2023,6 +2044,9 @@ bool soinfo::PrelinkImage() {
           has_text_relocations = true;
 #endif
         }
+        if (d->d_un.d_val & DF_SYMBOLIC) {
+          has_DT_SYMBOLIC = true;
+        }
         break;
 #if defined(__mips__)
       case DT_STRSZ:

linker/linker.h

@@ -197,7 +197,7 @@ struct soinfo {
 #if !defined(__LP64__)
   bool has_text_relocations;
 #endif
-  bool unused4; // DO NOT USE, maintained for compatibility
+  bool has_DT_SYMBOLIC;
 
   soinfo(const char* name, const struct stat* file_stat, int rtld_flags);