drm/panfrost: Fix GEM handle creation ref-counting
authorSteven Price <steven.price@arm.com>
Mon, 19 Dec 2022 14:01:30 +0000 (14:01 +0000)
committerSteven Price <steven.price@arm.com>
Wed, 21 Dec 2022 15:04:42 +0000 (15:04 +0000)
commit4217c6ac817451d5116687f3cc6286220dc43d49
treef615a26607d8077e1b5423f8715052032cf7ab10
parent4e699e34f923188175986ad8a74ab99f7034075e
drm/panfrost: Fix GEM handle creation ref-counting

panfrost_gem_create_with_handle() previously returned a BO but with the
only reference being from the handle, which user space could in theory
guess and release, causing a use-after-free. Additionally if the call to
panfrost_gem_mapping_get() in panfrost_ioctl_create_bo() failed then
a(nother) reference on the BO was dropped.

The _create_with_handle() is a problematic pattern, so ditch it and
instead create the handle in panfrost_ioctl_create_bo(). If the call to
panfrost_gem_mapping_get() fails then this means that user space has
indeed gone behind our back and freed the handle. In which case just
return an error code.

Reported-by: Rob Clark <robdclark@chromium.org>
Fixes: f3ba91228e8e ("drm/panfrost: Add initial panfrost driver")
Signed-off-by: Steven Price <steven.price@arm.com>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20221219140130.410578-1-steven.price@arm.com
drivers/gpu/drm/panfrost/panfrost_drv.c
drivers/gpu/drm/panfrost/panfrost_gem.c
drivers/gpu/drm/panfrost/panfrost_gem.h