diff options
| author | Nigel Kirkland <nkirkland2304@gmail.com> | 2023-08-17 22:43:01 +0300 |
|---|---|---|
| committer | Keith Busch <kbusch@kernel.org> | 2023-08-21 23:29:17 +0300 |
| commit | 8ae5b3a685dc59a8cf7ccfe0e850999ba9727a3c (patch) | |
| tree | 975996548735e899b1ffd015b776e64ef4086bc3 /drivers/nvme | |
| parent | 71be868472dc5beb82feb4da2d3eb9cba785d660 (diff) | |
| download | linux-8ae5b3a685dc59a8cf7ccfe0e850999ba9727a3c.tar.xz | |
nvme-fc: Prevent null pointer dereference in nvme_fc_io_getuuid()
The nvme_fc_fcp_op structure describing an AEN operation is initialized with a
null request structure pointer. An FC LLDD may make a call to
nvme_fc_io_getuuid passing a pointer to an nvmefc_fcp_req for an AEN operation.
Add validation of the request structure pointer before dereference.
Signed-off-by: Nigel Kirkland <nkirkland2304@gmail.com>
Reviewed-by: James Smart <jsmart2021@gmail.com>
Signed-off-by: Keith Busch <kbusch@kernel.org>
Diffstat (limited to 'drivers/nvme')
| -rw-r--r-- | drivers/nvme/host/fc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c index 1cd2bf82319a..a15b37750d6e 100644 --- a/drivers/nvme/host/fc.c +++ b/drivers/nvme/host/fc.c @@ -1924,7 +1924,7 @@ char *nvme_fc_io_getuuid(struct nvmefc_fcp_req *req) struct nvme_fc_fcp_op *op = fcp_req_to_fcp_op(req); struct request *rq = op->rq; - if (!IS_ENABLED(CONFIG_BLK_CGROUP_FC_APPID) || !rq->bio) + if (!IS_ENABLED(CONFIG_BLK_CGROUP_FC_APPID) || !rq || !rq->bio) return NULL; return blkcg_get_fc_appid(rq->bio); } |