diff options
| author | Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> | 2018-04-26 10:41:43 -0300 |
|---|---|---|
| committer | Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> | 2018-04-26 10:41:43 -0300 |
| commit | 81b49e969233ac5b9faf5c043bb0cc64bf9d6271 (patch) | |
| tree | 5cd448f269f2206f15da2af3995e13be3bbfd2ec /sysdeps/gnu | |
| parent | 7d80f48e93508476d0d17d368b34b28cd5dda2a5 (diff) | |
| download | glibc-81b49e969233ac5b9faf5c043bb0cc64bf9d6271.tar.gz glibc-81b49e969233ac5b9faf5c043bb0cc64bf9d6271.tar.xz glibc-81b49e969233ac5b9faf5c043bb0cc64bf9d6271.zip | |
Increase robustness of internal dlopen() by using RTLD_NOW [BZ #22766]
Prevent random runtime crashes due to missing symbols caused by mixed libnss_* versions. [BZ #22766] * include/dlfcn.h [__libc_dl_open]: Replace RTLD_LAZY with RTLD_NOW. * sysdeps/gnu/unwind-resume.c (__lib_gcc_s_init): Replace __libc_dlopen_mode() using RTLD_NOW with __libc_dlopen. * sysdeps/nptl/unwind-forcedunwind.c: Likewise. Signed-off-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'sysdeps/gnu')
| -rw-r--r-- | sysdeps/gnu/unwind-resume.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/sysdeps/gnu/unwind-resume.c b/sysdeps/gnu/unwind-resume.c index 7f9a1bf2c7..9283d925de 100644 --- a/sysdeps/gnu/unwind-resume.c +++ b/sysdeps/gnu/unwind-resume.c @@ -35,11 +35,8 @@ __libgcc_s_init (void) void *resume, *personality; void *handle; - /* Use RTLD_NOW here for consistency with pthread_cancel_init. - RTLD_NOW will rarely make a difference here because unwinding is - already in progress, so libgcc_s.so has already been loaded if - its unwinder is used. */ - handle = __libc_dlopen_mode (LIBGCC_S_SO, RTLD_NOW | __RTLD_DLOPEN); + /* See include/dlfcn.h. Use of __libc_dlopen requires RTLD_NOW. */ + handle = __libc_dlopen (LIBGCC_S_SO); if (handle == NULL || (resume = __libc_dlsym (handle, "_Unwind_Resume")) == NULL |