From c56ddaf7efdd5744c3f1adb7e40b1db5cbe01523 Mon Sep 17 00:00:00 2001
From: Jelmer Vernooij <jelmer@samba.org>
Date: Thu, 2 Jun 2005 13:21:11 +0000
Subject: r7195: - Fix echo pipe - Don't allocate strings - Give higher
 preference to the [out] part of variables when they   are being used by
 another [out] variable. Also make sure that   [in] variables never use [out]
 variables (i.e. switch_is() on an   [in] variable can no longer use an [out]
 variable). (This used to be commit 837c83d77a2d1990419c4f3e343616daf8da5799)

---
 source4/torture/rpc/echo.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

(limited to 'source4/torture/rpc/echo.c')

diff --git a/source4/torture/rpc/echo.c b/source4/torture/rpc/echo.c
index 14bfe573ce..96a716a312 100644
--- a/source4/torture/rpc/echo.c
+++ b/source4/torture/rpc/echo.c
@@ -107,15 +107,11 @@ static BOOL test_sourcedata(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx)
 	int i;
 	NTSTATUS status;
 	int len = 200000 + (random() % 5000);
-	uint8_t *data_out;
 	struct echo_SourceData r;
 
 	printf("\nTesting SourceData\n");
 
-	data_out = talloc_size(mem_ctx, len);
-
 	r.in.len = len;
-	r.out.data = data_out;
 
 	status = dcerpc_echo_SourceData(p, mem_ctx, &r);
 	if (!NT_STATUS_IS_OK(status)) {
@@ -124,9 +120,9 @@ static BOOL test_sourcedata(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx)
 	}
 
 	for (i=0;i<len;i++) {
-		uint8_t *v = (uint8_t *)data_out;
+		uint8_t *v = (uint8_t *)r.out.data;
 		if (v[i] != (i & 0xFF)) {
-			printf("bad data 0x%x at %d\n", (uint8_t)data_out[i], i);
+			printf("bad data 0x%x at %d\n", (uint8_t)r.out.data[i], i);
 			return False;
 		}
 	}
@@ -199,6 +195,7 @@ static BOOL test_testcall2(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx)
 
 	for (i=1;i<=7;i++) {
 		r.in.level = i;
+		r.out.info = talloc(mem_ctx, union echo_Info);
 
 		printf("\nTesting TestCall2 level %d\n", i);
 		status = dcerpc_echo_TestCall2(p, mem_ctx, &r);
-- 
cgit