Re: [PATCH] drm/syncobj: Fix fence and syncobj leak on dma_fence_chain_alloc failure

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] drm/syncobj: missing Fixes trailer for
  drm_syncobj_import_sync_file_fence()
--

commit de478817c7c05dbb269859f891ebb406ae0d952f
Author: tongzhiwei <tongzhiwei@cmss.chinamobile.com>

> drm/syncobj: Fix fence and syncobj leak on dma_fence_chain_alloc failure
>
> NOTE: this commit is worked by llm completely
>
> In drm_syncobj_import_sync_file_fence(), when point is non-zero and
> dma_fence_chain_alloc() returns NULL, the function returns -ENOMEM
> without releasing the fence reference (obtained via sync_file_get_fence)
> or the syncobj reference (obtained via drm_syncobj_find).  Both
> references are permanently leaked.
>
> Add dma_fence_put(fence) and drm_syncobj_put(syncobj) before returning
> -ENOMEM on the allocation failure path.
>
> Assisted-by: LLM
> Signed-off-by: tongzhiwei <tongzhiwei@cmss.chinamobile.com>

[Severity: Low]
This fixes a reference leak introduced with timeline sync_file import
support.  Should the commit message include a Fixes tag:

  Fixes: c2d3a7300695 ("drm/syncobj: Extend EXPORT_SYNC_FILE for timeline syncobjs")

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261106095730.166-1-tongzhiwei@cmss.chinamobile.com?part=1