summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2007-02-20 23:56:46 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 12:18:07 -0500
commitcedcf06b22bb30ca52a8bbb21fd331ec20ef9180 (patch)
treee70c9c9337eee16bbfc2472fb9388646b7c9df41
parent7f981b457f9c64e6f93149051d9703ee81fd3726 (diff)
downloadsamba-cedcf06b22bb30ca52a8bbb21fd331ec20ef9180.tar.gz
samba-cedcf06b22bb30ca52a8bbb21fd331ec20ef9180.tar.bz2
samba-cedcf06b22bb30ca52a8bbb21fd331ec20ef9180.zip
r21478: Add 65k length limit for split SPNEGO blobs.
Jeremy (This used to be commit 6be078da267677e3e558033c28099e3932a17712)
-rw-r--r--source3/smbd/sesssetup.c28
1 files changed, 19 insertions, 9 deletions
diff --git a/source3/smbd/sesssetup.c b/source3/smbd/sesssetup.c
index 6ca306ac25..4e27914c0d 100644
--- a/source3/smbd/sesssetup.c
+++ b/source3/smbd/sesssetup.c
@@ -693,7 +693,7 @@ static struct pending_auth_data *get_pending_auth_data(uint16 smbpid)
/****************************************************************************
Check the size of an SPNEGO blob. If we need more return NT_STATUS_MORE_PROCESSING_REQUIRED,
- else return NT_STATUS_OK.
+ else return NT_STATUS_OK. Don't allow the blob to be more than 64k.
****************************************************************************/
static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB *pblob)
@@ -715,17 +715,18 @@ static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB
/* Were we waiting for more data ? */
if (pad) {
DATA_BLOB tmp_blob;
+ size_t copy_len = MIN(65536, pblob->length);
/* Integer wrap paranoia.... */
- if (pad->partial_data.length + pblob->length < pad->partial_data.length ||
- pad->partial_data.length + pblob->length < pblob->length) {
+ if (pad->partial_data.length + copy_len < pad->partial_data.length ||
+ pad->partial_data.length + copy_len < copy_len) {
DEBUG(2,("check_spnego_blob_complete: integer wrap "
"pad->partial_data.length = %u, "
- "pblob->length = %u\n",
+ "copy_len = %u\n",
(unsigned int)pad->partial_data.length,
- (unsigned int)pblob->length ));
+ (unsigned int)copy_len ));
delete_partial_auth(pad);
return NT_STATUS_INVALID_PARAMETER;
@@ -734,21 +735,23 @@ static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB
DEBUG(10,("check_spnego_blob_complete: "
"pad->partial_data.length = %u, "
"pad->needed_len = %u, "
+ "copy_len = %u, "
"pblob->length = %u,\n",
(unsigned int)pad->partial_data.length,
(unsigned int)pad->needed_len,
+ (unsigned int)copy_len,
(unsigned int)pblob->length ));
tmp_blob = data_blob(NULL,
- pad->partial_data.length + pblob->length);
+ pad->partial_data.length + copy_len);
- /* Concatenate the two. */
+ /* Concatenate the two (up to copy_len) bytes. */
memcpy(tmp_blob.data,
pad->partial_data.data,
pad->partial_data.length);
memcpy(tmp_blob.data + pad->partial_data.length,
pblob->data,
- pblob->length);
+ copy_len);
/* Replace the partial data. */
data_blob_free(&pad->partial_data);
@@ -766,7 +769,7 @@ static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB
}
/* Still need more data. */
- pad->needed_len -= pblob->length;
+ pad->needed_len -= copy_len;
return NT_STATUS_MORE_PROCESSING_REQUIRED;
}
@@ -821,6 +824,13 @@ static NTSTATUS check_spnego_blob_complete(uint16 smbpid, uint16 vuid, DATA_BLOB
return NT_STATUS_OK;
}
+ /* Refuse the blob if it's bigger than 64k. */
+ if (needed_len > 65536) {
+ DEBUG(2,("check_spnego_blob_complete: needed_len too large (%u)\n",
+ (unsigned int)needed_len ));
+ return NT_STATUS_INVALID_PARAMETER;
+ }
+
/* We must store this blob until complete. */
pad = SMB_MALLOC(sizeof(struct pending_auth_data));
if (!pad) {