In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
drm\/xe: Hold a dma-buf reference for imported BOs<\/p>\n
An imported dma-buf BO is created as a ttm_bo_type_sg BO whose
\nreservation object is the exporter’s dma_buf->resv. The importer,
\nhowever, only takes a dma-buf reference after a successful
\ndma_buf_dynamic_attach(). Until then nothing keeps the exporter alive,
\nso if the exporter is freed while the BO still references its resv, a
\nlater access to that resv is a use-after-free:<\/p>\n
Oops: general protection fault, probably for non-canonical address
\n 0x6b6b6b6b6b6b6b9c
\n Workqueue: ttm ttm_bo_delayed_delete [ttm]
\n RIP: 0010:mutex_can_spin_on_owner+0x3f\/0xc0<\/p>\n
This can be reached on two paths:<\/p>\n
– dma_buf_dynamic_attach() fails, or
\n – ttm_bo_init_reserved() fails during BO creation.<\/p>\n
In both cases the BO already has bo->base.resv pointing at the exporter
\nresv, and sg BOs are always torn down via ttm_bo_delayed_delete(), which
\nlocks bo->base.resv asynchronously – potentially after the exporter has
\nbeen freed.<\/p>\n
Take the dma-buf reference in xe_bo_init_locked(), before
\nttm_bo_init_reserved(), so it also covers a creation failure there, and
\nrelease it in xe_ttm_bo_destroy(). The reference is held for the whole
\nBO lifetime, keeping the shared resv alive on every path.<\/p>\n
v2:
\n – Reworked the fix to avoid creating the imported sg BO before
\n dma_buf_dynamic_attach() succeeds.
\n – Attach with importer_priv == NULL and make invalidate_mappings ignore
\n incomplete imports.<\/p>\n
v3:
\n – Dropped the xe-side reordering approach since importer_priv must be
\n valid when dma_buf_dynamic_attach() publishes the attachment.
\n – Per Christian’s suggestion on the v1 thread, keyed the check on
\n import_attach rather than removing the sg guard entirely.
\n – Fixes both xe and amdgpu in a single TTM patch.<\/p>\n
v4:
\n – Moved import_attach check to after dma_resv_copy_fences() so fences
\n are copied before returning for successful imports (Thomas).
\n – Removed exporter-alive claim from commit message (Thomas).<\/p>\n
v5:
\n – Add drm\/xe patch to keep imported sg BOs off the LRU before attach
\n succeeds; the TTM fix alone is not sufficient for xe if the BO is
\n already LRU-visible. (Thomas)
\n v4 patch:
\n https:\/\/patchwork.freedesktop.org\/patch\/736663\/?series=169129&rev=2
\n – Patch 1 (drm\/ttm) carries Christian’s Reviewed-by from v4.<\/p>\n
v6:
\n – Reworked the fix based on Thomas’ suggestion. Instead of the TTM resv
\n individualization (v1-v5) plus the xe off-LRU\/placement handling (v5),
\n just hold a dma-buf reference for the imported BO lifetime so the
\n shared resv can never be freed while the BO still references it.
\n Single xe patch, no TTM change. (Thomas)
\n – Take the reference in xe_bo_init_locked() before ttm_bo_init_reserved()
\n so a TTM creation failure is covered too (Thomas).
\n – Dropped the v5 series (drm\/ttm + drm\/xe off-LRU); the off-LRU approach
\n also regressed in CI BAT via ttm_bo_pipeline_gutting() creating a ghost
\n BO that outlived the exporter.
\n Link to v5: https:\/\/patchwork.freedesktop.org\/series\/169984\/<\/p>\n
v7:
\n – Move changelog above — so it stays in the commit message.
\n – Reorder changelog entries oldest-to-newest. (Thomas)<\/p>\n
(cherry picked from commit 3516f3fae6be35642f8f06f8a218da6425c0306a)<\/p>\n","protected":false},"excerpt":{"rendered":"
In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
drm\/xe: Hold a dma-buf reference for imported BOs<\/p>\n
An imported dma-buf BO is created as a ttm_bo_type_sg BO whose
\nreservation objec<\/p>\n","protected":false},"template":"","meta":[],"categories":[],"tags":[],"class_list":["post-5405","vuln","type-vuln","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln\/5405","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln"}],"about":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/types\/vuln"}],"wp:attachment":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=5405"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=5405"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=5405"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}