sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-01-13 13:18:06 +03:00
Code

smbd:smb2: ignore an dhnq blob along with a dhnc in create

Browse Source 
This is according to MS-SMB2, 3.3.5.9.7
"Handling the SMB2_CREATE_DURABLE_HANDLE_RECONNECT Create Context"

Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
This commit is contained in:
Michael Adam 2013-09-26 07:48:42 +02:00 committed by Stefan Metzmacher
parent ea51681cc2
commit dd25679256
1 changed files with 26 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
source3/smbd/smb2_create.c
View File

@ -479,16 +479,36 @@ static struct tevent_req *smbd_smb2_create_send(TALLOC_CTX *mem_ctx,
}
if (dhnc) {
uint32_t num_blobs_allowed;
if (dhnc->data.length != 16) {
tevent_req_nterror(req, NT_STATUS_INVALID_PARAMETER);
return tevent_req_post(req, ev);
}
if (in_context_blobs.num_blobs != 1) {
/*
* DHNC should be the only one.
* TODO: This is only true for the oplock case!
* For leases, lease request is required additionally!
*/
/*
* According to MS-SMB2: 3.3.5.9.7, "Handling the
* SMB2_CREATE_DURABLE_HANDLE_RECONNECT Create Context",
* we should ignore an additional dhnq blob, but fail
* the request (with status OBJECT_NAME_NOT_FOUND) if
* any other extra create blob has been provided.
*
* (Note that the cases of an additional dh2q or dh2c blob
* which require a different error code, have been treated
* above.)
*
* TODO:
* This is only true for the oplock case:
* For leases, lease request is required additionally.
*/
if (dhnq) {
num_blobs_allowed = 2;
} else {
num_blobs_allowed = 1;
}
if (in_context_blobs.num_blobs != num_blobs_allowed) {
tevent_req_nterror(req, NT_STATUS_OBJECT_NAME_NOT_FOUND);
return tevent_req_post(req, ev);
}