[PATCH] D30909: [Analyzer] Finish taint propagation to derived symbols of tainted regions

Vlad Tsyrklevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 13 14:15:29 PDT 2017


vlad.tsyrklevich updated this revision to Diff 91614.
vlad.tsyrklevich added a comment.

Fix a stray assert()


https://reviews.llvm.org/D30909

Files:
  lib/StaticAnalyzer/Core/ProgramState.cpp


Index: lib/StaticAnalyzer/Core/ProgramState.cpp
===================================================================
--- lib/StaticAnalyzer/Core/ProgramState.cpp
+++ lib/StaticAnalyzer/Core/ProgramState.cpp
@@ -679,9 +679,9 @@
 
     SymRegions = SymRegions.add(SD->getRegion());
     NewState = NewState->set<DerivedSymTaint>(SD->getParentSymbol(), SymRegions);
+    assert(NewState);
   }
 
-  assert(NewState);
   return NewState;
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30909.91614.patch
Type: text/x-patch
Size: 438 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170313/46b26220/attachment.bin>


More information about the cfe-commits mailing list