Add dlfcn_test to glibc test suite.

Change-Id: I955e4f7dfcc23ea5c767f967b3532dc31663b876

tests/Android.mk

@@ -310,6 +310,7 @@ ifeq ($(HOST_OS)-$(HOST_ARCH),$(filter $(HOST_OS)-$(HOST_ARCH),linux-x86 linux-x
 
 bionic-unit-tests-glibc_src_files := \
     atexit_test.cpp \
+    dlfcn_test.cpp \
 
 bionic-unit-tests-glibc_whole_static_libraries := \
     libBionicStandardTests \
@@ -317,8 +318,12 @@ bionic-unit-tests-glibc_whole_static_libraries := \
 bionic-unit-tests-glibc_ldlibs := \
     -lrt -ldl \
 
+bionic-unit-tests-glibc_c_includes := \
+    bionic/libc \
+
 bionic-unit-tests-glibc_cflags := $(test_cflags)
 bionic-unit-tests-glibc_cppflags := $(test_cppflags)
+bionic-unit-tests-glibc_ldflags := -Wl,--export-dynamic
 
 module := bionic-unit-tests-glibc
 module_tag := optional

tests/dlfcn_test.cpp

@@ -188,14 +188,14 @@ TEST(dlfcn, dlopen_check_order) {
   // get_answer2() is defined in (b, d)
   void* sym = dlsym(RTLD_DEFAULT, "dlopen_test_get_answer");
   ASSERT_TRUE(sym == nullptr);
-  void* handle = dlopen("libtest_check_order.so", RTLD_NOW);
+  void* handle = dlopen("libtest_check_order.so", RTLD_NOW | RTLD_GLOBAL);
   ASSERT_TRUE(handle != nullptr);
   typedef int (*fn_t) (void);
   fn_t fn, fn2;
   fn = reinterpret_cast<fn_t>(dlsym(RTLD_DEFAULT, "dlopen_test_get_answer"));
-  ASSERT_TRUE(fn != NULL);
+  ASSERT_TRUE(fn != NULL) << dlerror();
   fn2 = reinterpret_cast<fn_t>(dlsym(RTLD_DEFAULT, "dlopen_test_get_answer2"));
-  ASSERT_TRUE(fn2 != NULL);
+  ASSERT_TRUE(fn2 != NULL) << dlerror();
 
   ASSERT_EQ(42, fn());
   ASSERT_EQ(43, fn2());
@@ -207,6 +207,7 @@ TEST(dlfcn, dlopen_check_order) {
 // libtest_with_dependency_loop_a.so
 TEST(dlfcn, dlopen_check_loop) {
   void* handle = dlopen("libtest_with_dependency_loop.so", RTLD_NOW);
+#if defined(__BIONIC__)
   ASSERT_TRUE(handle == nullptr);
   ASSERT_STREQ("dlopen failed: recursive link to \"libtest_with_dependency_loop_a.so\"", dlerror());
   // This symbol should never be exposed
@@ -220,6 +221,10 @@ TEST(dlfcn, dlopen_check_loop) {
   handle = dlopen("libtest_with_dependency_loop.so", RTLD_NOW | RTLD_NOLOAD);
   ASSERT_TRUE(handle == nullptr);
   ASSERT_STREQ("dlopen failed: library \"libtest_with_dependency_loop.so\" wasn't loaded and RTLD_NOLOAD prevented it", dlerror());
+#else // glibc allows recursive links
+  ASSERT_TRUE(handle != nullptr);
+  dlclose(handle);
+#endif
 }
 
 TEST(dlfcn, dlopen_failure) {

tests/libs/Android.build.testlib.mk

@@ -0,0 +1,22 @@
+#
+# Copyright (C) 2014 The Android Open Source Project
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+build_target := SHARED_LIBRARY
+build_type := host
+include $(TEST_PATH)/Android.build.mk
+build_type := target
+include $(TEST_PATH)/Android.build.mk
+

tests/libs/Android.mk

@@ -30,9 +30,7 @@ no-elf-hash-table-library_ldflags := \
 
 module := no-elf-hash-table-library
 module_tag := optional
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 endif
 
 # -----------------------------------------------------------------------------
@@ -46,15 +44,13 @@ libdlext_test_ldflags := \
 
 module := libdlext_test
 module_tag := optional
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # create symlink to libdlext_test.so for symlink test
 # -----------------------------------------------------------------------------
 # Use = instead of := to defer the evaluation of $@
-$(LOCAL_INSTALLED_MODULE): PRIVATE_POST_INSTALL_CMD = \
+$(TARGET_OUT)/lib/libdlext_test.so: PRIVATE_POST_INSTALL_CMD = \
     $(hide) cd $(dir $@) && ln -sf $(notdir $@) libdlext_test_v2.so
 
 ifneq ($(TARGET_2ND_ARCH),)
@@ -63,6 +59,13 @@ $(TARGET_OUT)/lib64/libdlext_test.so: PRIVATE_POST_INSTALL_CMD = \
     $(hide) cd $(dir $@) && ln -sf $(notdir $@) libdlext_test_v2.so
 endif
 
+# host symlinks
+$(HOST_OUT)/lib64/libdlext_test.so: PRIVATE_POST_INSTALL_CMD = \
+    $(hide) cd $(dir $@) && ln -sf $(notdir $@) libdlext_test_v2.so
+
+$(HOST_OUT)/lib/libdlext_test.so: PRIVATE_POST_INSTALL_CMD = \
+    $(hide) cd $(dir $@) && ln -sf $(notdir $@) libdlext_test_v2.so
+
 # -----------------------------------------------------------------------------
 # Library used by dlext tests - without GNU RELRO program header
 # -----------------------------------------------------------------------------
@@ -98,9 +101,7 @@ libtest_simple_src_files := \
     dlopen_testlib_simple.cpp
 
 module := libtest_simple
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # Libraries used by dlfcn tests to verify correct load order:
@@ -111,9 +112,7 @@ libtest_check_order_2_right_src_files := \
 
 libtest_check_order_2_right_cflags := -D__ANSWER=42
 module := libtest_check_order_2_right
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order_a.so
@@ -123,9 +122,7 @@ libtest_check_order_a_src_files := \
 
 libtest_check_order_a_cflags := -D__ANSWER=1
 module := libtest_check_order_a
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order_b.so
@@ -135,9 +132,7 @@ libtest_check_order_b_src_files := \
 
 libtest_check_order_b_cflags := -D__ANSWER=2 -D__ANSWER2=43
 module := libtest_check_order_b
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order_c.so
@@ -147,9 +142,7 @@ libtest_check_order_3_c_src_files := \
 
 libtest_check_order_3_c_cflags := -D__ANSWER=3
 module := libtest_check_order_3_c
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order_d.so
@@ -160,9 +153,7 @@ libtest_check_order_d_src_files := \
 libtest_check_order_d_shared_libraries := libtest_check_order_b
 libtest_check_order_d_cflags := -D__ANSWER=4 -D__ANSWER2=4
 module := libtest_check_order_d
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order_left.so
@@ -173,9 +164,7 @@ libtest_check_order_1_left_src_files := \
 libtest_check_order_1_left_shared_libraries := libtest_check_order_a libtest_check_order_b
 
 module := libtest_check_order_1_left
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_check_order.so
@@ -187,9 +176,7 @@ libtest_check_order_shared_libraries := libtest_check_order_1_left \
   libtest_check_order_2_right libtest_check_order_3_c
 
 module := libtest_check_order
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # Library with dependency loop used by dlfcn tests
@@ -202,9 +189,7 @@ libtest_with_dependency_loop_shared_libraries := \
     libtest_with_dependency_loop_a
 
 module := libtest_with_dependency_loop
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_with_dependency_loop_a.so
@@ -215,9 +200,7 @@ libtest_with_dependency_loop_a_shared_libraries := \
     libtest_with_dependency_loop_b_tmp
 
 module := libtest_with_dependency_loop_a
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_with_dependency_loop_b.so
@@ -228,9 +211,7 @@ libtest_with_dependency_loop_b_tmp_src_files := dlopen_testlib_invalid.cpp
 libtest_with_dependency_loop_b_tmp_ldflags := -Wl,-soname=libtest_with_dependency_loop_b.so
 
 module := libtest_with_dependency_loop_b_tmp
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_with_dependency_loop_b.so
@@ -239,9 +220,7 @@ libtest_with_dependency_loop_b_src_files := dlopen_testlib_invalid.cpp
 libtest_with_dependency_loop_b_shared_libraries := libtest_with_dependency_loop_c
 
 module := libtest_with_dependency_loop_b
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_with_dependency_loop_c.so
@@ -252,9 +231,7 @@ libtest_with_dependency_loop_c_shared_libraries := \
     libtest_with_dependency_loop_a
 
 module := libtest_with_dependency_loop_c
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # libtest_relo_check_dt_needed_order.so
@@ -269,15 +246,13 @@ libtest_relo_check_dt_needed_order_shared_libraries := \
 libtest_relo_check_dt_needed_order_src_files := dlopen_testlib_relo_check_dt_needed_order.cpp
 libtest_relo_check_dt_needed_order_1_src_files := dlopen_testlib_relo_check_dt_needed_order_1.cpp
 libtest_relo_check_dt_needed_order_2_src_files := dlopen_testlib_relo_check_dt_needed_order_2.cpp
-build_type := target
-build_target := SHARED_LIBRARY
 
 module := libtest_relo_check_dt_needed_order
-include $(TEST_PATH)/Android.build.mk
+include $(LOCAL_PATH)/Android.build.testlib.mk
 module := libtest_relo_check_dt_needed_order_1
-include $(TEST_PATH)/Android.build.mk
+include $(LOCAL_PATH)/Android.build.testlib.mk
 module := libtest_relo_check_dt_needed_order_2
-include $(TEST_PATH)/Android.build.mk
+include $(LOCAL_PATH)/Android.build.testlib.mk
 
 # -----------------------------------------------------------------------------
 # Library with dependency used by dlfcn tests
@@ -288,22 +263,22 @@ libtest_with_dependency_src_files := \
 libtest_with_dependency_shared_libraries := libdlext_test
 
 module := libtest_with_dependency
-build_type := target
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_target := SHARED_LIBRARY
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # Library used by ifunc tests
 # -----------------------------------------------------------------------------
+libtest_ifunc_src_files := \
+    dlopen_testlib_ifunc.c
+
+libtest_ifunc_clang_host := false
+module := libtest_ifunc
+build_target := SHARED_LIBRARY
+
+build_type := host
+include $(TEST_PATH)/Android.build.mk
+
 ifeq ($(TARGET_ARCH),$(filter $(TARGET_ARCH),arm64 x86 x86_64))
-    libtest_ifunc_src_files := \
-        dlopen_testlib_ifunc.c
-
-    LOCAL_SDK_VERSION := current
-    module := libtest_ifunc
-    build_type := target
-    build_target := SHARED_LIBRARY
-
     ifeq ($(TARGET_ARCH),arm64)
       libtest_ifunc_multilib := 64
       # TODO: This is a workaround - remove it once gcc
@@ -311,6 +286,7 @@ ifeq ($(TARGET_ARCH),$(filter $(TARGET_ARCH),arm64 x86 x86_64))
       libtest_ifunc_cflags := -mglibc
     endif
 
+    build_type := target
     include $(TEST_PATH)/Android.build.mk
 endif
 
@@ -322,11 +298,7 @@ libtest_atexit_src_files := \
     atexit_testlib.cpp
 
 module := libtest_atexit
-build_target := SHARED_LIBRARY
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_type := target
-include $(TEST_PATH)/Android.build.mk
-build_type := host
-include $(TEST_PATH)/Android.build.mk
 
 # -----------------------------------------------------------------------------
 # Library with weak function
@@ -335,8 +307,4 @@ libtest_dlsym_weak_func_src_files := \
     dlsym_weak_function.cpp
 
 module := libtest_dlsym_weak_func
-build_target := SHARED_LIBRARY
+include $(LOCAL_PATH)/Android.build.testlib.mk
-build_type := target
-include $(TEST_PATH)/Android.build.mk
-build_type := host
-include $(TEST_PATH)/Android.build.mk

tests/libs/dlopen_testlib_simple.cpp

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+#include <stdint.h>
 #include <stdlib.h>
 
 uint32_t dlopen_testlib_taxicab_number = 1729;