summaryrefslogtreecommitdiff
path: root/source4
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2012-11-24 11:28:57 +0100
committerMichael Adam <obnox@samba.org>2012-11-26 13:53:21 +0100
commit8336061096c259f5c3c93f869ff51bf4daab3fdc (patch)
treec4f90e4f3a700638c1e47e5c235addcf2daab1c6 /source4
parente4218e46c77e8d0c4f6c839024620c48f101e2f2 (diff)
downloadsamba-8336061096c259f5c3c93f869ff51bf4daab3fdc.tar.gz
samba-8336061096c259f5c3c93f869ff51bf4daab3fdc.tar.bz2
samba-8336061096c259f5c3c93f869ff51bf4daab3fdc.zip
s4:torture/rpc/handles: try to make all assoc_group tests less flakey
Just incrementing the assoc_group_id makes it too likely to hit a number that is already in use. Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Michael Adam <obnox@samba.org> Autobuild-User(master): Michael Adam <obnox@samba.org> Autobuild-Date(master): Mon Nov 26 13:53:22 CET 2012 on sn-devel-104
Diffstat (limited to 'source4')
-rw-r--r--source4/torture/rpc/handles.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/source4/torture/rpc/handles.c b/source4/torture/rpc/handles.c
index 3fdce5672e..5c08a5f371 100644
--- a/source4/torture/rpc/handles.c
+++ b/source4/torture/rpc/handles.c
@@ -471,6 +471,11 @@ static bool test_handles_mixed_shared(struct torture_context *torture)
torture_assert_ntstatus_equal(torture, status, NT_STATUS_UNSUCCESSFUL,
"opening lsa pipe4");
+ /*
+ * We use ~assoc_group_id instead of p1->assoc_group_id, because
+ * this way we are less likely to use an id which is already in use.
+ */
+ assoc_group_id = ~assoc_group_id;
torture_comment(torture, "connect samr pipe5 with assoc_group_id[0x%08X]- should fail\n", ++assoc_group_id);
status = torture_rpc_connection_transport(torture, &p5, &ndr_table_samr,
transport,