diff options
author | Ulrich Drepper <drepper@redhat.com> | 2007-06-09 19:45:57 +0000 |
---|---|---|
committer | Ulrich Drepper <drepper@redhat.com> | 2007-06-09 19:45:57 +0000 |
commit | 90cd2413df749eff9d055ef1868b616a65cf3fef (patch) | |
tree | a6029c8c37fdd4e053ab5ac6bab748faa1b7f893 /elf/do-lookup.h | |
parent | 3c457089679373802fae6ef65a1804e1c9074b67 (diff) | |
download | glibc-90cd2413df749eff9d055ef1868b616a65cf3fef.tar.gz glibc-90cd2413df749eff9d055ef1868b616a65cf3fef.tar.xz glibc-90cd2413df749eff9d055ef1868b616a65cf3fef.zip |
* elf/do-lookup.h (do_lookup_x): Read r_nlist before r_list and
make sure gcc doesn't mess around with this.
Diffstat (limited to 'elf/do-lookup.h')
-rw-r--r-- | elf/do-lookup.h | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/elf/do-lookup.h b/elf/do-lookup.h index 2585d83005..ab9a510ba1 100644 --- a/elf/do-lookup.h +++ b/elf/do-lookup.h @@ -1,5 +1,5 @@ /* Look up a symbol in the loaded objects. - Copyright (C) 1995-2004, 2005, 2006 Free Software Foundation, Inc. + Copyright (C) 1995-2004, 2005, 2006, 2007 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -29,8 +29,13 @@ do_lookup_x (const char *undef_name, uint_fast32_t new_hash, const struct r_found_version *const version, int flags, struct link_map *skip, int type_class) { - struct link_map **list = scope->r_list; size_t n = scope->r_nlist; + /* Make sure we read the value before proceeding. Otherwise we + might use r_list pointing to the initial scope and r_nlist being + the value after a resize. That is the only path in dl-open.c not + protected by GSCOPE. A read barrier here might be to expensive. */ + __asm volatile ("" : "+r" (n), "+m" (scope->r_list)); + struct link_map **list = scope->r_list; do { |