diff options
| author | Namjae Jeon <linkinjeon@kernel.org> | 2025-02-21 08:16:23 +0300 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2025-03-03 07:50:53 +0300 |
| commit | e2ff19f0b7a30e03516e6eb73b948e27a55bc9d2 (patch) | |
| tree | abc6234b9ad0a8c3f04d06b8dd5162817cacc16d | |
| parent | d6e13e19063db24f94b690159d0633aaf72a0f03 (diff) | |
| download | linux-e2ff19f0b7a30e03516e6eb73b948e27a55bc9d2.tar.xz | |
ksmbd: fix type confusion via race condition when using ipc_msg_send_request
req->handle is allocated using ksmbd_acquire_id(&ipc_ida), based on
ida_alloc. req->handle from ksmbd_ipc_login_request and
FSCTL_PIPE_TRANSCEIVE ioctl can be same and it could lead to type confusion
between messages, resulting in access to unexpected parts of memory after
an incorrect delivery. ksmbd check type of ipc response but missing add
continue to check next ipc reponse.
Cc: stable@vger.kernel.org
Reported-by: Norbert Szetei <norbert@doyensec.com>
Tested-by: Norbert Szetei <norbert@doyensec.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
| -rw-r--r-- | fs/smb/server/transport_ipc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/smb/server/transport_ipc.c b/fs/smb/server/transport_ipc.c index 0460ebea6ff0..3f185ae60dc5 100644 --- a/fs/smb/server/transport_ipc.c +++ b/fs/smb/server/transport_ipc.c @@ -281,6 +281,7 @@ static int handle_response(int type, void *payload, size_t sz) if (entry->type + 1 != type) { pr_err("Waiting for IPC type %d, got %d. Ignore.\n", entry->type + 1, type); + continue; } entry->response = kvzalloc(sz, KSMBD_DEFAULT_GFP); |