svn commit: samba r19285 - in branches/SAMBA_4_0_RELEASE/source/libnet: .

[abartlet at samba.org]

Author: abartlet
Date: 2006-10-14 21:42:48 +0000 (Sat, 14 Oct 2006)
New Revision: 19285

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=19285

Log:
Break up check for search count from check for errors.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0_RELEASE/source/libnet/libnet_join.c


Changeset:
Modified: branches/SAMBA_4_0_RELEASE/source/libnet/libnet_join.c
===================================================================
--- branches/SAMBA_4_0_RELEASE/source/libnet/libnet_join.c	2006-10-14 21:42:17 UTC (rev 19284)
+++ branches/SAMBA_4_0_RELEASE/source/libnet/libnet_join.c	2006-10-14 21:42:48 UTC (rev 19285)
@@ -246,14 +246,23 @@
 
 	/* search for the user's record */
 	ret = ldb_search(remote_ldb, account_dn, LDB_SCOPE_BASE, 
-			     NULL, attrs, &res);
-	if (ret != LDB_SUCCESS || res->count != 1) {
+			 NULL, attrs, &res);
+	if (ret != LDB_SUCCESS) {
 		r->out.error_string = talloc_asprintf(r, "ldb_search for %s failed - %s",
 						      account_dn_str, ldb_errstring(remote_ldb));
 		talloc_free(tmp_ctx);
 		return NT_STATUS_UNSUCCESSFUL;
 	}
 
+	talloc_steal(tmp_ctx, res);
+
+	if (res->count != 1) {
+		r->out.error_string = talloc_asprintf(r, "ldb_search for %s failed - found %d entries",
+						      account_dn_str, res->count);
+		talloc_free(tmp_ctx);
+		return NT_STATUS_UNSUCCESSFUL;
+	}
+
 	/* If we have a kvno recorded in AD, we need it locally as well */
 	kvno = ldb_msg_find_attr_as_uint(res->msgs[0], "msDS-KeyVersionNumber", 0);