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: mm: Prefer IS_ERR_OR_NULL over manual NULL check
Date: Thu, 12 Mar 2026 07:27:19 +1000
Message-ID: <review-patch33-20260310-b4-is_err_or_null-v1-33-bd63b656022d@avm.de>
In-Reply-To: <20260310-b4-is_err_or_null-v1-33-bd63b656022d@avm.de>
References: <abBlpGKO842B3yl9@google.com> <20260310-b4-is_err_or_null-v1-33-bd63b656022d@avm.de>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

The eight kmemleak changes are correct. The `kmemleak_scan_area` case reord=
ers `size` and `!IS_ERR_OR_NULL(ptr)` compared to the original =E2=80=94 th=
is is fine since `&&` is commutative for side-effect-free expressions:
```c
-	if (kmemleak_enabled && ptr && size && !IS_ERR(ptr))
+	if (kmemleak_enabled && size && !IS_ERR_OR_NULL(ptr))
```

---
Generated by Claude Code Patch Reviewer