[SCM] Samba Shared Repository - branch master updated - tevent-0-9-8-143-g27beb7f

Andrew Tridgell tridge at samba.org
Thu Sep 10 06:59:39 MDT 2009


The branch, master has been updated
       via  27beb7fc7efde5c26917357e6a53e61e4cd7e04a (commit)
      from  f85e77c43f896c649e139f7aafa43c9afda774a3 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 27beb7fc7efde5c26917357e6a53e61e4cd7e04a
Author: Andrew Tridgell <tridge at samba.org>
Date:   Thu Sep 10 22:58:11 2009 +1000

    s4/torture: fixed lots of crash bugs in the DRS tests

-----------------------------------------------------------------------

Summary of changes:
 source4/torture/rpc/drsuapi.c |   36 +++++++++++++++++++-----------------
 1 files changed, 19 insertions(+), 17 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/torture/rpc/drsuapi.c b/source4/torture/rpc/drsuapi.c
index 2075fa7..2b4b17a 100644
--- a/source4/torture/rpc/drsuapi.c
+++ b/source4/torture/rpc/drsuapi.c
@@ -728,14 +728,14 @@ bool torture_rpc_drsuapi(struct torture_context *torture)
         NTSTATUS status;
         struct dcerpc_pipe *p;
 	bool ret = true;
-	struct DsPrivate priv;
+	struct DsPrivate *priv;
 	struct cli_credentials *machine_credentials;
 
-	ZERO_STRUCT(priv);
+	priv = talloc_zero(torture, struct DsPrivate);
 
-	priv.join = torture_join_domain(torture, TEST_MACHINE_NAME, ACB_SVRTRUST, 
+	priv->join = torture_join_domain(torture, TEST_MACHINE_NAME, ACB_SVRTRUST, 
 				       &machine_credentials);
-	if (!priv.join) {
+	if (!priv->join) {
 		torture_fail(torture, "Failed to join as BDC");
 	}
 
@@ -743,34 +743,36 @@ bool torture_rpc_drsuapi(struct torture_context *torture)
 					&p, 
 					&ndr_table_drsuapi);
 	if (!NT_STATUS_IS_OK(status)) {
-		torture_leave_domain(torture, priv.join);
+		torture_leave_domain(torture, priv->join);
 		torture_fail(torture, "Unable to connect to DRSUAPI pipe");
 	}
 
 	/* cache pipe handle */
-	priv.pipe = p;
+	priv->pipe = p;
 
-	ret &= test_DsBind(p, torture, &priv);
+	ret &= test_DsBind(p, torture, priv);
 #if 0
-	ret &= test_QuerySitesByCost(p, torture, &priv);
+	ret &= test_QuerySitesByCost(p, torture, priv);
 #endif
-	ret &= test_DsGetDomainControllerInfo(p, torture, &priv);
+	ret &= test_DsGetDomainControllerInfo(p, torture, priv);
+
+	ret &= test_DsCrackNames(torture, priv);
 
-	ret &= test_DsCrackNames(torture, &priv);
+	ret &= test_DsWriteAccountSpn(p, torture, priv);
 
-	ret &= test_DsWriteAccountSpn(p, torture, &priv);
+	ret &= test_DsReplicaGetInfo(p, torture, priv);
 
-	ret &= test_DsReplicaGetInfo(p, torture, &priv);
+	ret &= test_DsReplicaSync(p, torture, priv);
 
-	ret &= test_DsReplicaSync(p, torture, &priv);
+	ret &= test_DsReplicaUpdateRefs(p, torture, priv);
 
-	ret &= test_DsReplicaUpdateRefs(p, torture, &priv);
+	ret &= test_DsGetNCChanges(p, torture, priv);
 
-	ret &= test_DsGetNCChanges(p, torture, &priv);
+	ret &= test_DsUnbind(p, torture, priv);
 
-	ret &= test_DsUnbind(p, torture, &priv);
+	torture_leave_domain(torture, priv->join);
 
-	torture_leave_domain(torture, priv.join);
+	talloc_free(priv);
 
 	return ret;
 }


-- 
Samba Shared Repository


More information about the samba-cvs mailing list