eandrews created this revision. eandrews added reviewers: aaron.ballman, tahonermann. Herald added subscribers: manas, ASDenysPetrov, dkrupp, donat.nagy, Szelethus, a.sidorin, baloghadamsoftware. Herald added a reviewer: NoQ. Herald added a project: All. eandrews requested review of this revision.
Fix static analyzer concern about potential null value dereference. findBackingIvar() dereferences Prop. PR checks that Prop exists before calling the function. https://reviews.llvm.org/D157429 Files: clang/lib/Analysis/BodyFarm.cpp Index: clang/lib/Analysis/BodyFarm.cpp =================================================================== --- clang/lib/Analysis/BodyFarm.cpp +++ clang/lib/Analysis/BodyFarm.cpp @@ -806,7 +806,7 @@ if (!IVar) { Prop = MD->findPropertyDecl(); - IVar = findBackingIvar(Prop); + IVar = Prop? findBackingIvar(Prop) : nullptr; } if (!IVar || !Prop)
Index: clang/lib/Analysis/BodyFarm.cpp =================================================================== --- clang/lib/Analysis/BodyFarm.cpp +++ clang/lib/Analysis/BodyFarm.cpp @@ -806,7 +806,7 @@ if (!IVar) { Prop = MD->findPropertyDecl(); - IVar = findBackingIvar(Prop); + IVar = Prop? findBackingIvar(Prop) : nullptr; } if (!IVar || !Prop)
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits