diff options
author | Florian Weimer <fweimer@redhat.com> | 2020-05-18 18:25:18 +0200 |
---|---|---|
committer | Florian Weimer <fweimer@redhat.com> | 2020-06-02 10:32:37 +0200 |
commit | c2322a561f74b8fa41b9d9c6b8eb57e28de46f3e (patch) | |
tree | 85673d97ee3c4a1d79d67e3e267a12c63cf7946f /nptl/pthread_getattr_default_np.c | |
parent | 86ed0774cf0de1d281ca3c773758c9fe6f2784b4 (diff) | |
download | glibc-c2322a561f74b8fa41b9d9c6b8eb57e28de46f3e.tar.gz glibc-c2322a561f74b8fa41b9d9c6b8eb57e28de46f3e.tar.xz glibc-c2322a561f74b8fa41b9d9c6b8eb57e28de46f3e.zip |
nptl: Change type of __default_pthread_attr
union pthread_attr_transparent has always the correct size, even if pthread_attr_t has padding that is not present in struct pthread_attr. This should not result in an observable behavioral change. The existing code appears to have been correct, but it was brittle because it was not clear which functions were allowed to write to an entire pthread_attr_t argument (e.g., by copying it). Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'nptl/pthread_getattr_default_np.c')
-rw-r--r-- | nptl/pthread_getattr_default_np.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/nptl/pthread_getattr_default_np.c b/nptl/pthread_getattr_default_np.c index 5c99f980e2..f3ce1c2885 100644 --- a/nptl/pthread_getattr_default_np.c +++ b/nptl/pthread_getattr_default_np.c @@ -22,8 +22,7 @@ int __pthread_getattr_default_np (pthread_attr_t *out) { lll_lock (__default_pthread_attr_lock, LLL_PRIVATE); - int ret = __pthread_attr_copy (out, - (pthread_attr_t *) &__default_pthread_attr); + int ret = __pthread_attr_copy (out, &__default_pthread_attr.external); lll_unlock (__default_pthread_attr_lock, LLL_PRIVATE); return ret; } |