s3: Fix a winbind race leading to 100% CPU
commit7172d1f74e94a5ed4df1aa9ace03a4a711cc3e39
authorVolker Lendecke <vl@samba.org>
Fri, 26 Aug 2011 14:54:18 +0000 (26 16:54 +0200)
committerKarolin Seeger <kseeger@samba.org>
Sat, 30 Jun 2012 11:44:41 +0000 (30 13:44 +0200)
treef3c33e74b530f0ae345637b1580d11e9b26e41c3
parent45ce22b9ffb43ea555c263f119e6587f5ede3e56
s3: Fix a winbind race leading to 100% CPU

This fixes a race condition that leads to the winbindd_children list becoming
corrupted. It happens when on a busy winbind SIGCHLD is a bit late.

Imagine a winbind with multiple requests in the queue for a single child. Child
dies, and before the SIGCHLD handler is called we find the socket to be dead.
wb_child_request_done is called, receiving an error from wb_simple_trans_recv.
It closes the socket. Then immediately the wb_child_request_trigger will do
another fork_domain_child before the signal handler is called. This means that
we do another fork_domain_child, we have child->sock==-1 at this point.
fork_domain_child will do a DLIST_ADD(winbindd_children, child) a second time
where the child is already part of that list. This corrupts the list. Then the
signal handler kicks in, spinning in

for (child = winbindd_children; child != NULL; child = child->next) {

forever. Not good. This patch makes sure that both conditions (sock==-1 and not
part of the list) for a winbindd_child struct match up.

Autobuild-User: Volker Lendecke <vlendec@samba.org>
Autobuild-Date: Fri Aug 26 18:51:24 CEST 2011 on sn-devel-104

Fix bug #9000 - winbindd hangs when disconnect domain connection.
(cherry picked from commit 41c2411286f76919546b677f98f1166f1e40c706)
source3/winbindd/winbindd_dual.c