diff options
author | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2022-02-14 01:38:03 +0100 |
---|---|---|
committer | Samuel Thibault <samuel.thibault@ens-lyon.org> | 2022-02-14 19:29:02 +0100 |
commit | 06dbfcced3101886029ea3a46bcc98887d60f61e (patch) | |
tree | d7e2af56eaacd37df5f19759cfba9c244ce15ff9 /sysdeps/htl | |
parent | 33038a7d917889547c711be158ed34739af26351 (diff) | |
download | glibc-06dbfcced3101886029ea3a46bcc98887d60f61e.tar.gz glibc-06dbfcced3101886029ea3a46bcc98887d60f61e.tar.xz glibc-06dbfcced3101886029ea3a46bcc98887d60f61e.zip |
htl: Fix initializing the key lock
The static pthread_once_t in the pt-key.h header was creating one pthread_once_t per includer. We have to use a shared common pthread_once_t instead.
Diffstat (limited to 'sysdeps/htl')
-rw-r--r-- | sysdeps/htl/pt-key-create.c | 1 | ||||
-rw-r--r-- | sysdeps/htl/pt-key.h | 7 |
2 files changed, 5 insertions, 3 deletions
diff --git a/sysdeps/htl/pt-key-create.c b/sysdeps/htl/pt-key-create.c index ca4908c55c..f8dc5ac0c5 100644 --- a/sysdeps/htl/pt-key-create.c +++ b/sysdeps/htl/pt-key-create.c @@ -24,6 +24,7 @@ #include <pthreadP.h> pthread_mutex_t __pthread_key_lock; +pthread_once_t __pthread_key_once = PTHREAD_ONCE_INIT; void (**__pthread_key_destructors) (void *arg); int __pthread_key_size; diff --git a/sysdeps/htl/pt-key.h b/sysdeps/htl/pt-key.h index 34c12d51ab..ef652ecc6f 100644 --- a/sysdeps/htl/pt-key.h +++ b/sysdeps/htl/pt-key.h @@ -47,14 +47,15 @@ extern int __pthread_key_invalid_count; /* Protects the above variables. This must be a recursive lock: the destructors may call pthread_key_delete. */ extern pthread_mutex_t __pthread_key_lock; + +/* Protects the initialization of the mutex above. */ +extern pthread_once_t __pthread_key_once; #include <assert.h> static inline void __pthread_key_lock_ready (void) { - static pthread_once_t o = PTHREAD_ONCE_INIT; - void do_init (void) { int err; @@ -73,5 +74,5 @@ __pthread_key_lock_ready (void) assert_perror (err); } - __pthread_once (&o, do_init); + __pthread_once (&__pthread_key_once, do_init); } |