diff options
| author | Jeremy Allison <jra@samba.org> | 2006-04-18 03:45:09 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2006-04-18 03:45:09 +0000 |
| commit | 691ff2d14b4e3baab5423e5782168e3e1814ce59 (patch) | |
| tree | ab0f0821012236af848554de263aa8df552062c6 | |
| parent | 88cba3dcd84933c4a1e476dacb7ce9b77d194ddd (diff) | |
| download | samba-691ff2d14b4e3baab5423e5782168e3e1814ce59.tar.gz | |
r15115: Fix error return on sessionsetup. Ensure no
data blob is added if the logon call failed.
Jeremy.
| -rw-r--r-- | source/smbd/sesssetup.c | 26 |
1 files changed, 15 insertions, 11 deletions
diff --git a/source/smbd/sesssetup.c b/source/smbd/sesssetup.c index 3280e2d5fcc..7c90263a5b0 100644 --- a/source/smbd/sesssetup.c +++ b/source/smbd/sesssetup.c @@ -95,21 +95,25 @@ static BOOL reply_sesssetup_blob(connection_struct *conn, char *outbuf, { char *p; - set_message(outbuf,4,0,True); + if (!NT_STATUS_IS_OK(nt_status) && !NT_STATUS_EQUAL(nt_status, NT_STATUS_MORE_PROCESSING_REQUIRED)) { + ERROR_NT(nt_status); + } else { + set_message(outbuf,4,0,True); - nt_status = nt_status_squash(nt_status); - SIVAL(outbuf, smb_rcls, NT_STATUS_V(nt_status)); - SSVAL(outbuf, smb_vwv0, 0xFF); /* no chaining possible */ - SSVAL(outbuf, smb_vwv3, blob.length); - p = smb_buf(outbuf); + nt_status = nt_status_squash(nt_status); + SIVAL(outbuf, smb_rcls, NT_STATUS_V(nt_status)); + SSVAL(outbuf, smb_vwv0, 0xFF); /* no chaining possible */ + SSVAL(outbuf, smb_vwv3, blob.length); + p = smb_buf(outbuf); - /* should we cap this? */ - memcpy(p, blob.data, blob.length); - p += blob.length; + /* should we cap this? */ + memcpy(p, blob.data, blob.length); + p += blob.length; - p += add_signature( outbuf, p ); + p += add_signature( outbuf, p ); - set_message_end(outbuf,p); + set_message_end(outbuf,p); + } show_msg(outbuf); return send_smb(smbd_server_fd(),outbuf); |