From: Günther Deschner <gd@samba.org>
Date: Fri, 5 Jun 2009 23:50:17 +0000 (+0200)
Subject: s3-winbindd: add some debug statements while tracking down a bug.
X-Git-Tag: tdb-1.1.5~247
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f19c9147a5f2222ce8c21d8e194ac7de1a0a196f;p=thirdparty%2Fsamba.git

s3-winbindd: add some debug statements while tracking down a bug.

Guenther
---

diff --git a/source3/winbindd/winbindd_group.c b/source3/winbindd/winbindd_group.c
index 3273a288e25..913f63112f4 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 1a358b2b440..b18f0ff5957 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;