commit 3c005d60194aadd92584258118c06782c9deab11
author Bjorn Andersson <bjorn.andersson@sonymobile.com> Thu May 22 17:53:01 2014 -0700
committer Dmitriy Ivanov <dimitry@google.com> Thu May 22 19:08:14 2014 -0700
tree 2b86b3c85f0f518237e7d92068c702fd5d34afe0
parent 0b10ffb40f3b426a7b5e734f90c41f56351729a0

Do not add arcs to the soinfo graph on dlopen

While introducing RTLD_NOLOAD in b648a8a5 a change in how soinfo structs are
connected was also introduced. When calling dlopen the library that is loaded
is added as a child to the soinfo from which the caller comes - i.e. building a
dependency graph.

Unfortunately this shows issues upon unloading, such as blowing the stack if
there are loops in the graph.

This change reverts that part of b648a8a5, keeping everything as a child of the
root soinfo.

Change-Id: I604ee9210613b19f693a568158e69707620a95db
Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>

linker/linker.cpp

diff --git a/linker/linker.cpp b/linker/linker.cpp
index 9f458b4..151d4ce 100644
--- a/linker/linker.cpp
+++ b/linker/linker.cpp
@@ -841,9 +841,6 @@
   soinfo* si = find_library(name, flags, extinfo);
   if (si != NULL) {
     si->CallConstructors();
-    if (caller != NULL) {
-      caller->add_child(si);
-    }
   }
   protect_data(PROT_READ);
   return si;