[PATCH] D130974: [analyzer] Fix for the crash in #56873
Domján Dániel via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 9 05:16:06 PDT 2022
isuckatcs added a comment.
> Some checker should have caught the uninitialized value earlier than the defaultEvalCall().
> I guess, the MallocCkecher could have checked for it in PreStmt<CXXNewExpr>.
> Or alternatively, the CallAndMessageChecker::preCall() already does something like this in the PreVisitProcessArg(). I know that CXXNewExpr is not a call, but you get the idea.
> WDYT, worth catching it?
I definitely think it's worth catching it. I'm working on a checker which addresses this in D131299 <https://reviews.llvm.org/D131299>. It was originally intended to be a part of MallocChecker but has been moved to a separate one.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130974/new/
https://reviews.llvm.org/D130974
More information about the cfe-commits
mailing list