dma-buf: fix an error pointer vs NULL bug
authorDan Carpenter <dan.carpenter@linaro.org>
Thu, 6 Jul 2023 12:37:51 +0000 (15:37 +0300)
committerSumit Semwal <sumit.semwal@linaro.org>
Thu, 6 Jul 2023 14:20:23 +0000 (19:50 +0530)
commit00ae1491f970acc454be0df63f50942d94825860
tree62e12495036ccdd3fd7fb9b0e4f40403302a2d62
parent7f34e01f77f811ecb2ef83e60301b38cf89af466
dma-buf: fix an error pointer vs NULL bug

Smatch detected potential error pointer dereference.

    drivers/gpu/drm/drm_syncobj.c:888 drm_syncobj_transfer_to_timeline()
    error: 'fence' dereferencing possible ERR_PTR()

The error pointer comes from dma_fence_allocate_private_stub().  One
caller expected error pointers and one expected NULL pointers.  Change
it to return NULL and update the caller which expected error pointers,
drm_syncobj_assign_null_handle(), to check for NULL instead.

Fixes: f781f661e8c9 ("dma-buf: keep the signaling time of merged fences v3")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Sumit Semwal <sumit.semwal@linaro.org>
Signed-off-by: Sumit Semwal <sumit.semwal@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/b09f1996-3838-4fa2-9193-832b68262e43@moroto.mountain
drivers/dma-buf/dma-fence.c
drivers/gpu/drm/drm_syncobj.c