Claude review: fbdev: uvesafb: fix potential memory leak in uvesafb_probe()

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

Patch Review

**Status: Correct**

This is the most interesting fix in the series. Instead of adding a new cleanup call, it fixes incorrect `goto` labels:

```c
-		goto out;
+		goto out_mode;
```

I verified in the source tree that `out_mode` (at line 1759) performs modelist destruction, modedb destruction, and cmap deallocation, while `out` (at line 1764) only frees `par->vbe_modes` and releases the framebuffer. Since `uvesafb_vbe_init()` populates the modelist and modedb before these error points, the `goto out` was indeed skipping necessary cleanup. The fix to `goto out_mode` is correct.

Note: calling `fb_dealloc_cmap()` at `out_mode` when `fb_alloc_cmap()` hasn't been called yet (in the `uvesafb_vbe_init_mode()` failure case) is safe — `fb_dealloc_cmap()` handles uninitialized cmaps gracefully.

---

---
Generated by Claude Code Patch Reviewer