]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
bhyve nvme: Fix out-of-bound IOV array access
authorChuck Tuffli <chuck@FreeBSD.org>
Thu, 9 Jun 2022 18:19:32 +0000 (11:19 -0700)
committerChuck Tuffli <chuck@FreeBSD.org>
Sat, 13 Aug 2022 19:16:02 +0000 (12:16 -0700)
commit88951aaaee73b87121b0f121224fe188a5b5e6e3
tree4e893323221e6082549468d8c852148ffce5c63e
parent14e3d3248a20fafcb097fdf353d83e19ac70846b
bhyve nvme: Fix out-of-bound IOV array access

Summary:
NVMe operations indicate the memory region(s) associated with a command
via physical region pages (PRPs). Since each PRP has a fixed size,
contiguous memory regions larger than the PRP size require multiple PRP
entries.

Instead of issuing a blockif call for each PRP, the NVMe emulation
concatenates multiple contiguous PRP entries into a single blockif
request. The test for contiguous regions has a bug such that it
mistakenly treats an initial PRP address of zero as a contiguous range
and concatenates it with the previous. But because there is no previous
IOV, the concatenation code corrupts the IO request structure and leads
to a segmentation fault when the blockif request completes.

Fix is to test for the existence of a previous range before trying to
concatenate the current range with the previous one.

While in the area, rename pci_nvme_append_iov_req()'s lba parameter to
offset to match its usage.

PR:             264177
Reported by:    Robert Morris <rtm@lcs.mit.edu>
Reviewed by: jhb
MFC after:      2 weeks
Differential Revision: https://reviews.freebsd.org/D35328
usr.sbin/bhyve/pci_nvme.c