From fbc00460f8deaa57bfbf16b12d29acec56578b63 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 24 Jun 2008 15:21:46 +0200 Subject: [PATCH] Remove a check on p->vuid in _netr_LogonSamLogon/off This is in preparation of removing p->vuid completely. I don't think this check here makes any sort of sense, it only proves that the user having done the session setup on its way to the schannel bind (ususally anonymous) did not do a logoff in between. But this would have been caught way before this in reply_pipe_write_and_X. --- source/rpc_server/srv_netlog_nt.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/source/rpc_server/srv_netlog_nt.c b/source/rpc_server/srv_netlog_nt.c index a0812283d65..7ece482d754 100644 --- a/source/rpc_server/srv_netlog_nt.c +++ b/source/rpc_server/srv_netlog_nt.c @@ -702,9 +702,6 @@ NTSTATUS _netr_LogonSamLogoff(pipes_struct *p, } - if (!get_valid_user_struct(p->vuid)) - return NT_STATUS_NO_SUCH_USER; - /* Using the remote machine name for the creds store: */ /* r->in.computer_name */ @@ -790,9 +787,6 @@ NTSTATUS _netr_LogonSamLogon(pipes_struct *p, return NT_STATUS_ACCESS_DENIED; } - if (!get_valid_user_struct(p->vuid)) - return NT_STATUS_NO_SUCH_USER; - if (process_creds) { fstring remote_machine; -- 2.47.3