Avoid a race condition in glibc between AIO and setresuid().
authorAndrew Tridgell <tridge@samba.org>
Sun, 24 Aug 2008 03:56:59 +0000 (13:56 +1000)
committerMichael Adam <obnox@samba.org>
Tue, 26 Aug 2008 12:39:42 +0000 (14:39 +0200)
commitd9f61dbdc91fae6560361f98d981b1f7bea80422
tree6be75509fe252fdf14a309dfe311947a0bb9ed50
parent52c39dd55d0a3eda3e9ed9a0981f546ca7b88cff
Avoid a race condition in glibc between AIO and setresuid().

See this test: http://samba.org/~tridge/junkcode/aio_uid.c

The problem is that setresuid() tries to be clever about threads, and
tries to change the euid of any threads that are running. If a AIO read
or write completes while this is going on then the signal from the thread
where the IO completed is lost, as it gets -1/EPERM from rt_sigqueueinfo()

The simplest fix is to try to use setreuid() instead of setresuid(),
as setreuid() doesn't try to be clever. Unfortunately this also means
we must use become_root()/unbecome_root() in the aio code.
(This used to be commit 56c5a6f024875bb79b0104beb36f6b0ec1e1e9f9)
source3/configure.in