summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mdw@samba.org>2010-11-27 19:52:46 +0100
committerMatthias Dieter Wallnöfer <mdw@samba.org>2010-11-27 21:50:42 +0100
commit24c5d6b5042fc34a526348ea8a66aab2c994e3dc (patch)
treedf978acdceef76371fe28ccadc3a3775133ee89b
parent5c8f0c5b9dcd4119ec1ef2e997f3ea75781eb53b (diff)
downloadsamba-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.c4
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;