fix deadlock in synccall after threaded fork
authorSamuel Holland <samuel@sholland.org>
Mon, 1 Jul 2019 03:44:28 +0000 (22:44 -0500)
committerRich Felker <dalias@aerifal.cx>
Mon, 1 Jul 2019 14:10:52 +0000 (10:10 -0400)
commit84ebec6ceea997568c1711e7d810ad6bb3a96346
tree205112963e85f6d7e7db5a2d929da6e8131b8ff0
parent3d178a7e2b75066593fbd5705742c5808395d90d
fix deadlock in synccall after threaded fork

synccall may be called by AS-safe functions such as setuid/setgid after
fork. although fork() resets libc.threads_minus_one, causing synccall to
take the single-threaded path, synccall still takes the thread list
lock. This lock may be held by another thread if for example fork()
races with pthread_create(). After fork(), the value of the lock is
meaningless, so clear it.

maintainer's note: commit 8f11e6127fe93093f81a52b15bb1537edc3fc8af and
e4235d70672d9751d7718ddc2b52d0b426430768 introduced this regression.
the state protected by this lock is the linked list, which is entirely
replaced in the child path of fork (next=prev=self), so resetting it
is semantically sound.
src/process/fork.c