From: Chih-Yen Chang <cc85nod@gmail.com>
Date: Fri, 5 May 2023 15:03:54 +0000 (+0900)
Subject: ksmbd: allocate one more byte for implied bcc[0]
X-Git-Tag: v6.6.7~2813^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=443d61d1fa9faa60ef925513d83742902390100f;p=platform%2Fkernel%2Flinux-starfive.git

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>
---

diff --git a/fs/ksmbd/connection.c b/fs/ksmbd/connection.c
index 4ed379f..4882a81 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;