rtld: add some dlopen tests

dlopen_basic just tests that libthr.so can be dlopen()ed, which will
just serve as a sanity check that "libthr.so" is a thing that can be
dlopened in case we get a weird failure in dlopen_recursing.

dlopen_recursing tests a regression reported after the libsys split,
where some dlopen() may cause infinite recursion and a resulting crash.
This case is inspired by bdrewery's description of what seemed to be
causing his issue.

The corresponding fix landed in commit
968a18975a ("rtld: ignore load_filtees() calls if we already [...]")

Reviewed by:	kib

(cherry picked from commit c5796f1572)
This commit is contained in:
Kyle Evans 2024-02-13 09:38:02 -06:00
parent 4187a7dbbe
commit 7e93233111
2 changed files with 54 additions and 0 deletions

View file

@ -13,6 +13,8 @@ ATF_TESTS_C+= ld_preload_fds
SRCS.$t= $t.c common.c
.endfor
ATF_TESTS_C+= dlopen_test
WARNS?= 3
.include <bsd.test.mk>

View file

@ -0,0 +1,52 @@
/*-
*
* Copyright (C) 2024 Kyle Evans <kevans@FreeBSD.org>
*
* SPDX-License-Identifier: BSD-2-Clause
*
*/
#include <dlfcn.h>
#include <atf-c.h>
ATF_TC_WITHOUT_HEAD(dlopen_basic);
ATF_TC_BODY(dlopen_basic, tc)
{
void *hdl, *sym;
hdl = dlopen("libthr.so", RTLD_NOW);
ATF_REQUIRE(hdl != NULL);
sym = dlsym(hdl, "pthread_create");
ATF_REQUIRE(sym != NULL);
dlclose(hdl);
sym = dlsym(hdl, "pthread_create");
ATF_REQUIRE(sym == NULL);
}
ATF_TC_WITHOUT_HEAD(dlopen_recursing);
ATF_TC_BODY(dlopen_recursing, tc)
{
void *hdl;
/*
* If this doesn't crash, we're OK; a regression at one point caused
* some infinite recursion here.
*/
hdl = dlopen("libthr.so", RTLD_NOW | RTLD_GLOBAL);
ATF_REQUIRE(hdl != NULL);
dlclose(hdl);
}
ATF_TP_ADD_TCS(tp)
{
ATF_TP_ADD_TC(tp, dlopen_basic);
ATF_TP_ADD_TC(tp, dlopen_recursing);
return atf_no_error();
}