nvmet-tcp: fix a crash in nvmet_req_complete()
authorMaurizio Lombardi <mlombard@redhat.com>
Fri, 22 Dec 2023 15:17:49 +0000 (16:17 +0100)
committerKeith Busch <kbusch@kernel.org>
Tue, 2 Jan 2024 20:56:19 +0000 (12:56 -0800)
commit0849a5441358cef02586fb2d60f707c0db195628
treeee23ac8e00260b310363013f0af28ef82987d881
parentefa56305908ba20de2104f1b8508c6a7401833be
nvmet-tcp: fix a crash in nvmet_req_complete()

in nvmet_tcp_handle_h2c_data_pdu(), if the host sends a data_offset
different from rbytes_done, the driver ends up calling nvmet_req_complete()
passing a status error.
The problem is that at this point cmd->req is not yet initialized,
the kernel will crash after dereferencing a NULL pointer.

Fix the bug by replacing the call to nvmet_req_complete() with
nvmet_tcp_fatal_error().

Fixes: 872d26a391da ("nvmet-tcp: add NVMe over TCP target driver")
Reviewed-by: Keith Busch <kbsuch@kernel.org>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
Signed-off-by: Keith Busch <kbusch@kernel.org>
drivers/nvme/target/tcp.c