svn commit: samba r1727 - branches/SAMBA_4_0/source/smb_server

[abartlet at samba.org]

Author: abartlet
Date: 2004-08-11 16:17:43 +0000 (Wed, 11 Aug 2004)
New Revision: 1727
WebSVN: http://websvn.samba.org/websvn/changeset.php?rep=samba&path=/&rev=1727&nolog=1
Log:
SPNEGO session setup replies need to include the blob, even on error
cases, so don't shorten them back to 0 byte data length.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/smb_server/reply.c

Changeset:
Modified: branches/SAMBA_4_0/source/smb_server/reply.c
===================================================================
--- branches/SAMBA_4_0/source/smb_server/reply.c	2004-08-11 16:16:28 UTC (rev 1726)
+++ branches/SAMBA_4_0/source/smb_server/reply.c	2004-08-11 16:17:43 UTC (rev 1727)
@@ -2048,14 +2048,13 @@
 	/* call the generic handler */
 	status = sesssetup_backend(req, &sess);
 
-	if (!NT_STATUS_IS_OK(status)) {
-		req_reply_error(req, status);
-		return;
-	}
-
 	/* construct reply */
 	req_setup_reply(req, 4, sess.spnego.out.secblob.length);
 
+	if (!NT_STATUS_IS_OK(status)) {
+		req_setup_error(req, status);
+	}
+
 	SSVAL(req->out.vwv, VWV(0), SMB_CHAIN_NONE);
 	SSVAL(req->out.vwv, VWV(1), 0);
 	SSVAL(req->out.vwv, VWV(2), sess.spnego.out.action);