Claude review: drm/qxl: fix local_gobj leak in qxl_gem_object_create_with_handle

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

Patch Review

**Bug analysis is correct.** Looking at the function in `drivers/gpu/drm/qxl/qxl_gem.c:82-114`:

1. `qxl_gem_object_create()` (line 95) creates a GEM object and returns it via `local_gobj` with a reference held.
2. `drm_gem_handle_create()` (line 101) takes its own reference on the object if it succeeds.
3. If `drm_gem_handle_create()` fails, the original code returned `r` immediately without dropping the reference from step 1 — leaking the object.

**The fix is correct:**
```c
r = drm_gem_handle_create(file_priv, local_gobj, handle);
if (r) {
    drm_gem_object_put(local_gobj);
    return r;
}
```

Adding `drm_gem_object_put(local_gobj)` in the error path properly releases the reference obtained from `qxl_gem_object_create()`.

**Note the existing code on the success path** (lines 107-111) already demonstrates the correct pattern — when `gobj` is NULL, it drops the allocate reference since the handle holds one. The error path was simply missing the same cleanup.

**Minor nit on the Fixes tag format:** The tag reads `Fixes: f64122c1f6ad("drm: add new QXL driver. (v1.4)")` but the standard kernel format has a space before the parenthesis: `Fixes: f64122c1f6ad ("drm: add new QXL driver. (v1.4)")`. This is cosmetic but worth fixing for consistency with `gitlint` and kernel conventions.

**No other issues.** The patch is minimal, correctly scoped, and addresses a real resource leak that has existed since the driver was introduced.

---
Generated by Claude Code Patch Reviewer