diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2007-07-30 08:58:39 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 15:01:21 -0500 |
| commit | 649d4bf8aa1e957d6a059dd265d5c9b313a43f15 (patch) | |
| tree | 467dee39c92a2909ea96814ac31ea8a065f9c5d8 /source4/auth | |
| parent | 06a6194eadef9fa9c9f6b3c200c41d2a59dc76af (diff) | |
| download | samba-649d4bf8aa1e957d6a059dd265d5c9b313a43f15.tar.gz samba-649d4bf8aa1e957d6a059dd265d5c9b313a43f15.tar.bz2 samba-649d4bf8aa1e957d6a059dd265d5c9b313a43f15.zip | |
r24074: Test both permitted logon hours and permitted workstations in the
RPC-SAMLOGON test.
This showed that, as noted by bug #4823, we didn't test for invalid
workstations. In fact, the code had been ported across, but because
untested code is broken code, it never worked...
Andrew Bartlett
(This used to be commit 5e07417ada56d189a911ef888b0c87adebe60763)
Diffstat (limited to 'source4/auth')
| -rw-r--r-- | source4/auth/sam.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/source4/auth/sam.c b/source4/auth/sam.c index 2fb0a239ff..6e160941d8 100644 --- a/source4/auth/sam.c +++ b/source4/auth/sam.c @@ -46,7 +46,6 @@ const char *user_attrs[] = { "pwdLastSet", "accountExpires", "logonHours", - "objectSid", /* check 'allowed workstations' */ @@ -213,7 +212,7 @@ _PUBLIC_ NTSTATUS authsam_account_ok(TALLOC_CTX *mem_ctx, DEBUG(10,("sam_account_ok: checking for workstation match '%s' and '%s'\n", workstations[i], logon_workstation)); - if (strequal(workstations[i], logon_workstation) == 0) { + if (strequal(workstations[i], logon_workstation)) { invalid_ws = False; break; } |