diff options
| author | Eric Andersen <andersen@codepoet.org> | 2004-02-14 11:30:39 +0000 |
|---|---|---|
| committer | Eric Andersen <andersen@codepoet.org> | 2004-02-14 11:30:39 +0000 |
| commit | deec524084aa2faad66f7aae7e8b5ba1ab6789fd (patch) | |
| tree | 8498414c429c57ce6b93a49c265aebcb8180c312 /ldso/ldso/dl-startup.c | |
| parent | 7bd4a2f4aaabca0e46015fe0216c086c7f61f1d4 (diff) | |
Joakim Tjernlund writes:
Hi it is me again.
This is the latest ldso patch. the NEW weak symbol handling works now
with a little special handling in _dl_find_hash(). You get to chose
if you want the new or old handling :)
There was 2 missing _dl_check_if_named_library_is_loaded() calls in _dlopen().
I then disabled the _dl_check_if_named_library_is_loaded() in dl-elf.c since
it is rendundant.
Question, why does some _dl_linux_resolver(), like i386, have 2 calls
to _dl_find_hash()? I think that is wrong, isn't it?
I really hope you can check this out soon ...
Diffstat (limited to 'ldso/ldso/dl-startup.c')
| -rw-r--r-- | ldso/ldso/dl-startup.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/ldso/ldso/dl-startup.c b/ldso/ldso/dl-startup.c index e9191e8a4..3a0c31419 100644 --- a/ldso/ldso/dl-startup.c +++ b/ldso/ldso/dl-startup.c @@ -513,11 +513,10 @@ found_got: /* We only do a partial dynamic linking right now. The user is not supposed to define any symbols that start with a '_dl', so we can do this with confidence. */ - if (!symname || symname[0] != '_' || - symname[1] != 'd' || symname[2] != 'l' || symname[3] != '_') - { + if (!symname || !_dl_symbol(symname)) { continue; } + symbol_addr = load_addr + symtab[symtab_index].st_value; if (!symbol_addr) { |