svn commit: samba r21539 - in branches/SAMBA_4_0/source/torture/basic: .

metze at samba.org metze at samba.org
Mon Feb 26 12:02:10 GMT 2007


Author: metze
Date: 2007-02-26 12:02:10 +0000 (Mon, 26 Feb 2007)
New Revision: 21539

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

Log:
try to get rid of all valgrind warnings...

metze
Modified:
   branches/SAMBA_4_0/source/torture/basic/aliases.c


Changeset:
Modified: branches/SAMBA_4_0/source/torture/basic/aliases.c
===================================================================
--- branches/SAMBA_4_0/source/torture/basic/aliases.c	2007-02-26 11:49:32 UTC (rev 21538)
+++ branches/SAMBA_4_0/source/torture/basic/aliases.c	2007-02-26 12:02:10 UTC (rev 21539)
@@ -84,8 +84,7 @@
 }
 
 /* look for qfsinfo aliases */
-static bool qfsinfo_aliases(struct torture_context *tctx, 
-							struct smbcli_state *cli)
+static bool qfsinfo_aliases(struct torture_context *tctx, struct smbcli_state *cli)
 {
 	struct smb_trans2 t2;
 	uint16_t setup = TRANSACT2_QFSINFO;
@@ -97,15 +96,15 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob(NULL, 2);
+	t2.in.params = data_blob_talloc_zero(tctx, 2);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	return gen_aliases(tctx, cli, &t2, 0);
 }
 
 /* look for qfileinfo aliases */
-static bool qfileinfo_aliases(struct torture_context *tctx, 
-							  struct smbcli_state *cli)
+static bool qfileinfo_aliases(struct torture_context *tctx, struct smbcli_state *cli)
 {
 	struct smb_trans2 t2;
 	uint16_t setup = TRANSACT2_QFILEINFO;
@@ -119,8 +118,9 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob(NULL, 4);
+	t2.in.params = data_blob_talloc_zero(tctx, 4);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	smbcli_unlink(cli->tree, fname);
 	fnum = create_complex_file(cli, cli, fname);
@@ -143,15 +143,13 @@
 
 
 /* look for qpathinfo aliases */
-static bool qpathinfo_aliases(struct torture_context *tctx, 
-							  struct smbcli_state *cli)
+static bool qpathinfo_aliases(struct torture_context *tctx, struct smbcli_state *cli)
 {
 	struct smb_trans2 t2;
 	uint16_t setup = TRANSACT2_QPATHINFO;
 	const char *fname = "\\qpathinfo_aliases.txt";
 	int fnum;
 
-	ZERO_STRUCT(t2);
 	t2.in.max_param = 2;
 	t2.in.max_data = smb_raw_max_trans_data(cli->tree, 2);
 	t2.in.max_setup = 0;
@@ -159,8 +157,9 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob_talloc(tctx, NULL, 6);
+	t2.in.params = data_blob_talloc_zero(tctx, 6);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	smbcli_unlink(cli->tree, fname);
 	fnum = create_complex_file(cli, cli, fname);
@@ -186,8 +185,7 @@
 
 
 /* look for trans2 findfirst aliases */
-static bool findfirst_aliases(struct torture_context *tctx, 
-							  struct smbcli_state *cli)
+static bool findfirst_aliases(struct torture_context *tctx, struct smbcli_state *cli)
 {
 	struct smb_trans2 t2;
 	uint16_t setup = TRANSACT2_FINDFIRST;
@@ -201,8 +199,9 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob_talloc(tctx, NULL, 12);
+	t2.in.params = data_blob_talloc_zero(tctx, 12);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	smbcli_unlink(cli->tree, fname);
 	fnum = create_complex_file(cli, cli, fname);
@@ -300,8 +299,7 @@
 
 
 /* look for setfileinfo aliases */
-static bool setfileinfo_aliases(struct torture_context *tctx, 
-								struct smbcli_state *cli)
+static bool setfileinfo_aliases(struct torture_context *tctx, struct smbcli_state *cli)
 {
 	struct smb_trans2 t2;
 	uint16_t setup = TRANSACT2_SETFILEINFO;
@@ -315,8 +313,9 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob(NULL, 6);
+	t2.in.params = data_blob_talloc_zero(tctx, 6);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	smbcli_unlink(cli->tree, fname);
 	fnum = create_complex_file(cli, cli, fname);
@@ -353,8 +352,9 @@
 	t2.in.timeout = 0;
 	t2.in.setup_count = 1;
 	t2.in.setup = &setup;
-	t2.in.params = data_blob_talloc(tctx, NULL, 4);
+	t2.in.params = data_blob_talloc_zero(tctx, 4);
 	t2.in.data = data_blob(NULL, 0);
+	ZERO_STRUCT(t2.out);
 
 	smbcli_unlink(cli->tree, fname);
 
@@ -384,19 +384,14 @@
 /* look for aliased info levels in trans2 calls */
 struct torture_suite *torture_trans2_aliases(void)
 {
-	struct torture_suite *suite = torture_suite_create(
-										talloc_autofree_context(), 
-										"ALIASES");
+	struct torture_suite *suite = torture_suite_create(talloc_autofree_context(), "ALIASES");
 
-	torture_suite_add_1smb_test(suite, "QFILEINFO aliases", 
-								qfsinfo_aliases);
+	torture_suite_add_1smb_test(suite, "QFILEINFO aliases", qfsinfo_aliases);
 	torture_suite_add_1smb_test(suite, "QFSINFO aliases", qfileinfo_aliases);
 	torture_suite_add_1smb_test(suite, "QPATHINFO aliases", qpathinfo_aliases);
 	torture_suite_add_1smb_test(suite, "FINDFIRST aliases", findfirst_aliases);
-	torture_suite_add_1smb_test(suite, "setfileinfo_aliases", 
-								setfileinfo_aliases);
-	torture_suite_add_1smb_test(suite, "setpathinfo_aliases", 
-								setpathinfo_aliases);
+	torture_suite_add_1smb_test(suite, "setfileinfo_aliases", setfileinfo_aliases);
+	torture_suite_add_1smb_test(suite, "setpathinfo_aliases", setpathinfo_aliases);
 
 	return suite;
 }



More information about the samba-cvs mailing list