drm/v3d: cleanup BOs properly when lookup_bos fails
authorMelissa Wen <mwen@igalia.com>
Mon, 5 Dec 2022 13:55:37 +0000 (12:55 -0100)
committerMelissa Wen <melissa.srw@gmail.com>
Mon, 19 Dec 2022 00:08:19 +0000 (23:08 -0100)
commitf98c5ec25162fbb019212c64cdd3bfde8af1fac7
treee2d5628cf7a51314556f4f8e8df04000407bd314
parent4b6cb2b67da883bc5095ee6d77f951f1cd7a1c24
drm/v3d: cleanup BOs properly when lookup_bos fails

When v3d_lookup_bos fails to `allocate validated BO pointers`,
job->bo_count was already set to args->bo_count, but job->bo points to
NULL. In this scenario, we must verify that job->bo is not NULL before
iterating on it to proper clean up a job. Also, drm_gem_object_put
already checks that the object passed is not NULL, doing the job->bo[i]
checker redundant.

Signed-off-by: Melissa Wen <mwen@igalia.com>
Reviewed-by: MaĆ­ra Canal <mcanal@igalia.com>
Signed-off-by: Melissa Wen <melissa.srw@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20221205135538.3545051-2-mwen@igalia.com
drivers/gpu/drm/v3d/v3d_gem.c