]> git.ipfire.org Git - thirdparty/glibc.git/commit
dlfcn: Guard __dlerror_main_freeres with __libc_once_get (once) [BZ# 24476]
authorMark Wielaard <mark@klomp.org>
Wed, 15 May 2019 15:14:01 +0000 (17:14 +0200)
committerMark Wielaard <mark@klomp.org>
Wed, 15 May 2019 21:34:55 +0000 (23:34 +0200)
commit11b451c8868d8a2b0edc5dfd44fc58d9ee538be0
tree54af71dd79004a0769dd70c9ab332c73b49a9ae3
parent5dde9ef79a62b305f054ae9594a32e687dafbea6
dlfcn: Guard __dlerror_main_freeres with __libc_once_get (once) [BZ# 24476]

dlerror.c (__dlerror_main_freeres) will try to free resources which only
have been initialized when init () has been called. That function is
called when resources are needed using __libc_once (once, init) where
once is a __libc_once_define (static, once) in the dlerror.c file.
Trying to free those resources if init () hasn't been called will
produce errors under valgrind memcheck. So guard the freeing of those
resources using __libc_once_get (once) and make sure we have a valid
key. Also add a similar guard to __dlerror ().

* dlfcn/dlerror.c (__dlerror_main_freeres): Guard using
__libc_once_get (once) and static_bug == NULL.
(__dlerror): Check we have a valid key, set result to static_buf
otherwise.

Reviewed-by: Carlos O'Donell <carlos@redhat.com>
ChangeLog
dlfcn/dlerror.c