[PATCH] D49057: [analyzer] Track multiple raw pointer symbols in DanglingInternalBufferChecker

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 8 11:00:00 PDT 2018


xazax.hun added a comment.

Thanks! The changes look good, I forgot to mark one double lookup though in my previous review.



================
Comment at: lib/StaticAnalyzer/Checkers/DanglingInternalBufferChecker.cpp:121
+      if (State->contains<RawPtrMap>(ObjRegion)) {
+        NewSet = *State->get<RawPtrMap>(ObjRegion);
+        if (NewSet.contains(RawPtr.getAsSymbol()))
----------------
Oh, there is also a double lookup here, sorry I did not spot it in the initial review.


https://reviews.llvm.org/D49057





More information about the cfe-commits mailing list