From f19c9147a5f2222ce8c21d8e194ac7de1a0a196f Mon Sep 17 00:00:00 2001
From: Günther Deschner <gd@samba.org>
Date: Sat, 6 Jun 2009 01:50:17 +0200
Subject: s3-winbindd: add some debug statements while tracking down a bug.

Guenther
---
 source3/winbindd/winbindd_group.c  | 16 ++++++++++++++--
 source3/winbindd/winbindd_passdb.c |  6 ++++++
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/source3/winbindd/winbindd_group.c b/source3/winbindd/winbindd_group.c
index 3273a288e2..913f63112f 100644
--- a/source3/winbindd/winbindd_group.c
+++ b/source3/winbindd/winbindd_group.c
@@ -460,6 +460,8 @@ static NTSTATUS expand_groups( TALLOC_CTX *ctx,
 	*new_glist = NULL;
 	*n_new_glist = 0;
 
+	DEBUG(10,("expand_groups:\n"));
+
 	for ( i=0; i<n_glist; i++ ) {
 		tmp_ctx = talloc_new( ctx );
 
@@ -469,8 +471,12 @@ static NTSTATUS expand_groups( TALLOC_CTX *ctx,
 						     &glist[i], &num_names,
 						     &sid_mem, &names,
 						     &name_types);
-		if ( !NT_STATUS_IS_OK(status) )
+		if (!NT_STATUS_IS_OK(status)) {
+			DEBUG(10,("expand_groups: lookup_groupmem for "
+				"sid %s failed with: %s\n",
+				sid_string_dbg(&glist[i]), nt_errstr(status)));
 			goto out;
+		}
 
 		/* Separate users and groups into two lists */
 
@@ -514,6 +520,11 @@ static NTSTATUS expand_groups( TALLOC_CTX *ctx,
  out:
 	TALLOC_FREE( tmp_ctx );
 
+	if (!NT_STATUS_IS_OK(status)) {
+		DEBUG(10,("expand_groups: returning with %s\n",
+			nt_errstr(status)));
+	}
+
 	return status;
 }
 
@@ -726,7 +737,8 @@ done:
 
 	talloc_destroy(mem_ctx);
 
-	DEBUG(10, ("fill_grent_mem returning %d\n", result));
+	DEBUG(10,("fill_grent_mem returning %s\n",
+		result == true ? "true" : "false"));
 
 	return result;
 }
diff --git a/source3/winbindd/winbindd_passdb.c b/source3/winbindd/winbindd_passdb.c
index 1a358b2b44..b18f0ff595 100644
--- a/source3/winbindd/winbindd_passdb.c
+++ b/source3/winbindd/winbindd_passdb.c
@@ -412,6 +412,9 @@ static NTSTATUS builtin_lookup_groupmem(struct winbindd_domain *domain,
 				DOM_SID **sid_mem, char ***names,
 				uint32 **name_types)
 {
+	DEBUG(10,("passdb: lookup_groupmem (builtin) %s sid=%s\n", domain->name,
+		  sid_string_dbg(group_sid)));
+
 	*num_names = 0;
 	*sid_mem = NULL;
 	*names = NULL;
@@ -571,6 +574,9 @@ static NTSTATUS sam_lookup_groupmem(struct winbindd_domain *domain,
 	struct lsa_name_info *lsa_names;
 	TALLOC_CTX *tmp_ctx;
 
+	DEBUG(10,("passdb: lookup_groupmem (sam) %s sid=%s\n", domain->name,
+		  sid_string_dbg(group_sid)));
+
 	if (!sid_check_is_in_our_domain(group_sid)) {
 		/* There's no groups, only aliases in BUILTIN */
 		return NT_STATUS_NO_SUCH_GROUP;
-- 
cgit