summaryrefslogtreecommitdiff
path: root/source3/smbd/sesssetup.c
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2008-11-01 16:24:42 +0100
committerVolker Lendecke <vl@samba.org>2008-11-01 19:41:07 +0100
commit7f9d6f80efbc211977b13ece80fff6adbea929ac (patch)
tree1660eb379144ff05ebb8d65b369cdf80b5bf5d45 /source3/smbd/sesssetup.c
parent8637716b4b7b75a289e22141c5761c4850da3f5d (diff)
downloadsamba-7f9d6f80efbc211977b13ece80fff6adbea929ac.tar.gz
samba-7f9d6f80efbc211977b13ece80fff6adbea929ac.tar.bz2
samba-7f9d6f80efbc211977b13ece80fff6adbea929ac.zip
Add a "buflen" struct member to smb_request
This removes some explicit inbuf references and also removes a pointless check in reply_echo. The buflen can never be more than 64k, this is just a 16 bit value.
Diffstat (limited to 'source3/smbd/sesssetup.c')
-rw-r--r--source3/smbd/sesssetup.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c
index b258386121..02931e49f4 100644
--- a/source3/smbd/sesssetup.c
+++ b/source3/smbd/sesssetup.c
@@ -1446,8 +1446,7 @@ void reply_sesssetup_and_X(struct smb_request *req)
* don't get client caps. */
remove_from_common_flags2(FLAGS2_32_BIT_ERROR_CODES);
- if ((passlen1 > MAX_PASS_LEN)
- || (passlen1 > smb_buflen(req->inbuf))) {
+ if ((passlen1 > MAX_PASS_LEN) || (passlen1 > req->buflen)) {
reply_nterror(req, nt_status_squash(
NT_STATUS_INVALID_PARAMETER));
END_PROFILE(SMBsesssetupX);