commit 772db5e6d485707dc87d419dbb2c56a184b2128f
author Dimitry Ivanov <dimitry@google.com> Fri Apr 21 16:44:48 2017 -0700
committer Dimitry Ivanov <dimitry@google.com> Fri Apr 21 17:59:40 2017 -0700
tree 9e4f42b8688e3194168272aa379003786e37ce0e
parent 59c1e60f8365faaf4e1a71d8dcc222c88e2dce65

loader: fix leak of child-to-parent links on dlclose()

Use children instead of parents in is_recursive() - this
allows us to remove code adding links from child to
parent when loading a library.

Bug: 36104177
Test: /data/nativetest/bionic-unit-tests/bionic-unit-tests32 --gtest_filter=dl*
Test: manual - dlopen/dlclose libandroid.so 10 times and see how many blocks are
      used after each dlclose() - make sure they are constant.

Change-Id: I776d47de92101c6ba47f400205ccbfe8844a7cb8

linker/linker.cpp

diff --git a/linker/linker.cpp b/linker/linker.cpp
index 54867dc..d200c22 100644
--- a/linker/linker.cpp
+++ b/linker/linker.cpp
@@ -884,12 +884,11 @@
   }
 
   if (si == parent) {
-    DL_ERR("recursive link to \"%s\"", si->name);
     return true;
   }
 
-  return !parent->get_parents().visit([&](soinfo* grandparent) {
-    return !is_recursive(si, grandparent);
+  return !si->get_children().visit([&](soinfo* child) {
+    return !is_recursive(child, parent);
   });
 }
 
@@ -928,6 +927,7 @@
     soinfo* needed_by = task->get_needed_by();
 
     if (is_recursive(si, needed_by)) {
+      DL_ERR("recursive link to \"%s\"", si->name);
       return false;
     }
 
@@ -1706,7 +1706,6 @@
 
 void soinfo::add_child(soinfo* child) {
   if (has_min_version(0)) {
-    child->parents.push_back(this);
     this->children.push_back(child);
   }
 }