svn commit: samba r12554 - in branches/SAMBA_4_0/source/rpc_server: .

[tridge at samba.org]

Author: tridge
Date: 2005-12-28 22:47:22 +0000 (Wed, 28 Dec 2005)
New Revision: 12554

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=12554

Log:

get rid of the pesky NTLMSSP warnings about being called after processing is finished


Modified:
   branches/SAMBA_4_0/source/rpc_server/dcerpc_server.c
   branches/SAMBA_4_0/source/rpc_server/dcesrv_auth.c


Changeset:
Modified: branches/SAMBA_4_0/source/rpc_server/dcerpc_server.c
===================================================================
--- branches/SAMBA_4_0/source/rpc_server/dcerpc_server.c	2005-12-28 22:43:12 UTC (rev 12553)
+++ branches/SAMBA_4_0/source/rpc_server/dcerpc_server.c	2005-12-28 22:47:22 UTC (rev 12554)
@@ -644,7 +644,7 @@
 
 
 /*
-  handle a bind request
+  handle a alter context request
 */
 static NTSTATUS dcesrv_alter(struct dcesrv_call_state *call)
 {

Modified: branches/SAMBA_4_0/source/rpc_server/dcesrv_auth.c
===================================================================
--- branches/SAMBA_4_0/source/rpc_server/dcesrv_auth.c	2005-12-28 22:43:12 UTC (rev 12553)
+++ branches/SAMBA_4_0/source/rpc_server/dcesrv_auth.c	2005-12-28 22:47:22 UTC (rev 12554)
@@ -230,7 +230,8 @@
 
 	/* on a pure interface change there is no auth_info structure
 	   setup */
-	if (!call->conn->auth_state.auth_info) {
+	if (!call->conn->auth_state.auth_info ||
+	    dce_conn->auth_state.auth_info->credentials.length == 0) {
 		return True;
 	}
 
@@ -258,10 +259,10 @@
 		dce_conn->auth_state.auth_info->auth_pad_length = 0;
 		dce_conn->auth_state.auth_info->auth_reserved = 0;
 		return True;
-	} else {
-		DEBUG(2, ("Failed to finish dcesrv auth alter_ack: %s\n", nt_errstr(status)));
-		return True;
 	}
+
+	DEBUG(2, ("Failed to finish dcesrv auth alter_ack: %s\n", nt_errstr(status)));
+	return False;
 }
 
 /*