drm/ttm: Always and only destroy bo->ttm_resv in ttm_bo_release_list
authorMichel Dänzer <[email protected]>
Fri, 3 Nov 2017 15:00:35 +0000 (16:00 +0100)
committerAlex Deucher <[email protected]>
Sat, 4 Nov 2017 13:48:01 +0000 (09:48 -0400)
commite1fc12c5d9ad06a2a74e97a91f1b0c5f4c723b50
treea138da480d215bdc6608d6c275bdc6874d4bbb92
parent37c5f2c99adf63adf13f1ca309a1ffce25c5589a
drm/ttm: Always and only destroy bo->ttm_resv in ttm_bo_release_list

Fixes a use-after-free due to a race condition in
ttm_bo_cleanup_refs_and_unlock, which allows one task to reserve a BO
and destroy its ttm_resv while another task is waiting for it to signal
in reservation_object_wait_timeout_rcu.

v2:
* Always initialize bo->ttm_resv in ttm_bo_init_reserved
 (Christian König)

Fixes: 0d2bd2ae045d "drm/ttm: fix memory leak while individualizing BOs"
Reviewed-by: Chunming Zhou <[email protected]> # v1
Reviewed-by: Christian König <[email protected]>
Signed-off-by: Michel Dänzer <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
drivers/gpu/drm/ttm/ttm_bo.c