]> git.proxmox.com Git - mirror_qemu.git/commit
hw/block/nvme: fix assert crash in nvme_subsys_ns
authorKlaus Jensen <k.jensen@samsung.com>
Wed, 7 Apr 2021 05:07:43 +0000 (07:07 +0200)
committerKlaus Jensen <k.jensen@samsung.com>
Wed, 7 Apr 2021 08:48:32 +0000 (10:48 +0200)
commitec20329748d02728b823443436fe26eadb04f8cc
tree17c4e239670f5d48e9a969755cb629d7a1db2aaa
parent8eb5c8069a5ccb8dadf35765b6f9cca10fb98b84
hw/block/nvme: fix assert crash in nvme_subsys_ns

nvme_subsys_ns() is used in contexts where the namespace identifier is
taken from an untrusted source. Commit 3921756dee6d ("hw/block/nvme:
assert namespaces array indices") tried to guard against this by
introducing an assert on the namespace identifier.

This is wrong since it is perfectly valid to call the function with an
invalid namespace identifier and like nvme_ns(), nvme_subsys_ns() should
simply return NULL.

Fixes: 3921756dee6d ("hw/block/nvme: assert namespaces array indices")
Fixes: 94d8d6d16781 ("hw/block/nvme: support allocated namespace type")
Cc: Minwoo Im <minwoo.im.dev@gmail.com>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
Reviewed-by: Minwoo Im <minwoo.im.dev@gmail.com>
hw/block/nvme-subsys.h