From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/bridge: it6505: fix use-after-free in it6505_parse_dt() Date: Sun, 12 Apr 2026 13:40:31 +1000 Message-ID: In-Reply-To: <20260407093800.291489-1-vulab@iscas.ac.cn> References: <20260407093800.291489-1-vulab@iscas.ac.cn> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/bridge: it6505: fix use-after-free in it6505_parse_dt() Author: Wentao Liang Patches: 1 Reviewed: 2026-04-12T13:40:31.451726 --- This is a single patch claiming to fix a use-after-free in `it6505_parse_dt= ()`. The claim is that `of_node_put(ep)` is called before `ep` is used insi= de the `if (ep)` block. While the diagnosis of a real bug is correct =E2=80= =94 calling `of_node_put()` before using the node is wrong =E2=80=94 there = is a significant gap in the fix: it only places `of_node_put(ep)` on the `i= f (ep)` (true) path and **omits it on the `else` path and on early-return/e= rror paths**. However, since `ep` is `NULL` in the `else` branch, `of_node_= put(NULL)` is a no-op, so this omission is actually correct behavior (not a= leak). The fix is functionally correct. **Verdict: The patch is correct and fixes a real bug, but is cosmetically i= mperfect in its commit message framing.** --- --- Generated by Claude Code Patch Reviewer