From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: fbdev: broadsheetfb: fix potential memory leak in broadsheetfb_probe()
Date: Sat, 16 May 2026 11:10:40 +1000
Message-ID: <review-patch2-20260514-fbdev-v1-2-b3a2474fa720@cse.iitm.ac.in>
In-Reply-To: <20260514-fbdev-v1-2-b3a2474fa720@cse.iitm.ac.in>
References: <20260514-fbdev-v1-0-b3a2474fa720@cse.iitm.ac.in> <20260514-fbdev-v1-2-b3a2474fa720@cse.iitm.ac.in>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

**Status: Correct**

Same pattern as patch 1 =E2=80=94 captures return value of `fb_deferred_io_=
init()`, adds `err_fbdefio` label between `err_cmap` and `err_vfree`:

```c
 err_cmap:
 	fb_dealloc_cmap(&info->cmap);
+err_fbdefio:
+	fb_deferred_io_cleanup(info);
 err_vfree:
```

The label ordering is correct: `fb_alloc_cmap()` failure now goes to `err_f=
bdefio` (cleaning up deferred IO before vfree), and `err_cmap` falls throug=
h to `err_fbdefio`.

---

---
Generated by Claude Code Patch Reviewer