s3: Fix bug 7844: Race in winbind
authorVolker Lendecke <vl@samba.org>
Mon, 6 Dec 2010 20:45:21 +0000 (12:45 -0800)
committerJeremy Allison <jra@samba.org>
Mon, 6 Dec 2010 22:21:02 +0000 (23:21 +0100)
commitf7d97868e4a5fe2c5f5a273311012022d45011bb
tree261f3c5454555a429e7d55d2036c9fae445430ef
parentab1b857f58f58dbc871fc444aa1dfefe4a3bfaeb
s3: Fix bug 7844: Race in winbind

If a child dies, the parent process right away closes the socket.
This is wrong, with tevent we still have events pending. This works
fine for epoll but does not for at least the FreeBSD select variant.
Tevent sticks a closed socket into the select masks. This then
returns an error EBADF. When this happens, the parent winbind dies
instead of forking a new child.

This moves the socket close from the SIGCHLD cleanup function to
the socket receiver. I could not reproduce the parent death anymore
and it did not create an obvious fd leak.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Mon Dec  6 23:21:02 CET 2010 on sn-devel-104
source3/winbindd/winbindd_dual.c