diff options
| author | Florian Weimer <fweimer@redhat.com> | 2021-10-01 18:16:41 +0200 |
|---|---|---|
| committer | Florian Weimer <fweimer@redhat.com> | 2021-10-01 18:16:41 +0200 |
| commit | 176c88f5214d8107d330971cbbfbbba5186a111f (patch) | |
| tree | ba4a801f6020a994a68a4d6ffa67274b486a4a91 /support | |
| parent | 2313ab153de29849f8fb0817ed3789fa1745225a (diff) | |
| download | glibc-176c88f5214d8107d330971cbbfbbba5186a111f.tar.gz glibc-176c88f5214d8107d330971cbbfbbba5186a111f.tar.xz glibc-176c88f5214d8107d330971cbbfbbba5186a111f.zip | |
support: Add check for TID zero in support_wait_for_thread_exit
Some kernel versions (observed with kernel 5.14 and earlier) can list
"0" entries in /proc/self/task. This happens when a thread exits
while the task list is being constructed. Treat this entry as not
present, like the proposed kernel patch does:
[PATCH] procfs: Do not list TID 0 in /proc/<pid>/task
<https://lore.kernel.org/all/8735pn5dx7.fsf@oldenburg.str.redhat.com/>
Fixes commit 032d74eaf6179100048a5bf0ce942e97dc8b9a60 ("support: Add
support_wait_for_thread_exit").
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Tested-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'support')
| -rw-r--r-- | support/support_wait_for_thread_exit.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/support/support_wait_for_thread_exit.c b/support/support_wait_for_thread_exit.c index 658a813810..5e3be421a7 100644 --- a/support/support_wait_for_thread_exit.c +++ b/support/support_wait_for_thread_exit.c @@ -43,7 +43,10 @@ support_wait_for_thread_exit (void) return; } - if (strcmp (e->d_name, ".") == 0 || strcmp (e->d_name, "..") == 0) + /* In some kernels, "0" entries denote a thread that has just + exited. */ + if (strcmp (e->d_name, ".") == 0 || strcmp (e->d_name, "..") == 0 + || strcmp (e->d_name, "0") == 0) continue; int task_tid = atoi (e->d_name); |