fix sem_close unmapping of still-referenced semaphore
authorRich Felker <dalias@aerifal.cx>
Wed, 28 Oct 2020 20:13:45 +0000 (16:13 -0400)
committerRich Felker <dalias@aerifal.cx>
Wed, 28 Oct 2020 20:13:45 +0000 (16:13 -0400)
commitf70375df85d26235a45e74559afd69be59e5ff99
treed83ea1381c3cf18c251fb7d03f6b1d11e9ad6389
parent613ccabeb0c10ac818e36ef53e09753d23785cbf
fix sem_close unmapping of still-referenced semaphore

sem_open is required to return the same sem_t pointer for all
references to the same named semaphore when it's opened more than once
in the same process. thus we keep a table of all the mapped semaphores
and their reference counts. the code path for sem_close checked the
reference count, but then proceeded to unmap the semaphore regardless
of whether the count had reached zero.

add an immediate unlock-and-return for the nonzero refcnt case so the
property of performing the munmap syscall after releasing the lock can
be preserved.
src/thread/sem_open.c