summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGünther Deschner <gd@samba.org>2010-03-19 13:55:35 +0100
committerStefan Metzmacher <metze@samba.org>2010-03-29 15:35:47 +0200
commita154bb56297c7bdfe7d7505afc409e0333100770 (patch)
tree1ceec3dc773993edf44ece050d9d2c044340b60a
parent959a128c7b9303b59e92273b01fefde02d752015 (diff)
downloadsamba-a154bb56297c7bdfe7d7505afc409e0333100770.tar.gz
samba-a154bb56297c7bdfe7d7505afc409e0333100770.tar.bz2
samba-a154bb56297c7bdfe7d7505afc409e0333100770.zip
s4:torture/rpc/testjoin.c: add explicit check for NTSTATUS r.out.result
Guenther Signed-off-by: Stefan Metzmacher <metze@samba.org>
-rw-r--r--source4/torture/rpc/testjoin.c53
1 files changed, 47 insertions, 6 deletions
diff --git a/source4/torture/rpc/testjoin.c b/source4/torture/rpc/testjoin.c
index 519f7fcc9c..919ba207ff 100644
--- a/source4/torture/rpc/testjoin.c
+++ b/source4/torture/rpc/testjoin.c
@@ -71,10 +71,13 @@ static NTSTATUS DeleteUser_byname(struct dcerpc_binding_handle *b,
n.out.types = &types;
status = dcerpc_samr_LookupNames_r(b, mem_ctx, &n);
- if (NT_STATUS_IS_OK(status)) {
+ if (!NT_STATUS_IS_OK(status)) {
+ return status;
+ }
+ if (NT_STATUS_IS_OK(n.out.result)) {
rid = n.out.rids->ids[0];
} else {
- return status;
+ return n.out.result;
}
r.in.domain_handle = handle;
@@ -87,6 +90,10 @@ static NTSTATUS DeleteUser_byname(struct dcerpc_binding_handle *b,
printf("OpenUser(%s) failed - %s\n", name, nt_errstr(status));
return status;
}
+ if (!NT_STATUS_IS_OK(r.out.result)) {
+ printf("OpenUser(%s) failed - %s\n", name, nt_errstr(r.out.result));
+ return r.out.result;
+ }
d.in.user_handle = &user_handle;
d.out.user_handle = &user_handle;
@@ -94,6 +101,9 @@ static NTSTATUS DeleteUser_byname(struct dcerpc_binding_handle *b,
if (!NT_STATUS_IS_OK(status)) {
return status;
}
+ if (!NT_STATUS_IS_OK(d.out.result)) {
+ return d.out.result;
+ }
return NT_STATUS_OK;
}
@@ -172,6 +182,11 @@ struct test_join *torture_create_testuser(struct torture_context *torture,
printf("samr_Connect failed - %s\n", errstr);
return NULL;
}
+ if (!NT_STATUS_IS_OK(c.out.result)) {
+ const char *errstr = nt_errstr(c.out.result);
+ printf("samr_Connect failed - %s\n", errstr);
+ return NULL;
+ }
printf("Opening domain %s\n", domain);
@@ -185,6 +200,10 @@ struct test_join *torture_create_testuser(struct torture_context *torture,
printf("LookupDomain failed - %s\n", nt_errstr(status));
goto failed;
}
+ if (!NT_STATUS_IS_OK(l.out.result)) {
+ printf("LookupDomain failed - %s\n", nt_errstr(l.out.result));
+ goto failed;
+ }
talloc_steal(join, *l.out.sid);
join->dom_sid = *l.out.sid;
@@ -201,6 +220,10 @@ struct test_join *torture_create_testuser(struct torture_context *torture,
printf("OpenDomain failed - %s\n", nt_errstr(status));
goto failed;
}
+ if (!NT_STATUS_IS_OK(o.out.result)) {
+ printf("OpenDomain failed - %s\n", nt_errstr(o.out.result));
+ goto failed;
+ }
printf("Creating account %s\n", username);
@@ -215,16 +238,20 @@ again:
r.out.rid = &rid;
status = dcerpc_samr_CreateUser2_r(b, join, &r);
+ if (!NT_STATUS_IS_OK(status)) {
+ printf("CreateUser2 failed - %s\n", nt_errstr(status));
+ goto failed;
+ }
- if (NT_STATUS_EQUAL(status, NT_STATUS_USER_EXISTS)) {
+ if (NT_STATUS_EQUAL(r.out.result, NT_STATUS_USER_EXISTS)) {
status = DeleteUser_byname(b, join, &domain_handle, name.string);
if (NT_STATUS_IS_OK(status)) {
goto again;
}
}
- if (!NT_STATUS_IS_OK(status)) {
- printf("CreateUser2 failed - %s\n", nt_errstr(status));
+ if (!NT_STATUS_IS_OK(r.out.result)) {
+ printf("CreateUser2 failed - %s\n", nt_errstr(r.out.result));
goto failed;
}
@@ -234,7 +261,7 @@ again:
pwp.out.info = &info;
status = dcerpc_samr_GetUserPwInfo_r(b, join, &pwp);
- if (NT_STATUS_IS_OK(status)) {
+ if (NT_STATUS_IS_OK(status) && NT_STATUS_IS_OK(pwp.out.result)) {
policy_min_pw_len = pwp.out.info->min_password_length;
}
@@ -265,6 +292,10 @@ again:
printf("SetUserInfo failed - %s\n", nt_errstr(status));
goto failed;
}
+ if (!NT_STATUS_IS_OK(s.out.result)) {
+ printf("SetUserInfo failed - %s\n", nt_errstr(s.out.result));
+ goto failed;
+ }
ZERO_STRUCT(u);
s.in.user_handle = &join->user_handle;
@@ -294,6 +325,10 @@ again:
printf("SetUserInfo failed - %s\n", nt_errstr(status));
goto failed;
}
+ if (!NT_STATUS_IS_OK(s.out.result)) {
+ printf("SetUserInfo failed - %s\n", nt_errstr(s.out.result));
+ goto failed;
+ }
if (random_password) {
*random_password = random_pw;
@@ -398,6 +433,9 @@ _PUBLIC_ struct test_join *torture_join_domain(struct torture_context *tctx,
if (!NT_STATUS_IS_OK(status)) {
printf("SetUserInfo (non-critical) failed - %s\n", nt_errstr(status));
}
+ if (!NT_STATUS_IS_OK(s.out.result)) {
+ printf("SetUserInfo (non-critical) failed - %s\n", nt_errstr(s.out.result));
+ }
*machine_credentials = cli_credentials_init(tj);
cli_credentials_set_conf(*machine_credentials, tctx->lp_ctx);
@@ -520,6 +558,9 @@ _PUBLIC_ void torture_leave_domain(struct torture_context *torture, struct test_
/* Delete machine account */
status = dcerpc_samr_DeleteUser_r(join->p->binding_handle, join, &d);
if (!NT_STATUS_IS_OK(status)) {
+ printf("DeleteUser failed\n");
+ }
+ if (!NT_STATUS_IS_OK(d.out.result)) {
printf("Delete of machine account %s failed\n",
join->netbios_name);
} else {