Attachment 3671 Details for Bug 5814 – Patch for 3.0.x.

[patch] Patch for 3.0.x.

look (text/plain), 1.17 KB, created by Jeremy Allison on 2008-10-08 13:36:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2008-10-08 13:36:59 UTC

Size: 1.17 KB

patch

obsolete

>diff --git a/source/nsswitch/winbindd_dual.c b/source/nsswitch/winbindd_dual.c
>index 84c9a1c..106bb48 100644
>--- a/source/nsswitch/winbindd_dual.c
>+++ b/source/nsswitch/winbindd_dual.c
>@@ -294,6 +294,18 @@ static void schedule_async_request(struct winbindd_child *child)
> 		return;		/* Busy */
> 	}
> 
>+	/*
>+	 * This may be a reschedule, so we might
>+	 * have an existing timeout event pending on
>+	 * the first entry in the child->requests list
>+	 * (we only send one request at a time).
>+	 * Ensure we free it before we reschedule.
>+	 * Bug #5814, from hargagan <shargagan@novell.com>.
>+	 * JRA.
>+	 */
>+
>+	TALLOC_FREE(request->reply_timeout_event);
>+
> 	if ((child->pid == 0) && (!fork_domain_child(child))) {
> 		/* fork_domain_child failed.
> 		   Cancel all outstanding requests */
>@@ -539,6 +551,17 @@ void winbind_child_died(pid_t pid)
> 	child->event.flags = 0;
> 	child->pid = 0;
> 
>+	if (child->requests) {
>+		/*
>+		 * schedule_async_request() will also
>+		 * clear this event but the call is
>+		 * idempotent so it doesn't hurt to
>+		 * cover all possible future code
>+		 * paths. JRA.
>+		 */
>+		TALLOC_FREE(child->requests->reply_timeout_event);
>+	}
>+
> 	schedule_async_request(child);
> }
>

Actions: View

Attachments on bug 5814: 3663 | 3671