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

[Vlad Tsyrklevich via Phabricator via cfe-commits]

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;
 }
 



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;
 }
 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits