[SCM] Samba Shared Repository - branch master updated

Michael Adam obnox at samba.org
Tue Sep 11 14:10:02 MDT 2012


The branch, master has been updated
       via  df5f962 s4:torture:smb2: fix error reporting in the oplock-brl3 test
       via  439c907 s4:torture:smb2: fix error reporting in the oplock-brl2 test
       via  e9df135 s4:torture:smb2: fix error reporting in the oplock-brl1 test
       via  c960e55 s4:torture:smb2: remove an unused variable from the oplock-exclusive2 test
       via  7c6f1fb s4:torture:smb2: remove an unused variable from the oplock-exclusive1 test
      from  7c1fd74 At Michael's suggestion, factor common code into a function. My bad :-).

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


- Log -----------------------------------------------------------------
commit df5f962c60bb273c4759334ed6cc490d2a91d5ba
Author: Michael Adam <obnox at samba.org>
Date:   Tue Sep 11 16:28:45 2012 +0200

    s4:torture:smb2: fix error reporting in the oplock-brl3 test
    
    (Error was set to an unused variable)
    
    Autobuild-User(master): Michael Adam <obnox at samba.org>
    Autobuild-Date(master): Tue Sep 11 22:09:18 CEST 2012 on sn-devel-104

commit 439c907f6b2b02fefa5f1913cbbb8ce971d2611a
Author: Michael Adam <obnox at samba.org>
Date:   Tue Sep 11 16:28:15 2012 +0200

    s4:torture:smb2: fix error reporting in the oplock-brl2 test
    
    (Error was set to an unused variable)

commit e9df135ca52e07af1539cf15e6339d30ae525225
Author: Michael Adam <obnox at samba.org>
Date:   Tue Sep 11 16:27:19 2012 +0200

    s4:torture:smb2: fix error reporting in the oplock-brl1 test
    
    (Error was set to an unused variable)

commit c960e55fbfbc12f85447af04778a7ac5888d499d
Author: Michael Adam <obnox at samba.org>
Date:   Tue Sep 11 16:26:44 2012 +0200

    s4:torture:smb2: remove an unused variable from the oplock-exclusive2 test

commit 7c6f1fb294635f25e5e70592f37871902c7525ca
Author: Michael Adam <obnox at samba.org>
Date:   Tue Sep 11 16:25:50 2012 +0200

    s4:torture:smb2: remove an unused variable from the oplock-exclusive1 test

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

Summary of changes:
 source4/torture/smb2/oplock.c |   15 +++------------
 1 files changed, 3 insertions(+), 12 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/torture/smb2/oplock.c b/source4/torture/smb2/oplock.c
index d489943..fc939cc 100644
--- a/source4/torture/smb2/oplock.c
+++ b/source4/torture/smb2/oplock.c
@@ -364,7 +364,6 @@ static bool test_smb2_oplock_exclusive1(struct torture_context *tctx,
 	NTSTATUS status;
 	bool ret = true;
 	union smb_open io;
-	union smb_unlink unl;
 	struct smb2_handle h1;
 	struct smb2_handle h;
 
@@ -411,8 +410,6 @@ static bool test_smb2_oplock_exclusive1(struct torture_context *tctx,
 	CHECK_VAL(break_info.failures, 0);
 
 	torture_comment(tctx, "unlink it - should also be no break\n");
-	unl.unlink.in.pattern = fname;
-	unl.unlink.in.attrib = 0;
 	status = smb2_util_unlink(tree2, fname);
 	torture_assert_ntstatus_equal(tctx, status, NT_STATUS_SHARING_VIOLATION,
 				      "Incorrect status");
@@ -435,7 +432,6 @@ static bool test_smb2_oplock_exclusive2(struct torture_context *tctx,
 	NTSTATUS status;
 	bool ret = true;
 	union smb_open io;
-	union smb_unlink unl;
 	struct smb2_handle h, h1, h2;
 
 	status = torture_smb2_testdir(tree1, BASEDIR, &h);
@@ -490,8 +486,6 @@ static bool test_smb2_oplock_exclusive2(struct torture_context *tctx,
 
 	/* now we have 2 level II oplocks... */
 	torture_comment(tctx, "try to unlink it - should cause a break\n");
-	unl.unlink.in.pattern = fname;
-	unl.unlink.in.attrib = 0;
 	status = smb2_util_unlink(tree2, fname);
 	torture_assert_ntstatus_ok(tctx, status, "Error unlinking the file");
 	torture_wait_for_oplock_break(tctx);
@@ -2973,7 +2967,6 @@ static bool test_smb2_oplock_brl1(struct torture_context *tctx,
 	/*int fname, f;*/
 	bool ret = true;
 	uint8_t buf[1000];
-	bool correct = true;
 	union smb_open io;
 	NTSTATUS status;
 	struct smb2_lock lck;
@@ -3025,7 +3018,7 @@ static bool test_smb2_oplock_brl1(struct torture_context *tctx,
 	status = smb2_util_write(tree1, h1,buf, 0, sizeof(buf));
 	if (!NT_STATUS_EQUAL(status, NT_STATUS_OK)) {
 		torture_comment(tctx, "Failed to create file\n");
-		correct = false;
+		ret = false;
 		goto done;
 	}
 
@@ -3096,7 +3089,6 @@ static bool test_smb2_oplock_brl2(struct torture_context *tctx, struct smb2_tree
 	/*int fname, f;*/
 	bool ret = true;
 	uint8_t buf[1000];
-	bool correct = true;
 	union smb_open io;
 	NTSTATUS status;
 	struct smb2_handle h, h1;
@@ -3148,7 +3140,7 @@ static bool test_smb2_oplock_brl2(struct torture_context *tctx, struct smb2_tree
 	status = smb2_util_write(tree1, h1, buf, 0, sizeof(buf));
 	if (!NT_STATUS_EQUAL(status, NT_STATUS_OK)) {
 		torture_comment(tctx, "Failed to create file\n");
-		correct = false;
+		ret = false;
 		goto done;
 	}
 
@@ -3198,7 +3190,6 @@ static bool test_smb2_oplock_brl3(struct torture_context *tctx, struct smb2_tree
 	const char *fname = BASEDIR "\\test_batch_brl.dat";
 	bool ret = true;
 	uint8_t buf[1000];
-	bool correct = true;
 	union smb_open io;
 	NTSTATUS status;
 	struct smb2_handle h, h1, h2;
@@ -3249,7 +3240,7 @@ static bool test_smb2_oplock_brl3(struct torture_context *tctx, struct smb2_tree
 
 	if (!NT_STATUS_EQUAL(status, NT_STATUS_OK)) {
 		torture_comment(tctx, "Failed to create file\n");
-		correct = false;
+		ret = false;
 		goto done;
 	}
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list