[clang] [alpha.webkit.RetainPtrCtorAdoptChecker] Check nullity before calling IgnoreParenCasts. (PR #137556)
Ryosuke Niwa via cfe-commits
cfe-commits at lists.llvm.org
Sun Apr 27 15:18:14 PDT 2025
https://github.com/rniwa created https://github.com/llvm/llvm-project/pull/137556
None
>From 46d5c45c4c7a6abe0255aa31e2c7435d7f3e8289 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa <rniwa at webkit.org>
Date: Sun, 27 Apr 2025 15:16:40 -0700
Subject: [PATCH] [alpha.webkit.RetainPtrCtorAdoptChecker] Check nullity before
calling IgnoreParenCasts.
---
.../Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/clang/lib/StaticAnalyzer/Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp
index bba1d48e6db11..34c98daa78049 100644
--- a/clang/lib/StaticAnalyzer/Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp
+++ b/clang/lib/StaticAnalyzer/Checkers/WebKit/RetainPtrCtorAdoptChecker.cpp
@@ -444,9 +444,10 @@ class RetainPtrCtorAdoptChecker
return false;
if (!ObjCMsgExpr->isInstanceMessage())
return false;
- auto *Receiver = ObjCMsgExpr->getInstanceReceiver()->IgnoreParenCasts();
+ auto *Receiver = ObjCMsgExpr->getInstanceReceiver();
if (!Receiver)
return false;
+ Receiver = Receiver->IgnoreParenCasts();
if (auto *Inner = dyn_cast<ObjCMessageExpr>(Receiver)) {
if (InnerExpr)
*InnerExpr = Inner;
More information about the cfe-commits
mailing list