summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2002-08-28 12:45:19 +0000
committerJeremy Allison <jra@samba.org>2002-08-28 12:45:19 +0000
commit3a21fa3bb96e4d4ba5410435fdc6178c591db8c1 (patch)
treead6da5243a9d8359e80a52a35f7eb9f662e9705d
parentd1e68f2ad401c5bcef4a5452e0d836b601651bab (diff)
downloadsamba-3a21fa3bb96e4d4ba5410435fdc6178c591db8c1.tar.gz
samba-3a21fa3bb96e4d4ba5410435fdc6178c591db8c1.tar.bz2
samba-3a21fa3bb96e4d4ba5410435fdc6178c591db8c1.zip
Move -4 to correct side of comparison.
Jeremy. (This used to be commit d825026658b59364f9adc425bca4017be8e979e2)
-rw-r--r--source3/smbd/process.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/smbd/process.c b/source3/smbd/process.c
index 6b9e184afe..2c35eadb18 100644
--- a/source3/smbd/process.c
+++ b/source3/smbd/process.c
@@ -653,8 +653,8 @@ static int switch_message(int type,char *inbuf,char *outbuf,int size,int bufsize
errno = 0;
last_message = type;
- /* Make sure this is an SMB packet */
- if ((strncmp(smb_base(inbuf),"\377SMB",4) != 0) || (size - 4 < smb_size)) {
+ /* Make sure this is an SMB packet. smb_size contains NetBIOS header so subtract 4 from it. */
+ if ((strncmp(smb_base(inbuf),"\377SMB",4) != 0) || (size < (smb_size - 4))) {
DEBUG(2,("Non-SMB packet of length %d. Terminating server\n",smb_len(inbuf)));
exit_server("Non-SMB packet");
return(-1);