s4:torture/rpc: make use of dcerpc_binding_set_flags()
authorStefan Metzmacher <metze@samba.org>
Tue, 4 Feb 2014 09:05:29 +0000 (10:05 +0100)
committerGünther Deschner <gd@samba.org>
Thu, 13 Feb 2014 10:54:18 +0000 (11:54 +0100)
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
source4/torture/rpc/backupkey.c
source4/torture/rpc/dsgetinfo.c
source4/torture/rpc/session_key.c

index 1c8f07f093604607965aeb10cf2a6b7120c2187a..997f828f24d488d426ce439b52cb702219c45691 100644 (file)
@@ -510,7 +510,10 @@ static struct bkrp_BackupKey *createRetreiveBackupKeyGUIDStruct(struct torture_c
                return NULL;
        }
 
-       binding->flags = binding->flags & (DCERPC_SEAL|DCERPC_AUTH_SPNEGO);
+       status = dcerpc_binding_set_flags(binding, DCERPC_SEAL|DCERPC_AUTH_SPNEGO, 0);
+       if (!NT_STATUS_IS_OK(status)) {
+               return NULL;
+       }
 
        ZERO_STRUCT(data);
        status = GUID_from_string(BACKUPKEY_RETRIEVE_BACKUP_KEY_GUID, g);
index df640e3bcc277d65759ea6a70b860fc16b431488..b622f60f92b8f260b0faa79590f0bf11334a3d1d 100644 (file)
@@ -127,7 +127,11 @@ static struct DsGetinfoTest *test_create_context(struct torture_context *tctx)
                printf("Bad binding string %s\n", binding);
                return NULL;
        }
-       ctx->drsuapi_binding->flags |= DCERPC_SIGN | DCERPC_SEAL;
+       status = dcerpc_binding_set_flags(ctx->drsuapi_binding, DCERPC_SIGN | DCERPC_SEAL, 0);
+       if (!NT_STATUS_IS_OK(status)) {
+               printf("dcerpc_binding_set_flags - %s\n", nt_errstr(status));
+               return NULL;
+       }
 
        /* ctx->admin ...*/
        ctx->admin.credentials                          = cmdline_credentials;
index 8e3f14021155f202880df64d022311706ea89ffc..11f6a0b994915debffc3dd415285acc42b66048a 100644 (file)
@@ -161,7 +161,8 @@ static bool test_secrets(struct torture_context *torture, const void *_data)
        torture_assert_ntstatus_ok(torture, torture_rpc_binding(torture, &binding), 
                                   "Getting bindoptions");
 
-       binding->flags |= settings->bindoptions;
+       status = dcerpc_binding_set_flags(binding, settings->bindoptions, 0);
+       torture_assert_ntstatus_ok(torture, status, "dcerpc_binding_set_flags");
 
        status = dcerpc_pipe_connect_b(torture, &p, binding,
                                       &ndr_table_lsarpc,