From: Volker Lendecke <vl@samba.org>
Date: Tue, 24 Jun 2008 13:21:46 +0000 (+0200)
Subject: Remove a check on p->vuid in _netr_LogonSamLogon/off
X-Git-Tag: samba-3.3.0pre1~748
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=fbc00460f8deaa57bfbf16b12d29acec56578b63;p=thirdparty%2Fsamba.git

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.
---

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;