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);

Reply via email to