r11967: Fix more 64-bit warnings.
[bbaumbach/samba-autobuild/.git] / source4 / torture / rpc / samsync.c
index e393cd40fe7e67df27e3faaea8d92b8ed305a3e3..ad29d96b7121597932f96f958766a29d12ebe96b 100644 (file)
@@ -30,7 +30,8 @@
 #include "system/time.h"
 
 #define TEST_MACHINE_NAME "samsynctest"
-#define TEST_MACHINE_NAME2 "samsynctest2"
+#define TEST_WKSTA_MACHINE_NAME "samsynctest2"
+#define TEST_USER_NAME "samsynctestuser"
 
 /*
   try a netlogon SamLogon
@@ -135,15 +136,15 @@ static struct policy_handle *samsync_open_domain(TALLOC_CTX *mem_ctx,
                                                 const char *domain, 
                                                 struct dom_sid **sid)
 {
-       struct samr_String name;
+       struct lsa_String name;
        struct samr_OpenDomain o;
        struct samr_LookupDomain l;
-       struct policy_handle *domain_handle = talloc_p(mem_ctx, struct policy_handle);
+       struct policy_handle *domain_handle = talloc(mem_ctx, struct policy_handle);
        NTSTATUS nt_status;
 
        name.string = domain;
        l.in.connect_handle = samsync_state->connect_handle;
-       l.in.domain = &name;
+       l.in.domain_name = &name;
 
        nt_status = dcerpc_samr_LookupDomain(samsync_state->p_samr, mem_ctx, &l);
        if (!NT_STATUS_IS_OK(nt_status)) {
@@ -209,23 +210,24 @@ static struct sec_desc_buf *samsync_query_lsa_sec_desc(TALLOC_CTX *mem_ctx,
 
 #define TEST_UINT64_EQUAL(i1, i2) do {\
        if (i1 != i2) {\
-              printf("uint64 mismatch: " #i1 ": 0x%08x%08x (%lld) != " #i2 ": 0x%08x%08x (%lld)\n", \
-                    (uint32_t)(i1 >> 32), (uint32_t)(i1 & 0xFFFFFFFF), i1, \
-                     (uint32_t)(i2 >> 32), (uint32_t)(i2 & 0xFFFFFFFF), i2);\
+              printf("%s: uint64 mismatch: " #i1 ": 0x%016llx (%lld) != " #i2 ": 0x%016llx (%lld)\n", \
+                    __location__, \
+                    (long long)i1, (long long)i1, \
+                    (long long)i2, (long long)i2);\
              ret = False;\
        } \
 } while (0)
 #define TEST_INT_EQUAL(i1, i2) do {\
        if (i1 != i2) {\
-             printf("integer mismatch: " #i1 ":%d != " #i2 ": %d\n", \
-                    i1, i2);\
+             printf("%s: integer mismatch: " #i1 ":%d != " #i2 ": %d\n", \
+                    __location__, i1, i2);\
              ret = False;\
        } \
 } while (0)
 #define TEST_TIME_EQUAL(t1, t2) do {\
        if (t1 != t2) {\
-             printf("NTTIME mismatch: " #t1 ":%s != " #t2 ": %s\n", \
-                    nt_time_string(mem_ctx, t1),  nt_time_string(mem_ctx, t2));\
+             printf("%s: NTTIME mismatch: " #t1 ":%s != " #t2 ": %s\n", \
+                    __location__, nt_time_string(mem_ctx, t1),  nt_time_string(mem_ctx, t2));\
              ret = False;\
        } \
 } while (0)
@@ -233,16 +235,16 @@ static struct sec_desc_buf *samsync_query_lsa_sec_desc(TALLOC_CTX *mem_ctx,
 #define TEST_STRING_EQUAL(s1, s2) do {\
        if (!((!s1.string || s1.string[0]=='\0') && (!s2.string || s2.string[0]=='\0')) \
            && strcmp_safe(s1.string, s2.string) != 0) {\
-             printf("string mismatch: " #s1 ":%s != " #s2 ": %s\n", \
-                    s1.string, s2.string);\
+             printf("%s: string mismatch: " #s1 ":%s != " #s2 ": %s\n", \
+                    __location__, s1.string, s2.string);\
              ret = False;\
        } \
 } while (0)
 
 #define TEST_SID_EQUAL(s1, s2) do {\
        if (!dom_sid_equal(s1, s2)) {\
-             printf("dom_sid mismatch: " #s1 ":%s != " #s2 ": %s\n", \
-                    dom_sid_string(mem_ctx, s1), dom_sid_string(mem_ctx, s2));\
+             printf("%s: dom_sid mismatch: " #s1 ":%s != " #s2 ": %s\n", \
+                    __location__, dom_sid_string(mem_ctx, s1), dom_sid_string(mem_ctx, s2));\
              ret = False;\
        } \
 } while (0)
@@ -284,7 +286,7 @@ static BOOL samsync_handle_domain(TALLOC_CTX *mem_ctx, struct samsync_state *sam
        case SAM_DATABASE_DOMAIN:
                break;
        case SAM_DATABASE_BUILTIN:
-               if (StrCaseCmp("BUILTIN", domain->domain_name.string) != 0) {
+               if (strcasecmp_m("BUILTIN", domain->domain_name.string) != 0) {
                        printf("BUILTIN domain has different name: %s\n", domain->domain_name.string);
                }
                break;
@@ -298,7 +300,7 @@ static BOOL samsync_handle_domain(TALLOC_CTX *mem_ctx, struct samsync_state *sam
                samsync_state->domain_name[database_id] = 
                        talloc_reference(samsync_state, domain->domain_name.string);
        } else {
-               if (StrCaseCmp(samsync_state->domain_name[database_id], domain->domain_name.string) != 0) {
+               if (strcasecmp_m(samsync_state->domain_name[database_id], domain->domain_name.string) != 0) {
                        printf("Domain has name varies!: %s != %s\n", samsync_state->domain_name[database_id], 
                               domain->domain_name.string);
                        return False;
@@ -307,16 +309,18 @@ static BOOL samsync_handle_domain(TALLOC_CTX *mem_ctx, struct samsync_state *sam
 
        if (!samsync_state->domain_handle[database_id]) {
                samsync_state->domain_handle[database_id]
-                       = samsync_open_domain(mem_ctx, samsync_state, samsync_state->domain_name[database_id], 
-                                             &dom_sid);
+                       = talloc_reference(samsync_state, 
+                                          samsync_open_domain(mem_ctx, samsync_state, samsync_state->domain_name[database_id], 
+                                                              &dom_sid));
        }
        if (samsync_state->domain_handle[database_id]) {
                samsync_state->sid[database_id] = talloc_reference(samsync_state, dom_sid);
+               talloc_reference(dom_sid, dom_sid->sub_auths);
        }
 
        printf("\tsequence_nums[%d/%s]=%llu\n",
               database_id, domain->domain_name.string,
-              samsync_state->seq_num[database_id]);
+              (long long)samsync_state->seq_num[database_id]);
 
        for (i=0;i<ARRAY_SIZE(levels);i++) {
                q[levels[i]].in.domain_handle = samsync_state->domain_handle[database_id];
@@ -367,7 +371,7 @@ static BOOL samsync_handle_policy(TALLOC_CTX *mem_ctx, struct samsync_state *sam
                samsync_state->domain_name[SAM_DATABASE_DOMAIN] = 
                        talloc_reference(samsync_state, policy->primary_domain_name.string);
        } else {
-               if (StrCaseCmp(samsync_state->domain_name[SAM_DATABASE_DOMAIN], policy->primary_domain_name.string) != 0) {
+               if (strcasecmp_m(samsync_state->domain_name[SAM_DATABASE_DOMAIN], policy->primary_domain_name.string) != 0) {
                        printf("PRIMARY domain has name varies between DOMAIN and POLICY!: %s != %s\n", samsync_state->domain_name[SAM_DATABASE_DOMAIN], 
                               policy->primary_domain_name.string);
                        return False;
@@ -382,14 +386,14 @@ static BOOL samsync_handle_policy(TALLOC_CTX *mem_ctx, struct samsync_state *sam
 
        printf("\tsequence_nums[%d/PRIVS]=%llu\n",
               database_id, 
-              samsync_state->seq_num[database_id]);
+              (long long)samsync_state->seq_num[database_id]);
        return True;
 }
 
 static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsync_state,
                                int database_id, struct netr_DELTA_ENUM *delta) 
 {
-       uint32 rid = delta->delta_id_union.rid;
+       uint32_t rid = delta->delta_id_union.rid;
        struct netr_DELTA_USER *user = delta->delta_union.user;
        struct netr_SamInfo3 *info3;
        struct samr_Password lm_hash;
@@ -405,6 +409,7 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
        struct samr_QueryUserInfo q;
        struct policy_handle user_handle;
 
+       struct samr_GetGroupsForUser getgroups;
        if (!samsync_state->domain_name || !samsync_state->domain_handle[database_id]) {
                printf("SamSync needs domain information before the users\n");
                return False;
@@ -427,7 +432,27 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
        TEST_SEC_DESC_EQUAL(user->sdbuf, samr, &user_handle);
 
        nt_status = dcerpc_samr_QueryUserInfo(samsync_state->p_samr, mem_ctx, &q);
+       if (!NT_STATUS_IS_OK(nt_status)) {
+               printf("QueryUserInfo level %u failed - %s\n", 
+                      q.in.level, nt_errstr(nt_status));
+               ret = False;
+       }
+
+       getgroups.in.user_handle = &user_handle;
+       
+       nt_status = dcerpc_samr_GetGroupsForUser(samsync_state->p_samr, mem_ctx, &getgroups);
+       if (!NT_STATUS_IS_OK(nt_status)) {
+               printf("GetGroupsForUser failed - %s\n",
+                      nt_errstr(nt_status));
+               ret = False;
+       }
+
        if (!test_samr_handle_Close(samsync_state->p_samr, mem_ctx, &user_handle)) {
+               printf("samr_handle_Close failed - %s\n", 
+                      nt_errstr(nt_status));
+               ret = False;
+       }
+       if (!ret) {
                return False;
        }
 
@@ -499,10 +524,6 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
                data.data = user->user_private_info.SensitiveData;
                data.length = user->user_private_info.DataLength;
                creds_arcfour_crypt(samsync_state->creds, data.data, data.length);
-#if 0          
-               printf("Sensitive Data for %s:\n", username);
-               dump_data(0, data.data, data.length);
-#endif
                nt_status = ndr_pull_struct_blob(&data, mem_ctx, &keys, (ndr_pull_flags_fn_t)ndr_pull_netr_USER_KEYS);
                if (NT_STATUS_IS_OK(nt_status)) {
                        if (keys.keys.keys2.lmpassword.length == 16) {
@@ -513,14 +534,28 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
                                sam_rid_crypt(rid, keys.keys.keys2.ntpassword.pwd.hash, nt_hash.hash, 0);
                                nt_hash_p = &nt_hash;
                        }
+               } else {
+                       printf("Failed to parse Sensitive Data for %s:\n", username);
+#if 0
+                       dump_data(0, data.data, data.length);
+#endif
+                       return False;
                }
-               
        }
 
-       nt_status = test_SamLogon(samsync_state->p, mem_ctx, samsync_state->creds, 
+       if (nt_hash_p) {
+               DATA_BLOB nt_hash_blob = data_blob_const(nt_hash_p, 16);
+               DEBUG(100,("ACCOUNT [%s\\%-25s] NTHASH %s\n", samsync_state->domain_name[0], username, data_blob_hex_string(mem_ctx, &nt_hash_blob)));
+       }
+       if (lm_hash_p) {
+               DATA_BLOB lm_hash_blob = data_blob_const(lm_hash_p, 16);
+               DEBUG(100,("ACCOUNT [%s\\%-25s] LMHASH %s\n", samsync_state->domain_name[0], username, data_blob_hex_string(mem_ctx, &lm_hash_blob)));
+       }
+
+       nt_status = test_SamLogon(samsync_state->p_netlogon_wksta, mem_ctx, samsync_state->creds_netlogon_wksta, 
                                  domain,
                                  username, 
-                                 TEST_MACHINE_NAME,
+                                 TEST_WKSTA_MACHINE_NAME,
                                  lm_hash_p,
                                  nt_hash_p,
                                  &info3);
@@ -553,6 +588,9 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
                if (!lm_hash_p && !nt_hash_p) {
                        return True;
                }
+       } else if (NT_STATUS_EQUAL(nt_status, NT_STATUS_PASSWORD_MUST_CHANGE)) {
+               /* We would need to know the server's current time to test this properly */
+               return True;
        } else if (NT_STATUS_IS_OK(nt_status)) {
                TEST_INT_EQUAL(user->rid, info3->base.rid);
                TEST_INT_EQUAL(user->primary_gid, info3->base.primary_gid);
@@ -564,7 +602,10 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
                if (info3->base.account_name.string) {
                        TEST_STRING_EQUAL(user->account_name, info3->base.account_name);
                }
-               TEST_STRING_EQUAL(user->full_name, info3->base.full_name);
+               /* this is NULL from Win2k3 */
+               if (info3->base.full_name.string) {
+                       TEST_STRING_EQUAL(user->full_name, info3->base.full_name);
+               }
                TEST_STRING_EQUAL(user->logon_script, info3->base.logon_script);
                TEST_STRING_EQUAL(user->profile_path, info3->base.profile_path);
                TEST_STRING_EQUAL(user->home_directory, info3->base.home_directory);
@@ -586,6 +627,32 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
                      || (info3->base.last_logoff == 0x7fffffffffffffffLL))) {
                        TEST_TIME_EQUAL(user->last_logoff, info3->base.last_logoff);
                }
+
+               TEST_INT_EQUAL(getgroups.out.rids->count, info3->base.groups.count);
+               if (getgroups.out.rids->count == info3->base.groups.count) {
+                       int i, j;
+                       int count = getgroups.out.rids->count;
+                       BOOL *matched = talloc_zero_array(mem_ctx, BOOL, getgroups.out.rids->count);
+                               
+                       for (i = 0; i < count; i++) {
+                               for (j = 0; j < count; j++) {
+                                       if ((getgroups.out.rids->rids[i].rid == 
+                                            info3->base.groups.rids[j].rid)
+                                           && (getgroups.out.rids->rids[i].attributes == 
+                                               info3->base.groups.rids[j].attributes)) {
+                                                       matched[i] = True;
+                                               }
+                               }
+                       }
+
+                       for (i = 0; i < getgroups.out.rids->count; i++) {
+                               if (matched[i] == False) {
+                                       ret = False;
+                                       printf("Could not find group RID %u found in getgroups in NETLOGON reply\n",
+                                              getgroups.out.rids->rids[i].rid); 
+                               }
+                       }
+               }
                return ret;
        } else {
                printf("Could not validate password for user %s\\%s: %s\n",
@@ -598,7 +665,7 @@ static BOOL samsync_handle_user(TALLOC_CTX *mem_ctx, struct samsync_state *samsy
 static BOOL samsync_handle_alias(TALLOC_CTX *mem_ctx, struct samsync_state *samsync_state,
                                 int database_id, struct netr_DELTA_ENUM *delta) 
 {
-       uint32 rid = delta->delta_id_union.rid;
+       uint32_t rid = delta->delta_id_union.rid;
        struct netr_DELTA_ALIAS *alias = delta->delta_union.alias;
        NTSTATUS nt_status;
        BOOL ret = True;
@@ -647,7 +714,7 @@ static BOOL samsync_handle_alias(TALLOC_CTX *mem_ctx, struct samsync_state *sams
 static BOOL samsync_handle_group(TALLOC_CTX *mem_ctx, struct samsync_state *samsync_state,
                                 int database_id, struct netr_DELTA_ENUM *delta) 
 {
-       uint32 rid = delta->delta_id_union.rid;
+       uint32_t rid = delta->delta_id_union.rid;
        struct netr_DELTA_GROUP *group = delta->delta_union.group;
        NTSTATUS nt_status;
        BOOL ret = True;
@@ -699,8 +766,8 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
 {
        struct netr_DELTA_SECRET *secret = delta->delta_union.secret;
        const char *name = delta->delta_id_union.name;
-       struct samsync_secret *new = talloc_p(samsync_state, struct samsync_secret);
-       struct samsync_secret *old = talloc_p(mem_ctx, struct samsync_secret);
+       struct samsync_secret *new = talloc(samsync_state, struct samsync_secret);
+       struct samsync_secret *old = talloc(mem_ctx, struct samsync_secret);
        struct lsa_QuerySecret q;
        struct lsa_OpenSecret o;
        struct policy_handle sec_handle;
@@ -722,6 +789,7 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
        new->secret = data_blob_talloc(new, secret->current_cipher.cipher_data, secret->current_cipher.maxlen);
        new->mtime = secret->current_cipher_set_time;
 
+       new = talloc_reference(samsync_state, new);
        DLIST_ADD(samsync_state->secrets, new);
 
        old->name = talloc_reference(old, name);
@@ -754,7 +822,7 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
        ZERO_STRUCT(old_mtime);
 
        /* fetch the secret back again */
-       q.in.handle = &sec_handle;
+       q.in.sec_handle = &sec_handle;
        q.in.new_val = &bufp1;
        q.in.new_mtime = &new_mtime;
        q.in.old_val = &bufp2;
@@ -778,8 +846,12 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
                lsa_blob1.data = q.out.old_val->buf->data;
                lsa_blob1.length = q.out.old_val->buf->length;
 
-               lsa_blob_out = sess_decrypt_blob(mem_ctx, &lsa_blob1, &session_key);
-               
+               status = sess_decrypt_blob(mem_ctx, &lsa_blob1, &session_key, &lsa_blob_out);
+               if (!NT_STATUS_IS_OK(status)) {
+                       printf("Failed to decrypt secrets OLD blob: %s\n", nt_errstr(status));
+                       return False;
+               }
+
                if (!q.out.old_mtime) {
                        printf("OLD mtime not available on LSA for secret %s\n", old->name);
                        ret = False;
@@ -793,7 +865,7 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
 
                if (old->secret.length != lsa_blob_out.length) {
                        printf("Returned secret %s doesn't match: %d != %d\n",
-                              old->name, old->secret.length, lsa_blob_out.length);
+                              old->name, (int)old->secret.length, (int)lsa_blob_out.length);
                        ret = False;
                } else if (memcmp(lsa_blob_out.data, 
                           old->secret.data, old->secret.length) != 0) {
@@ -814,7 +886,11 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
                lsa_blob1.data = q.out.new_val->buf->data;
                lsa_blob1.length = q.out.new_val->buf->length;
 
-               lsa_blob_out = sess_decrypt_blob(mem_ctx, &lsa_blob1, &session_key);
+               status = sess_decrypt_blob(mem_ctx, &lsa_blob1, &session_key, &lsa_blob_out);
+               if (!NT_STATUS_IS_OK(status)) {
+                       printf("Failed to decrypt secrets OLD blob\n");
+                       return False;
+               }
                
                if (!q.out.new_mtime) {
                        printf("NEW mtime not available on LSA for secret %s\n", new->name);
@@ -829,7 +905,7 @@ static BOOL samsync_handle_secret(TALLOC_CTX *mem_ctx, struct samsync_state *sam
 
                if (new->secret.length != lsa_blob_out.length) {
                        printf("Returned secret %s doesn't match: %d != %d\n",
-                              new->name, new->secret.length, lsa_blob_out.length);
+                              new->name, (int)new->secret.length, (int)lsa_blob_out.length);
                        ret = False;
                } else if (memcmp(lsa_blob_out.data, 
                           new->secret.data, new->secret.length) != 0) {
@@ -854,7 +930,7 @@ static BOOL samsync_handle_trusted_domain(TALLOC_CTX *mem_ctx, struct samsync_st
        struct netr_DELTA_TRUSTED_DOMAIN *trusted_domain = delta->delta_union.trusted_domain;
        struct dom_sid *dom_sid = delta->delta_id_union.sid;
 
-       struct samsync_trusted_domain *new = talloc_p(samsync_state, struct samsync_trusted_domain);
+       struct samsync_trusted_domain *new = talloc(samsync_state, struct samsync_trusted_domain);
        struct lsa_OpenTrustedDomain t;
        struct policy_handle trustdom_handle;
        struct lsa_QueryTrustedDomainInfo q;
@@ -897,11 +973,12 @@ static BOOL samsync_handle_trusted_domain(TALLOC_CTX *mem_ctx, struct samsync_st
                TEST_STRING_EQUAL(info[8]->full_info.info_ex.netbios_name, trusted_domain->domain_name);
        }
        TEST_STRING_EQUAL(info[1]->name.netbios_name, trusted_domain->domain_name);
-       TEST_INT_EQUAL(info[3]->flags.flags, trusted_domain->flags);
+       TEST_INT_EQUAL(info[3]->posix_offset.posix_offset, trusted_domain->posix_offset);
 /*
   We would like to do this, but it is NOT_SUPPORTED on win2k3
        TEST_SEC_DESC_EQUAL(trusted_domain->sdbuf, lsa, &trustdom_handle);
 */
+       new = talloc_reference(samsync_state, new);
        DLIST_ADD(samsync_state->trusted_domains, new);
 
        return ret;
@@ -947,11 +1024,15 @@ static BOOL samsync_handle_account(TALLOC_CTX *mem_ctx, struct samsync_state *sa
                return False;
        }
 
-       if (!e.out.privs) {
-               return account->privilege_entries != 0;
+       if ((account->privilege_entries && !e.out.privs)) {
+               printf("Account %s has privileges in SamSync, but not LSA\n",
+                      dom_sid_string(mem_ctx, dom_sid));
+               return False;
        }
 
-       if ((account->privilege_entries && !e.out.privs)) {
+       if (!account->privilege_entries && e.out.privs && e.out.privs->count) {
+               printf("Account %s has privileges in LSA, but not SamSync\n",
+                      dom_sid_string(mem_ctx, dom_sid));
                return False;
        }
 
@@ -995,8 +1076,9 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                              TALLOC_CTX *mem_ctx)
 {
        NTSTATUS status;
+       TALLOC_CTX *loop_ctx, *delta_ctx, *trustdom_ctx;
        struct netr_DatabaseSync r;
-       const uint32_t database_ids[] = {SAM_DATABASE_DOMAIN, SAM_DATABASE_BUILTIN, SAM_DATABASE_PRIVS}; 
+       const enum netr_SamDatabaseID database_ids[] = {SAM_DATABASE_DOMAIN, SAM_DATABASE_BUILTIN, SAM_DATABASE_PRIVS}; 
        int i, d;
        BOOL ret = True;
        struct samsync_trusted_domain *t;
@@ -1016,9 +1098,10 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                printf("Testing DatabaseSync of id %d\n", r.in.database_id);
 
                do {
+                       loop_ctx = talloc_named(mem_ctx, 0, "DatabaseSync loop context");
                        creds_client_authenticator(samsync_state->creds, &r.in.credential);
 
-                       status = dcerpc_netr_DatabaseSync(samsync_state->p, mem_ctx, &r);
+                       status = dcerpc_netr_DatabaseSync(samsync_state->p, loop_ctx, &r);
                        if (!NT_STATUS_IS_OK(status) &&
                            !NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES)) {
                                printf("DatabaseSync - %s\n", nt_errstr(status));
@@ -1033,41 +1116,85 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                        r.in.sync_context = r.out.sync_context;
 
                        for (d=0; d < r.out.delta_enum_array->num_deltas; d++) {
+                               delta_ctx = talloc_named(loop_ctx, 0, "DatabaseSync delta context");
                                switch (r.out.delta_enum_array->delta_enum[d].delta_type) {
                                case NETR_DELTA_DOMAIN:
-                                       ret &= samsync_handle_domain(mem_ctx, samsync_state, 
-                                                                    r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_domain(delta_ctx, samsync_state, 
+                                                                  r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_DOMAIN\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_GROUP:
-                                       ret &= samsync_handle_group(mem_ctx, samsync_state, 
-                                                                   r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_group(delta_ctx, samsync_state, 
+                                                                 r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_USER\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_USER:
-                                       ret &= samsync_handle_user(mem_ctx, samsync_state, 
-                                                                  r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_user(delta_ctx, samsync_state, 
+                                                                r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_USER\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_ALIAS:
-                                       ret &= samsync_handle_alias(mem_ctx, samsync_state, 
-                                                                   r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_alias(delta_ctx, samsync_state, 
+                                                                 r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_ALIAS\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_POLICY:
-                                       ret &= samsync_handle_policy(mem_ctx, samsync_state, 
-                                                                    r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_policy(delta_ctx, samsync_state, 
+                                                                  r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_POLICY\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_TRUSTED_DOMAIN:
-                                       ret &= samsync_handle_trusted_domain(mem_ctx, samsync_state, 
-                                                                            r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_trusted_domain(delta_ctx, samsync_state, 
+                                                                          r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_TRUSTED_DOMAIN\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_ACCOUNT:
-                                       ret &= samsync_handle_account(mem_ctx, samsync_state, 
-                                                                    r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_account(delta_ctx, samsync_state, 
+                                                                   r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_ACCOUNT\n");
+                                               ret = False;
+                                       }
                                        break;
                                case NETR_DELTA_SECRET:
-                                       ret &= samsync_handle_secret(mem_ctx, samsync_state, 
-                                                                    r.in.database_id, &r.out.delta_enum_array->delta_enum[d]);
+                                       if (!samsync_handle_secret(delta_ctx, samsync_state, 
+                                                                  r.in.database_id, &r.out.delta_enum_array->delta_enum[d])) {
+                                               printf("Failed to handle DELTA_SECRET\n");
+                                               ret = False;
+                                       }
+                                       break;
+                               case NETR_DELTA_DELETE_GROUP:
+                               case NETR_DELTA_RENAME_GROUP:
+                               case NETR_DELTA_DELETE_USER:
+                               case NETR_DELTA_RENAME_USER:
+                               case NETR_DELTA_GROUP_MEMBER:
+                               case NETR_DELTA_DELETE_ALIAS:
+                               case NETR_DELTA_RENAME_ALIAS:
+                               case NETR_DELTA_ALIAS_MEMBER:
+                               case NETR_DELTA_DELETE_TRUST:
+                               case NETR_DELTA_DELETE_ACCOUNT:
+                               case NETR_DELTA_DELETE_SECRET:
+                               case NETR_DELTA_DELETE_GROUP2:
+                               case NETR_DELTA_DELETE_USER2:
+                               case NETR_DELTA_MODIFY_COUNT:
+                                       printf("Unhandled delta type %d\n", r.out.delta_enum_array->delta_enum[d].delta_type);
+                                       ret = False;
                                        break;
                                }
+                               talloc_free(delta_ctx);
                        }
+                       talloc_free(loop_ctx);
                } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES));
                
        }
@@ -1077,21 +1204,23 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                printf("Never got a DOMAIN object in samsync!\n");
                return False;
        }
+
+       trustdom_ctx = talloc_named(mem_ctx, 0, "test_DatabaseSync Trusted domains context");
        
-       username = talloc_asprintf(mem_ctx, "%s$", domain);
+       username = talloc_asprintf(trustdom_ctx, "%s$", domain);
        for (t=samsync_state->trusted_domains; t; t=t->next) {
-               char *secret_name = talloc_asprintf(mem_ctx, "G$$%s", t->name);
+               char *secret_name = talloc_asprintf(trustdom_ctx, "G$$%s", t->name);
                for (s=samsync_state->secrets; s; s=s->next) {
-                       if (StrCaseCmp(s->name, secret_name) == 0) {
+                       if (strcasecmp_m(s->name, secret_name) == 0) {
                                NTSTATUS nt_status;
                                struct samr_Password nt_hash;
                                mdfour(nt_hash.hash, s->secret.data, s->secret.length);
                                
                                printf("Checking password for %s\\%s\n", t->name, username);
-                               nt_status = test_SamLogon(samsync_state->p_netlogon_wksta, mem_ctx, samsync_state->creds_netlogon_wksta, 
+                               nt_status = test_SamLogon(samsync_state->p_netlogon_wksta, trustdom_ctx, samsync_state->creds_netlogon_wksta, 
                                                          t->name,
                                                          username, 
-                                                         TEST_MACHINE_NAME2,
+                                                         TEST_WKSTA_MACHINE_NAME,
                                                          NULL, 
                                                          &nt_hash,
                                                          NULL);
@@ -1109,10 +1238,10 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                                
                                /* break it */
                                nt_hash.hash[0]++;
-                               nt_status = test_SamLogon(samsync_state->p_netlogon_wksta, mem_ctx, samsync_state->creds_netlogon_wksta, 
+                               nt_status = test_SamLogon(samsync_state->p_netlogon_wksta, trustdom_ctx, samsync_state->creds_netlogon_wksta, 
                                                          t->name,
                                                          username, 
-                                                         TEST_MACHINE_NAME2,
+                                                         TEST_WKSTA_MACHINE_NAME,
                                                          NULL,
                                                          &nt_hash,
                                                          NULL);
@@ -1121,13 +1250,13 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
                                        printf("Verifiction of trust password to %s: should have failed (wrong password), instead: %s\n", 
                                               t->name, nt_errstr(nt_status));
                                        ret = False;
-                                       ret = False;
                                }
                                
                                break;
                        }
                }
        }
+       talloc_free(trustdom_ctx);
        return ret;
 }
 
@@ -1138,6 +1267,7 @@ static BOOL test_DatabaseSync(struct samsync_state *samsync_state,
 static BOOL test_DatabaseDeltas(struct samsync_state *samsync_state, TALLOC_CTX *mem_ctx)
 {
        NTSTATUS status;
+       TALLOC_CTX *loop_ctx;
        struct netr_DatabaseDeltas r;
        const uint32_t database_ids[] = {0, 1, 2}; 
        int i;
@@ -1154,22 +1284,26 @@ static BOOL test_DatabaseDeltas(struct samsync_state *samsync_state, TALLOC_CTX
 
                if (r.in.sequence_num == 0) continue;
 
-               r.in.sequence_num -= 1;
+               /* this shows that the bdc doesn't need to do a single call for
+                * each seqnumber, and the pdc doesn't need to know about old values
+                * -- metze
+                */
+               r.in.sequence_num -= 10;
 
 
                printf("Testing DatabaseDeltas of id %d at %llu\n", 
                       r.in.database_id, r.in.sequence_num);
 
                do {
+                       loop_ctx = talloc_named(mem_ctx, 0, "test_DatabaseDeltas loop context");
                        creds_client_authenticator(samsync_state->creds, &r.in.credential);
 
-                       status = dcerpc_netr_DatabaseDeltas(samsync_state->p, mem_ctx, &r);
+                       status = dcerpc_netr_DatabaseDeltas(samsync_state->p, loop_ctx, &r);
                        if (!NT_STATUS_IS_OK(status) &&
                            !NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES) &&
                            !NT_STATUS_EQUAL(status, NT_STATUS_SYNCHRONIZATION_REQUIRED)) {
                                printf("DatabaseDeltas - %s\n", nt_errstr(status));
                                ret = False;
-                               break;
                        }
 
                        if (!creds_client_check(samsync_state->creds, &r.out.return_authenticator.cred)) {
@@ -1177,6 +1311,7 @@ static BOOL test_DatabaseDeltas(struct samsync_state *samsync_state, TALLOC_CTX
                        }
 
                        r.in.sequence_num++;
+                       talloc_free(loop_ctx);
                } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES));
        }
 
@@ -1191,6 +1326,7 @@ static BOOL test_DatabaseSync2(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                               struct creds_CredentialState *creds)
 {
        NTSTATUS status;
+       TALLOC_CTX *loop_ctx;
        struct netr_DatabaseSync2 r;
        const uint32_t database_ids[] = {0, 1, 2}; 
        int i;
@@ -1209,14 +1345,14 @@ static BOOL test_DatabaseSync2(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                printf("Testing DatabaseSync2 of id %d\n", r.in.database_id);
 
                do {
+                       loop_ctx = talloc_named(mem_ctx, 0, "test_DatabaseSync2 loop context");
                        creds_client_authenticator(creds, &r.in.credential);
 
-                       status = dcerpc_netr_DatabaseSync2(p, mem_ctx, &r);
+                       status = dcerpc_netr_DatabaseSync2(p, loop_ctx, &r);
                        if (!NT_STATUS_IS_OK(status) &&
                            !NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES)) {
                                printf("DatabaseSync2 - %s\n", nt_errstr(status));
                                ret = False;
-                               break;
                        }
 
                        if (!creds_client_check(creds, &r.out.return_authenticator.cred)) {
@@ -1224,6 +1360,7 @@ static BOOL test_DatabaseSync2(struct dcerpc_pipe *p, TALLOC_CTX *mem_ctx,
                        }
 
                        r.in.sync_context = r.out.sync_context;
+                       talloc_free(loop_ctx);
                } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES));
        }
 
@@ -1239,11 +1376,12 @@ BOOL torture_rpc_samsync(void)
        BOOL ret = True;
        struct test_join *join_ctx;
        struct test_join *join_ctx2;
+       struct test_join *user_ctx;
        const char *machine_password;
-       const char *machine_password2;
+       const char *wksta_machine_password;
        const char *binding = lp_parm_string(-1, "torture", "binding");
-       struct dcerpc_binding b;
-       struct dcerpc_binding b_netlogon_wksta;
+       struct dcerpc_binding *b;
+       struct dcerpc_binding *b_netlogon_wksta;
        struct samr_Connect c;
        struct samr_SetDomainInfo s;
        struct policy_handle *domain_policy;
@@ -1251,30 +1389,49 @@ BOOL torture_rpc_samsync(void)
        struct lsa_ObjectAttribute attr;
        struct lsa_QosInfo qos;
        struct lsa_OpenPolicy2 r;
+       struct cli_credentials *credentials;
+       struct cli_credentials *credentials_wksta;
 
        struct samsync_state *samsync_state;
 
-       mem_ctx = talloc_init("torture_rpc_netlogon");
+       char *test_machine_account;
+
+       char *test_wksta_machine_account;
 
-       join_ctx = torture_join_domain(TEST_MACHINE_NAME, lp_workgroup(), ACB_SVRTRUST, 
-                                      &machine_password);
+       mem_ctx = talloc_init("torture_rpc_netlogon");
+       
+       test_machine_account = talloc_asprintf(mem_ctx, "%s$", TEST_MACHINE_NAME);
+       join_ctx = torture_create_testuser(test_machine_account, lp_workgroup(), ACB_SVRTRUST, 
+                                          &machine_password);
        if (!join_ctx) {
+               talloc_free(mem_ctx);
                printf("Failed to join as BDC\n");
                return False;
        }
        
-       join_ctx2 = torture_join_domain(TEST_MACHINE_NAME2, lp_workgroup(), ACB_WSTRUST, 
-                                      &machine_password2);
+       test_wksta_machine_account = talloc_asprintf(mem_ctx, "%s$", TEST_WKSTA_MACHINE_NAME);
+       join_ctx2 = torture_create_testuser(test_wksta_machine_account, lp_workgroup(), ACB_WSTRUST, 
+                                           &wksta_machine_password);
        if (!join_ctx2) {
+               talloc_free(mem_ctx);
                printf("Failed to join as member\n");
                return False;
        }
        
-       samsync_state = talloc_zero_p(mem_ctx, struct samsync_state);
+       user_ctx = torture_create_testuser(TEST_USER_NAME,
+                                          lp_workgroup(),
+                                          ACB_NORMAL, NULL);
+       if (!user_ctx) {
+               talloc_free(mem_ctx);
+               printf("Failed to create test account\n");
+               return False;
+       }
+
+       samsync_state = talloc_zero(mem_ctx, struct samsync_state);
 
        samsync_state->p_samr = torture_join_samr_pipe(join_ctx);
-       samsync_state->connect_handle = talloc_zero_p(samsync_state, struct policy_handle);
-       samsync_state->lsa_handle = talloc_zero_p(samsync_state, struct policy_handle);
+       samsync_state->connect_handle = talloc_zero(samsync_state, struct policy_handle);
+       samsync_state->lsa_handle = talloc_zero(samsync_state, struct policy_handle);
        c.in.system_name = NULL;
        c.in.access_mask = SEC_FLAG_MAXIMUM_ALLOWED;
        c.out.connect_handle = samsync_state->connect_handle;
@@ -1295,7 +1452,7 @@ BOOL torture_rpc_samsync(void)
        
        s.in.domain_handle = domain_policy;
        s.in.level = 4;
-       s.in.info = talloc_p(mem_ctx, union samr_DomainInfo);
+       s.in.info = talloc(mem_ctx, union samr_DomainInfo);
        
        s.in.info->info4.comment.string
                = talloc_asprintf(mem_ctx, 
@@ -1316,7 +1473,8 @@ BOOL torture_rpc_samsync(void)
        }
        
 
-       status = torture_rpc_connection(&samsync_state->p_lsa, 
+       status = torture_rpc_connection(samsync_state,
+                                       &samsync_state->p_lsa, 
                                        DCERPC_LSARPC_NAME,
                                        DCERPC_LSARPC_UUID,
                                        DCERPC_LSARPC_VERSION);
@@ -1357,22 +1515,32 @@ BOOL torture_rpc_samsync(void)
                goto failed;
        }
 
-       b.flags &= ~DCERPC_AUTH_OPTIONS;
-       b.flags |= DCERPC_SCHANNEL_BDC | DCERPC_SIGN;
+       b->flags &= ~DCERPC_AUTH_OPTIONS;
+       b->flags |= DCERPC_SCHANNEL | DCERPC_SIGN;
+
+       credentials = cli_credentials_init(mem_ctx);
 
-       status = dcerpc_pipe_connect_b(&samsync_state->p, &b, 
+       cli_credentials_set_workstation(credentials, TEST_MACHINE_NAME, CRED_SPECIFIED);
+       cli_credentials_set_domain(credentials, lp_workgroup(), CRED_SPECIFIED);
+       cli_credentials_set_username(credentials, test_machine_account, CRED_SPECIFIED);
+       cli_credentials_set_password(credentials, machine_password, CRED_SPECIFIED);
+       cli_credentials_set_secure_channel_type(credentials,
+                                               SEC_CHAN_BDC);
+
+       status = dcerpc_pipe_connect_b(samsync_state,
+                                      &samsync_state->p, b, 
                                       DCERPC_NETLOGON_UUID,
                                       DCERPC_NETLOGON_VERSION,
-                                      lp_workgroup(), 
-                                      TEST_MACHINE_NAME,
-                                      machine_password);
-
+                                      credentials, NULL);
+       
        if (!NT_STATUS_IS_OK(status)) {
+               printf("Failed to connect to server as a BDC: %s\n", nt_errstr(status));
                ret = False;
                goto failed;
        }
 
-       status = dcerpc_schannel_creds(samsync_state->p->conn->security_state.generic_state, mem_ctx, &samsync_state->creds);
+       status = dcerpc_schannel_creds(samsync_state->p->conn->security_state.generic_state, 
+                                      samsync_state, &samsync_state->creds);
        if (!NT_STATUS_IS_OK(status)) {
                ret = False;
        }
@@ -1386,45 +1554,59 @@ BOOL torture_rpc_samsync(void)
                goto failed;
        }
 
-       b_netlogon_wksta.flags &= ~DCERPC_AUTH_OPTIONS;
-       b_netlogon_wksta.flags |= DCERPC_SCHANNEL_WORKSTATION | DCERPC_SIGN;
+       b_netlogon_wksta->flags &= ~DCERPC_AUTH_OPTIONS;
+       b_netlogon_wksta->flags |= DCERPC_SCHANNEL | DCERPC_SIGN;
+
+       credentials_wksta = cli_credentials_init(mem_ctx);
+
+       cli_credentials_set_workstation(credentials_wksta, TEST_WKSTA_MACHINE_NAME, CRED_SPECIFIED);
+       cli_credentials_set_domain(credentials_wksta, lp_workgroup(), CRED_SPECIFIED);
+       cli_credentials_set_username(credentials_wksta, test_wksta_machine_account, CRED_SPECIFIED);
+       cli_credentials_set_password(credentials_wksta, wksta_machine_password, CRED_SPECIFIED);
+       cli_credentials_set_secure_channel_type(credentials_wksta,
+                                               SEC_CHAN_WKSTA);
 
-       status = dcerpc_pipe_connect_b(&samsync_state->p_netlogon_wksta, &b_netlogon_wksta, 
+       status = dcerpc_pipe_connect_b(samsync_state, 
+                                      &samsync_state->p_netlogon_wksta, 
+                                      b_netlogon_wksta, 
                                       DCERPC_NETLOGON_UUID,
                                       DCERPC_NETLOGON_VERSION,
-                                      lp_workgroup(), 
-                                      TEST_MACHINE_NAME2,
-                                      machine_password2);
+                                      credentials_wksta, NULL);
 
        if (!NT_STATUS_IS_OK(status)) {
+               printf("Failed to connect to server as a Workstation: %s\n", nt_errstr(status));
                ret = False;
                goto failed;
        }
 
        status = dcerpc_schannel_creds(samsync_state->p_netlogon_wksta->conn->security_state.generic_state, 
-                                      mem_ctx, &samsync_state->creds_netlogon_wksta);
+                                      samsync_state, &samsync_state->creds_netlogon_wksta);
        if (!NT_STATUS_IS_OK(status)) {
+               printf("Failed to obtail schanel creds!\n");
                ret = False;
        }
 
        if (!test_DatabaseSync(samsync_state, mem_ctx)) {
+               printf("DatabaseSync failed\n");
                ret = False;
        }
 
        if (!test_DatabaseDeltas(samsync_state, mem_ctx)) {
+               printf("DatabaseDeltas failed\n");
                ret = False;
        }
 
        if (!test_DatabaseSync2(samsync_state->p, mem_ctx, samsync_state->creds)) {
+               printf("DatabaseSync2 failed\n");
                ret = False;
        }
 failed:
-       torture_rpc_close(samsync_state->p);
 
        torture_leave_domain(join_ctx);
        torture_leave_domain(join_ctx2);
+       torture_leave_domain(user_ctx);
 
-       talloc_destroy(mem_ctx);
+       talloc_free(mem_ctx);
 
        return ret;
 }