mirror of
https://github.com/qemu/qemu.git
synced 2024-12-13 05:33:34 +08:00
hw/block/nvme: make sure ncqr and nsqr is valid
0xffff is not an allowed value for NCQR and NSQR in Set Features on Number of Queues. Signed-off-by: Klaus Jensen <k.jensen@samsung.com> Acked-by: Keith Busch <kbusch@kernel.org> Reviewed-by: Maxim Levitsky <mlevitsk@redhat.com> Reviewed-by: Dmitry Fomichev <dmitry.fomichev@wdc.com> Message-Id: <20200706061303.246057-14-its@irrelevant.dk>
This commit is contained in:
parent
7c46310d29
commit
9932551154
@ -1310,6 +1310,14 @@ static uint16_t nvme_set_feature(NvmeCtrl *n, NvmeCmd *cmd, NvmeRequest *req)
|
||||
blk_set_enable_write_cache(n->conf.blk, dw11 & 1);
|
||||
break;
|
||||
case NVME_NUMBER_OF_QUEUES:
|
||||
/*
|
||||
* NVMe v1.3, Section 5.21.1.7: 0xffff is not an allowed value for NCQR
|
||||
* and NSQR.
|
||||
*/
|
||||
if ((dw11 & 0xffff) == 0xffff || ((dw11 >> 16) & 0xffff) == 0xffff) {
|
||||
return NVME_INVALID_FIELD | NVME_DNR;
|
||||
}
|
||||
|
||||
trace_pci_nvme_setfeat_numq((dw11 & 0xFFFF) + 1,
|
||||
((dw11 >> 16) & 0xFFFF) + 1,
|
||||
n->params.max_ioqpairs,
|
||||
|
Loading…
Reference in New Issue
Block a user