summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChih-Yen Chang <cc85nod@gmail.com>2023-05-06 00:03:54 +0900
committerSteve French <stfrench@microsoft.com>2023-05-16 10:26:14 -0500
commit443d61d1fa9faa60ef925513d83742902390100f (patch)
tree01b7aef0a5bf4155865d326da98b2afe0df936ac
parentf0a96d1aafd8964e1f9955c830a3e5cb3c60a90f (diff)
ksmbd: allocate one more byte for implied bcc[0]
ksmbd_smb2_check_message allows client to return one byte more, so we need to allocate additional memory in ksmbd_conn_handler_loop to avoid out-of-bound access. Cc: stable@vger.kernel.org Signed-off-by: Chih-Yen Chang <cc85nod@gmail.com> Acked-by: Namjae Jeon <linkinjeon@kernel.org> Signed-off-by: Steve French <stfrench@microsoft.com>
-rw-r--r--fs/ksmbd/connection.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/ksmbd/connection.c b/fs/ksmbd/connection.c
index 4ed379f9b1aa..4882a812ea86 100644
--- a/fs/ksmbd/connection.c
+++ b/fs/ksmbd/connection.c
@@ -351,7 +351,8 @@ int ksmbd_conn_handler_loop(void *p)
break;
/* 4 for rfc1002 length field */
- size = pdu_size + 4;
+ /* 1 for implied bcc[0] */
+ size = pdu_size + 4 + 1;
conn->request_buf = kvmalloc(size, GFP_KERNEL);
if (!conn->request_buf)
break;