Attachment 9774 Details for Bug 10487 – fix of getgrent

[patch] fix of getgrent

getgrent.patch (text/plain), 1.28 KB, created by Jura Sasek on 2014-03-14 10:26:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jura Sasek

Created: 2014-03-14 10:26:55 UTC

Size: 1.28 KB

patch

obsolete

> Samba Winbind GETGRENT fails when any trusted domain is not available
>
>--- a/nsswitch/wb_common.c	2014-02-07 17:26:50.046234000 +0100
>+++ b/nsswitch/wb_common.c	2014-02-07 17:33:40.280002000 +0100
>@@ -462,7 +462,7 @@
> 
> 		if (ret == 0) {
> 			/* Not ready for read yet... */
>-			if (total_time >= 30) {
>+			if (total_time >= 1000) {
> 				/* Timeout */
> 				winbind_close_sock();
> 				return -1;
>--- a/source3/winbindd/wb_next_grent.c	2014-02-07 17:27:14.079647000 +0100
>+++ b/source3/winbindd/wb_next_grent.c	2014-02-07 17:58:36.599520000 +0100
>@@ -103,18 +103,16 @@
> 	TALLOC_FREE(subreq);
> 	if (tevent_req_nterror(req, status)) {
> 		/* Ignore errors here, just log it */
>-		DEBUG(10, ("query_user_list for domain %s returned %s\n",
>+		DEBUG(3, ("query_user_list for domain %s returned: %s\n",
> 			   state->gstate->domain->name,
> 			   nt_errstr(status)));
>-		return;
> 	}
> 	if (!NT_STATUS_IS_OK(result)) {
> 		/* Ignore errors here, just log it */
>-		DEBUG(10, ("query_user_list for domain %s returned %s/%s\n",
>+		DEBUG(1, ("query_user_list for domain %s failed with: %s/%s\n",
> 			   state->gstate->domain->name,
> 			   nt_errstr(status), nt_errstr(result)));
>-		tevent_req_nterror(req, result);
>-		return;
>+		state->gstate->num_groups = 0;
> 	}
> 
> 	state->gstate->num_groups = state->next_groups.num_principals;

Actions: View

Attachments on bug 10487: 9755 | 9774