drm/msm: Fix submit error-path leaks
authorRob Clark <robdclark@chromium.org>
Tue, 9 May 2023 20:30:41 +0000 (13:30 -0700)
committerRob Clark <robdclark@chromium.org>
Thu, 11 May 2023 14:41:48 +0000 (07:41 -0700)
commit68dc6c2d5eec45515855cce99256162f45651a0b
treece26c9cff410ac341543d90f3b9b1edb9381caf4
parentdb40d2928d245f3a6cba9a6a2547ec955b00f0fc
drm/msm: Fix submit error-path leaks

For errors after msm_submitqueue_get(), we need to drop the submitqueue
reference.  Additionally after get_unused_fd() we need to drop the fd.
The ordering for dropping the queue lock and put_unused_fd() is not
important, so just move this all into out_post_unlock.

v2: Only drop queue ref if submit doesn't take it
v3: Fix unitialized submit ref in error path
v4: IS_ERR_OR_NULL()

Reported-by: pinkperfect2021@gmail.com
Fixes: f0de40a131d9 drm/msm: ("Reorder lock vs submit alloc")
Signed-off-by: Rob Clark <robdclark@chromium.org>
Patchwork: https://patchwork.freedesktop.org/patch/536073/
Link: https://lore.kernel.org/r/20230509203041.440619-1-robdclark@gmail.com
drivers/gpu/drm/msm/msm_gem_submit.c