Fix a race condition in winbind leading to a crash
authorVolker Lendecke <vl@samba.org>
Fri, 25 Jul 2008 10:08:03 +0000 (12:08 +0200)
committerKarolin Seeger <kseeger@samba.org>
Fri, 25 Jul 2008 14:23:40 +0000 (16:23 +0200)
commitd1db27e6c94cb283fb70d6dbf531c65829d0f460
tree1e775d452a6ed229e85a044581a01cbc6fd7a5fe
parentc61578fc7c9dce8f22695532ee26179ba35b6a8f
Fix a race condition in winbind leading to a crash

When SIGCHLD handling is delayed for some reason, sending a request to a child
can fail early because the child has died already. In this case
async_main_request_sent() directly called the continuation function without
properly removing the malfunctioning child process and the requests in the
queue. The next request would then crash in the DLIST_ADD_END() in
async_request() because the request pending for the child had been
talloc_free()'ed and yet still was referenced in the list.

This one is *old*...

Volker
(cherry picked from commit 8691709626b0d461de91b8fc9d10c730d1f183dd)
source/winbindd/winbindd_dual.c