[SCM] Samba Shared Repository - branch master updated

Günther Deschner gd at samba.org
Fri May 16 15:49:04 MDT 2014


The branch, master has been updated
       via  93093fa s4-torture: fix some build warnings in rpc samr test.
       via  efff779 s4-torture: fix test_openprinter_wrap fake test result.
      from  b8e5b68 talloc: version 2.1.1

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


- Log -----------------------------------------------------------------
commit 93093fa08da355c6b154078cb219e9f9e781f2fe
Author: Günther Deschner <gd at samba.org>
Date:   Mon May 12 23:33:53 2014 +0200

    s4-torture: fix some build warnings in rpc samr test.
    
    Guenther
    
    Signed-off-by: Günther Deschner <gd at samba.org>
    Reviewed-by: Andreas Schneider <asn at samba.org>
    
    Autobuild-User(master): Günther Deschner <gd at samba.org>
    Autobuild-Date(master): Fri May 16 23:48:35 CEST 2014 on sn-devel-104

commit efff779c72feb0bf8bac3214e4132e2473bc01c8
Author: Günther Deschner <gd at samba.org>
Date:   Mon May 12 23:06:49 2014 +0200

    s4-torture: fix test_openprinter_wrap fake test result.
    
    Guenther
    
    Signed-off-by: Günther Deschner <gd at samba.org>
    Reviewed-by: Andreas Schneider <asn at samba.org>

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

Summary of changes:
 source4/torture/rpc/samr.c           |   21 ++++++++++-----------
 source4/torture/rpc/spoolss_access.c |    2 +-
 2 files changed, 11 insertions(+), 12 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/torture/rpc/samr.c b/source4/torture/rpc/samr.c
index 293f335..0cdd242 100644
--- a/source4/torture/rpc/samr.c
+++ b/source4/torture/rpc/samr.c
@@ -363,19 +363,19 @@ static bool test_SetUserInfo(struct dcerpc_binding_handle *b, struct torture_con
 			   SAMR_FIELD_COMMENT);
 
 	test_account_name = talloc_asprintf(tctx, "%sxx7-1", base_account_name);
-	TEST_USERINFO_STRING(7, account_name,  1, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name,  1, account_name, test_account_name, 0);
 	test_account_name = talloc_asprintf(tctx, "%sxx7-3", base_account_name);
-	TEST_USERINFO_STRING(7, account_name,  3, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name,  3, account_name, test_account_name, 0);
 	test_account_name = talloc_asprintf(tctx, "%sxx7-5", base_account_name);
-	TEST_USERINFO_STRING(7, account_name,  5, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name,  5, account_name, test_account_name, 0);
 	test_account_name = talloc_asprintf(tctx, "%sxx7-6", base_account_name);
-	TEST_USERINFO_STRING(7, account_name,  6, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name,  6, account_name, test_account_name, 0);
 	test_account_name = talloc_asprintf(tctx, "%sxx7-7", base_account_name);
-	TEST_USERINFO_STRING(7, account_name,  7, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name,  7, account_name, test_account_name, 0);
 	test_account_name = talloc_asprintf(tctx, "%sxx7-21", base_account_name);
-	TEST_USERINFO_STRING(7, account_name, 21, account_name, base_account_name, 0);
+	TEST_USERINFO_STRING(7, account_name, 21, account_name, test_account_name, 0);
 	test_account_name = base_account_name;
-	TEST_USERINFO_STRING(21, account_name, 21, account_name, base_account_name,
+	TEST_USERINFO_STRING(21, account_name, 21, account_name, test_account_name,
 			   SAMR_FIELD_ACCOUNT_NAME);
 
 	TEST_USERINFO_STRING(6, full_name,  1, full_name, "xx6-1 full_name", 0);
@@ -2303,7 +2303,6 @@ static bool test_ChangePasswordUser2_ntstatus(struct dcerpc_pipe *p, struct tort
 					      const char *password, NTSTATUS status)
 {
 	struct samr_ChangePasswordUser2 r;
-	bool ret = true;
 	struct lsa_String server, account;
 	struct samr_CryptPassword nt_pass, lm_pass;
 	struct samr_Password nt_verifier, lm_verifier;
@@ -2970,7 +2969,7 @@ static bool test_QueryUserInfo_pwdlastset(struct dcerpc_binding_handle *b,
 	NTSTATUS status;
 	uint16_t levels[] = { /* 3, */ 5, 21 };
 	int i;
-	NTTIME pwdlastset3 = 0;
+	/* NTTIME pwdlastset3 = 0; */
 	NTTIME pwdlastset5 = 0;
 	NTTIME pwdlastset21 = 0;
 
@@ -3009,7 +3008,7 @@ static bool test_QueryUserInfo_pwdlastset(struct dcerpc_binding_handle *b,
 
 		switch (levels[i]) {
 		case 3:
-			pwdlastset3 = info->info3.last_password_change;
+			/* pwdlastset3 = info->info3.last_password_change; */
 			break;
 		case 5:
 			pwdlastset5 = info->info5.last_password_change;
@@ -7252,7 +7251,7 @@ static bool test_QueryDomainInfo2(struct dcerpc_binding_handle *b,
 		}
 	}
 
-	return true;
+	return ret;
 }
 
 /* Test whether querydispinfo level 5 and enumdomgroups return the same
diff --git a/source4/torture/rpc/spoolss_access.c b/source4/torture/rpc/spoolss_access.c
index 411dac1..28cecf0 100644
--- a/source4/torture/rpc/spoolss_access.c
+++ b/source4/torture/rpc/spoolss_access.c
@@ -845,7 +845,7 @@ static bool test_openprinter_wrap(struct torture_context *tctx,
 
 	talloc_free(t);
 
-	return true;
+	return ret;
 }
 
 struct torture_suite *torture_rpc_spoolss_access(TALLOC_CTX *mem_ctx)


-- 
Samba Shared Repository


More information about the samba-cvs mailing list