diff options
| author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-11-27 19:52:46 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-11-27 21:50:42 +0100 |
| commit | 24c5d6b5042fc34a526348ea8a66aab2c994e3dc (patch) | |
| tree | df978acdceef76371fe28ccadc3a3775133ee89b | |
| parent | 5c8f0c5b9dcd4119ec1ef2e997f3ea75781eb53b (diff) | |
| download | samba-24c5d6b5042fc34a526348ea8a66aab2c994e3dc.tar.gz samba-24c5d6b5042fc34a526348ea8a66aab2c994e3dc.tar.bz2 samba-24c5d6b5042fc34a526348ea8a66aab2c994e3dc.zip | |
s4:torture/rpc/samr.c - fix uninitialised "status" variables
Detected by Tru64. I wonder how this tests even worked.
| -rw-r--r-- | source4/torture/rpc/samr.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/source4/torture/rpc/samr.c b/source4/torture/rpc/samr.c index 223146ee4e..a57cdb84c6 100644 --- a/source4/torture/rpc/samr.c +++ b/source4/torture/rpc/samr.c @@ -7229,6 +7229,7 @@ static bool test_EnumDomainUsers(struct dcerpc_binding_handle *b, torture_assert_ntstatus_ok(tctx, r.out.result, "failed to enumerate users"); } + status = r.out.result; total_num_entries += num_entries; } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES)); @@ -7269,6 +7270,7 @@ static bool test_EnumDomainGroups(struct dcerpc_binding_handle *b, torture_assert_ntstatus_ok(tctx, r.out.result, "failed to enumerate groups"); } + status = r.out.result; total_num_entries += num_entries; } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES)); @@ -7309,6 +7311,7 @@ static bool test_EnumDomainAliases(struct dcerpc_binding_handle *b, torture_assert_ntstatus_ok(tctx, r.out.result, "failed to enumerate aliases"); } + status = r.out.result; total_num_entries += num_entries; } while (NT_STATUS_EQUAL(status, STATUS_MORE_ENTRIES)); @@ -7353,6 +7356,7 @@ static bool test_QueryDisplayInfo_level(struct dcerpc_binding_handle *b, torture_assert_ntstatus_ok(tctx, r.out.result, "failed to query displayinfo"); } + status = r.out.result; if (*r.out.returned_size == 0) { break; |