py:dcerpc/raw_protocol: consistently call self.recv_pdu(timeout=0.01) after auth3
authorStefan Metzmacher <metze@samba.org>
Wed, 21 Nov 2018 10:49:40 +0000 (11:49 +0100)
committerJeremy Allison <jra@samba.org>
Sat, 12 Jan 2019 02:13:40 +0000 (03:13 +0100)
When we don't expect a FAULT, we should wait a little bit to check
there's no response to auth3 request. This reduces the raw_procol
test from 45s down to 35s total runtime against Windows.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=7113
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11892

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
python/samba/tests/dcerpc/raw_protocol.py

index 384b7d47bcae1d1387bc0dcbcab58ac9cfa00fa2..f29255737a853f6db6497c95449da14d497ee3be 100755 (executable)
@@ -2982,7 +2982,7 @@ class TestDCERPC_BIND(RawDCERPCTest):
         req = self.generate_auth3(call_id=0,
                                   auth_info=auth_info)
         self.send_pdu(req)
-        rep = self.recv_pdu()
+        rep = self.recv_pdu(timeout=0.01)
         self.assertIsNone(rep)
         self.assertIsConnected()
 
@@ -4231,6 +4231,8 @@ class TestDCERPC_BIND(RawDCERPCTest):
         req = self.generate_auth3(call_id=0,
                                   auth_info=auth_info)
         self.send_pdu(req)
+        rep = self.recv_pdu(timeout=0.01)
+        self.assertIsNone(rep)
         self.assertIsConnected()
 
         # And now try a request without auth_info