From: Alexey Izbyshev <izbyshev@ispras.ru>
To: musl@lists.openwall.com
Subject: dlsym(handle) may search in unrelated libraries
Date: Wed, 06 Feb 2019 00:02:39 +0300 [thread overview]
Message-ID: <b400e2a8e98e3731964e0afa171185b6@ispras.ru> (raw)
Hello!
I've discovered a bug in musl dynamic loader (tested on 1.1.21) which is
demonstrated by the following simple example:
$ cat bar.c
int bar = 42;
$ musl-gcc -fPIC -shared bar.c -o libbar.so
$ cat foo.c
extern int bar;
int *foo = &bar;
$ musl-gcc -fPIC -shared foo.c -L. -lbar -Wl,-rpath='$ORIGIN' -o
libfoo.so
$ cat main.c
#include <dlfcn.h>
#include <stdio.h>
int main(void) {
if (!dlopen("libfoo.so", RTLD_NOW))
return 1;
void *h = dlopen("libc.so.6", RTLD_NOW);
printf("%p\n", dlsym(h, "bar"));
}
$ musl-gcc main.c -Wl,-rpath='$ORIGIN' -ldl
$ ./a.out
0x7fd7ebe96020
dlsym(handle) is supposed to search only in the library referred to by
the handle and in its dependencies. "libc.so.6" doesn't have
dependencies and doesn't have a definition for "bar", so dlsym(h, "bar")
should return NULL, but it finds "bar" in libbar.so instead.
The problem occurs because of the following:
1) Initially, "deps" in dso structure for libc.so.6 is NULL.
2) When dlopen("libc.so.6") is called, "first_load" is true, despite
that it's not actually the first load (ldso/dynlink.c:1835):
/* First load handling */
int first_load = !p->deps;
if (first_load) {
load_deps(p);
3) load_deps() then iterates over the dso list starting from
"libc.so.6", treating all libraries found in DT_NEEDED of each processed
dso as dependencies of "libc.so.6". However, the dso list already
contains "libfoo.so" loaded earlier, so "libbar.so" (which is needed by
"libfoo.so") is treated as a dependency of "libc.so.6". As a result,
dlsym(h, "bar") succeeds.
It's also notable that "libfoo.so" and "libbar.so" were loaded with
RTLD_LOCAL, but this bug effectively makes their symbols available in
such searches regardless of the scope of a library used with dlsym().
ISTM that load_deps(p) was written to work only in real "first load"
situations, where "p" is initially the last dso in the list, and new
dsos are only added to the list in the course of recursive loading of
the dependencies of "p".
Could this be fixed? Thanks!
(Please CC me on replying, I'm not subscribed to the list.)
Alexey
next reply other threads:[~2019-02-05 21:02 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-05 21:02 Alexey Izbyshev [this message]
2019-02-06 13:40 ` Alexey Izbyshev
2019-02-06 16:02 ` Markus Wichmann
2019-02-06 17:02 ` Alexey Izbyshev
2019-02-06 20:25 ` Markus Wichmann
2019-02-06 21:23 ` Alexey Izbyshev
2019-02-07 5:33 ` Markus Wichmann
2019-02-07 13:42 ` Alexey Izbyshev
2019-02-07 17:43 ` Markus Wichmann
2019-02-07 20:37 ` Markus Wichmann
2019-02-07 21:29 ` Rich Felker
2019-02-07 16:54 ` Rich Felker
2019-02-07 18:36 ` Markus Wichmann
2019-02-07 18:57 ` Rich Felker
2019-02-07 20:31 ` Markus Wichmann
2019-02-07 21:33 ` Rich Felker
2019-02-07 21:37 ` Rich Felker
2019-02-08 10:19 ` A. Wilcox
2019-02-08 12:00 ` Szabolcs Nagy
2019-02-08 16:09 ` Rich Felker
2019-02-09 22:53 Alexey Izbyshev
2019-02-10 1:03 ` Rich Felker
2019-02-26 15:07 ` Rich Felker
2019-03-04 2:11 ` Rich Felker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b400e2a8e98e3731964e0afa171185b6@ispras.ru \
--to=izbyshev@ispras.ru \
--cc=musl@lists.openwall.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).