nvmet: Identify-Active Namespace ID List command should reject invalid nsid
[ Upstream commit899d2e5a4e] 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> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
2443c10be3
commit
5ac3d0a487
@@ -587,6 +587,16 @@ static void nvmet_execute_identify_nslist(struct nvmet_req *req)
|
||||
u16 status = 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);
|
||||
if (!list) {
|
||||
status = NVME_SC_INTERNAL;
|
||||
|
||||
Reference in New Issue
Block a user