1
0
mirror of https://github.com/samba-team/samba.git synced 2025-03-11 16:58:40 +03:00

tests/krb5: Correctly assert that we found a LOGON_INFO PAC buffer

Because ‘found_logon_info’ was invariably true, we would miss the case
in which set_pac_sids() failed to accomplish anything, having been
unable to find the LOGON_INFO PAC buffer.

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
This commit is contained in:
Joseph Sutton 2023-08-25 10:53:00 +12:00 committed by Andrew Bartlett
parent 04cdb13c08
commit 3f01cf9169

View File

@ -1603,7 +1603,7 @@ class KDCBaseTest(TestCaseInTempDir, RawKerberosTest):
else:
self.fail(f'invalid SID type {sid_type}')
found_logon_info = True
found_logon_info = False
pac_buffers = pac.buffers
for pac_buffer in pac_buffers: