diff options
| author | Keith Busch <kbusch@kernel.org> | 2025-07-29 11:12:47 -0700 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2025-07-31 06:35:51 -0700 |
| commit | 4e6e151cf92bbaa0622a4da351ff444e4fd9b865 (patch) | |
| tree | c537e7468fa834fc27e96192b2caa7b9f532dff9 | |
| parent | 528589947c1802b9357c2a9b96d88cc4a11cd88b (diff) | |
nvme-pci: fix leak on sgl setup error
We need to free the descriptor that was allocated. We also don't
necessarily need to unmap each sgl entry, which was previously being
attempted unconditionally.
Signed-off-by: Keith Busch <kbusch@kernel.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
| -rw-r--r-- | drivers/nvme/host/pci.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c index 071efec25346..2c6d9506b172 100644 --- a/drivers/nvme/host/pci.c +++ b/drivers/nvme/host/pci.c @@ -935,7 +935,7 @@ static blk_status_t nvme_pci_setup_data_sgl(struct request *req, nvme_pci_sgl_set_seg(&iod->cmd.common.dptr.sgl, sgl_dma, mapped); if (unlikely(iter->status)) - nvme_free_sgls(req); + nvme_unmap_data(req); return iter->status; } |