From 0590e7ef3685d629057971cf132838bf7e223ee3 Mon Sep 17 00:00:00 2001 From: Günther Deschner Date: Fri, 19 Mar 2010 12:26:27 +0100 Subject: s4:torture/rpc/atsvc.c: add explicit check for NTSTATUS r.out.result Guenther Signed-off-by: Stefan Metzmacher --- source4/torture/rpc/atsvc.c | 28 ++++++++++++---------------- 1 file changed, 12 insertions(+), 16 deletions(-) (limited to 'source4/torture/rpc/atsvc.c') diff --git a/source4/torture/rpc/atsvc.c b/source4/torture/rpc/atsvc.c index 4b6f67cdab..52bb64b9d4 100644 --- a/source4/torture/rpc/atsvc.c +++ b/source4/torture/rpc/atsvc.c @@ -24,7 +24,6 @@ static bool test_JobGetInfo(struct dcerpc_pipe *p, struct torture_context *tctx, uint32_t job_id) { - NTSTATUS status; struct dcerpc_binding_handle *b = p->binding_handle; struct atsvc_JobGetInfo r; struct atsvc_JobInfo *info = talloc(tctx, struct atsvc_JobInfo); @@ -36,9 +35,9 @@ static bool test_JobGetInfo(struct dcerpc_pipe *p, struct torture_context *tctx, r.in.job_id = job_id; r.out.job_info = &info; - status = dcerpc_atsvc_JobGetInfo_r(b, tctx, &r); - - torture_assert_ntstatus_ok(tctx, status, "JobGetInfo failed"); + torture_assert_ntstatus_ok(tctx, dcerpc_atsvc_JobGetInfo_r(b, tctx, &r), + "JobGetInfo failed"); + torture_assert_ntstatus_ok(tctx, r.out.result, "JobGetInfo failed"); return true; } @@ -46,7 +45,6 @@ static bool test_JobGetInfo(struct dcerpc_pipe *p, struct torture_context *tctx, static bool test_JobDel(struct dcerpc_pipe *p, struct torture_context *tctx, uint32_t min_job_id, uint32_t max_job_id) { - NTSTATUS status; struct dcerpc_binding_handle *b = p->binding_handle; struct atsvc_JobDel r; @@ -54,16 +52,15 @@ static bool test_JobDel(struct dcerpc_pipe *p, struct torture_context *tctx, uin r.in.min_job_id = min_job_id; r.in.max_job_id = max_job_id; - status = dcerpc_atsvc_JobDel_r(b, tctx, &r); - - torture_assert_ntstatus_ok(tctx, status, "JobDel failed"); + torture_assert_ntstatus_ok(tctx, dcerpc_atsvc_JobDel_r(b, tctx, &r), + "JobDel failed"); + torture_assert_ntstatus_ok(tctx, r.out.result, "JobDel failed"); return true; } static bool test_JobEnum(struct torture_context *tctx, struct dcerpc_pipe *p) { - NTSTATUS status; struct dcerpc_binding_handle *b = p->binding_handle; struct atsvc_JobEnum r; struct atsvc_enum_ctr ctr; @@ -79,9 +76,9 @@ static bool test_JobEnum(struct torture_context *tctx, struct dcerpc_pipe *p) r.in.resume_handle = r.out.resume_handle = &resume_handle; r.out.total_entries = &total_entries; - status = dcerpc_atsvc_JobEnum_r(b, tctx, &r); - - torture_assert_ntstatus_ok(tctx, status, "JobEnum failed"); + torture_assert_ntstatus_ok(tctx, dcerpc_atsvc_JobEnum_r(b, tctx, &r), + "JobEnum failed"); + torture_assert_ntstatus_ok(tctx, r.out.result, "JobEnum failed"); for (i = 0; i < r.out.ctr->entries_read; i++) { if (!test_JobGetInfo(p, tctx, r.out.ctr->first_entry[i].job_id)) { @@ -94,7 +91,6 @@ static bool test_JobEnum(struct torture_context *tctx, struct dcerpc_pipe *p) static bool test_JobAdd(struct torture_context *tctx, struct dcerpc_pipe *p) { - NTSTATUS status; struct dcerpc_binding_handle *b = p->binding_handle; struct atsvc_JobAdd r; struct atsvc_JobInfo info; @@ -107,9 +103,9 @@ static bool test_JobAdd(struct torture_context *tctx, struct dcerpc_pipe *p) info.command = "foo.exe"; r.in.job_info = &info; - status = dcerpc_atsvc_JobAdd_r(b, tctx, &r); - - torture_assert_ntstatus_ok(tctx, status, "JobAdd failed"); + torture_assert_ntstatus_ok(tctx, dcerpc_atsvc_JobAdd_r(b, tctx, &r), + "JobAdd failed"); + torture_assert_ntstatus_ok(tctx, r.out.result, "JobAdd failed"); /* Run EnumJobs again in case there were no jobs to begin with */ -- cgit