Claude review: fbdev: efifb: fix memory leak in efifb_probe()

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

Patch Review

**Status: Correct, but needs verification of the Fixes claim**

```c
 efifb_setup(si, option);
+kfree(option);
```

I verified that `fb_get_options()` calls `kstrdup()` to duplicate the option string, giving ownership to the caller. I also verified that `efifb_setup()` passes `options` by value to `strsep()`, which modifies only the local copy of the pointer — the caller's `option` variable still points to the start of the allocated buffer. So `kfree(option)` is safe.

The key question is whether the referenced commit `73ce73c30ba9` actually changed `fb_get_options()` to use `kstrdup()`. The current code in `fb_cmdline.c` does indeed use `kstrdup()` with a comment saying "The caller owns the string returned in @option and is responsible for releasing the memory." This confirms the fix is correct.

Note this does NOT leak on the `fb_get_options()` failure path since that returns `-ENODEV` before reaching the `kfree`.

---

---
Generated by Claude Code Patch Reviewer