Claude review: gpu/drm/drm_syncobj: fix syncobj refcount leak on invalid flags in find_fence

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: gpu/drm/drm_syncobj: fix syncobj refcount leak on invalid flags in find_fence
Author: Wentao Liang <vulab@iscas.ac.cn>
Patches: 1
Reviewed: 2026-05-25T16:51:16.005810

---

This is a single patch claiming to fix a refcount leak in `drm_syncobj_find_fence()`. **The patch is buggy — it introduces a NULL pointer dereference while attempting to fix a refcount leak, and the original code for the flags check was actually correct (no leak).**

The analysis requires looking at the order of operations in the original code vs. the patched code:

**Original code (pre-patch):**
```c
struct drm_syncobj *syncobj = drm_syncobj_find(file_private, handle);
...
if (flags & ~DRM_SYNCOBJ_WAIT_FLAGS_WAIT_FOR_SUBMIT)
    return -EINVAL;

if (!syncobj)
    return -ENOENT;
```

**`out` label:**
```c
out:
    drm_syncobj_put(syncobj);
    return ret;
```

---
Generated by Claude Code Patch Reviewer