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)
committerVolker Lendecke <vl@samba.org>
Fri, 25 Jul 2008 10:10:57 +0000 (12:10 +0200)
commit8691709626b0d461de91b8fc9d10c730d1f183dd
treebf92f5504fca0f42f3b492c837308f5ffb2cdcef
parentc6ca0f7d462c51122ccb1068306410ccdbee7b00
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
source/winbindd/winbindd_dual.c