nvmet: Identify-Active Namespace ID List command should reject invalid nsid
nsid values of 0xFFFFFFFE and 0XFFFFFFFF should be rejected with
a status code of "Invalid Namespace or Format".
See NVMe Base Specification, Active Namespace ID list (CNS 02h).
Fixes: a07b4970f4 ("nvmet: add a generic NVMe target")
Signed-off-by: Maurizio Lombardi <mlombard@redhat.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Keith Busch <kbusch@kernel.org>
This commit is contained in:
committed by
Keith Busch
parent
28982ad73d
commit
899d2e5a4e
@@ -587,6 +587,16 @@ static void nvmet_execute_identify_nslist(struct nvmet_req *req)
|
|||||||
u16 status = 0;
|
u16 status = 0;
|
||||||
int i = 0;
|
int i = 0;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* NSID values 0xFFFFFFFE and NVME_NSID_ALL are invalid
|
||||||
|
* See NVMe Base Specification, Active Namespace ID list (CNS 02h).
|
||||||
|
*/
|
||||||
|
if (min_nsid == 0xFFFFFFFE || min_nsid == NVME_NSID_ALL) {
|
||||||
|
req->error_loc = offsetof(struct nvme_identify, nsid);
|
||||||
|
status = NVME_SC_INVALID_NS | NVME_STATUS_DNR;
|
||||||
|
goto out;
|
||||||
|
}
|
||||||
|
|
||||||
list = kzalloc(buf_size, GFP_KERNEL);
|
list = kzalloc(buf_size, GFP_KERNEL);
|
||||||
if (!list) {
|
if (!list) {
|
||||||
status = NVME_SC_INTERNAL;
|
status = NVME_SC_INTERNAL;
|
||||||
|
|||||||
Reference in New Issue
Block a user