added a test for lsa_Close()

(This used to be commit dbadbbf1a7de011305c7f9d5d8d92467cb6e1c61)

2 files changed, 31 insertions, 1 deletions

diff --git a/source4/librpc/idl/lsa.idl b/source4/librpc/idl/lsa.idl
index f71a4e4c8c..8a1b4d546f 100644
--- a/source4/librpc/idl/lsa.idl
+++ b/source4/librpc/idl/lsa.idl
@@ -13,6 +13,7 @@
 		[in,ref]     policy_handle *handle
 		);
 	
+
 	/******************/
 	/* Function: 0x01 */
 	NTSTATUS lsa_Delete ();
@@ -51,7 +52,9 @@
 		security_descriptor *sec_desc;
 		lsa_QosInfo *sec_qos;
 	} lsa_ObjectAttribute;
-	
+
+	/* notice the screwup with the system_name - thats why MS created
+	   OpenPolicy2 */
 	NTSTATUS lsa_OpenPolicy (
 		[in]       uint16 *system_name,
 		[in,ref]   lsa_ObjectAttribute *attr,
@@ -60,6 +63,7 @@
 		);
 	
 
+
 	/******************/
 	/* Function: 0x07 */
 	NTSTATUS lsa_QueryInfoPolicy ();
diff --git a/source4/torture/rpc/lsa.c b/source4/torture/rpc/lsa.c
index 9df5dc0078..a22e2d6e32 100644
--- a/source4/torture/rpc/lsa.c
+++ b/source4/torture/rpc/lsa.c
@@ -329,6 +329,28 @@ static BOOL test_EnumAccounts(struct dcerpc_pipe *p,
 	return True;
 }
 
+
+static BOOL test_Close(struct dcerpc_pipe *p, 
+		       TALLOC_CTX *mem_ctx, 
+		       struct policy_handle *handle)
+{
+	NTSTATUS status;
+	struct lsa_Close r;
+
+	printf("\ntesting Close\n");
+
+	r.in.handle = handle;
+	status = dcerpc_lsa_Close(p, mem_ctx, &r);
+	if (!NT_STATUS_IS_OK(status)) {
+		printf("Close failed - %s\n", nt_errstr(status));
+		return False;
+	}
+
+	printf("\n");
+
+	return True;
+}
+
 BOOL torture_rpc_lsa(int dummy)
 {
         NTSTATUS status;
@@ -356,6 +378,10 @@ BOOL torture_rpc_lsa(int dummy)
 		ret = False;
 	}
 	
+	if (!test_Close(p, mem_ctx, &handle)) {
+		ret = False;
+	}
+	
         torture_rpc_close(p);
 
 	return ret;