nvmet-tcp: fix kernel crash if commands allocation fails

[ Upstream commit 5572a55a6f830ee3f3a994b6b962a5c327d28cb3 ]

If the commands allocation fails in nvmet_tcp_alloc_cmds()
the kernel crashes in nvmet_tcp_release_queue_work() because of
a NULL pointer dereference.

  nvmet: failed to install queue 0 cntlid 1 ret 6
  Unable to handle kernel NULL pointer dereference at
         virtual address 0000000000000008

Fix the bug by setting queue->nr_cmds to zero in case
nvmet_tcp_alloc_cmd() fails.

Fixes: 872d26a391 ("nvmet-tcp: add NVMe over TCP target driver")
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Keith Busch <kbusch@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Maurizio Lombardi 2024-08-21 16:28:26 +02:00 коммит произвёл Greg Kroah-Hartman
Родитель 945be49f4e
Коммит 91dad30c56
1 изменённых файлов: 3 добавлений и 1 удалений

Просмотреть файл

@ -1819,8 +1819,10 @@ static u16 nvmet_tcp_install_queue(struct nvmet_sq *sq)
} }
queue->nr_cmds = sq->size * 2; queue->nr_cmds = sq->size * 2;
if (nvmet_tcp_alloc_cmds(queue)) if (nvmet_tcp_alloc_cmds(queue)) {
queue->nr_cmds = 0;
return NVME_SC_INTERNAL; return NVME_SC_INTERNAL;
}
return 0; return 0;
} }