Skip to content

Commit fcc1d8e

Browse files
committed
RDMA/umem: Fix double dma_buf_unpin in failure path
jira KERNEL-1052 cve CVE-2026-43128 Rebuild_History Non-Buildable kernel-5.14.0-687.10.1.el9_8 commit-author Jacob Moroni <jmoroni@google.com> commit 104016e In ib_umem_dmabuf_get_pinned_with_dma_device(), the call to ib_umem_dmabuf_map_pages() can fail. If this occurs, the dmabuf is immediately unpinned but the umem_dmabuf->pinned flag is still set. Then, when ib_umem_release() is called, it calls ib_umem_dmabuf_revoke() which will call dma_buf_unpin() again. Fix this by removing the immediate unpin upon failure and just let the ib_umem_release/revoke path handle it. This also ensures the proper unmap-unpin unwind ordering if the dmabuf_map_pages call happened to fail due to dma_resv_wait_timeout (and therefore has a non-NULL umem_dmabuf->sgt). Fixes: 1e4df4a ("RDMA/umem: Allow pinned dmabuf umem usage") Signed-off-by: Jacob Moroni <jmoroni@google.com> Link: https://patch.msgid.link/20260224234153.1207849-1-jmoroni@google.com Signed-off-by: Leon Romanovsky <leon@kernel.org> (cherry picked from commit 104016e) Signed-off-by: Jonathan Maple <jmaple@ciq.com>
1 parent 293c7f9 commit fcc1d8e

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

drivers/infiniband/core/umem_dmabuf.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -220,13 +220,11 @@ ib_umem_dmabuf_get_pinned_with_dma_device(struct ib_device *device,
220220

221221
err = ib_umem_dmabuf_map_pages(umem_dmabuf);
222222
if (err)
223-
goto err_unpin;
223+
goto err_release;
224224
dma_resv_unlock(umem_dmabuf->attach->dmabuf->resv);
225225

226226
return umem_dmabuf;
227227

228-
err_unpin:
229-
dma_buf_unpin(umem_dmabuf->attach);
230228
err_release:
231229
dma_resv_unlock(umem_dmabuf->attach->dmabuf->resv);
232230
ib_umem_release(&umem_dmabuf->umem);

0 commit comments

Comments
 (0)