From: Zhongxing Xu <xuzhongxing@gmail.com>
Date: Sat, 13 Jun 2009 01:31:11 +0000 (+0000)
Subject: Stop tracking non-compound value for struct. It may be caused by imprecise cast
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f0ec39a99b543547609a36a1bca08836db7849a6;p=clang

Stop tracking non-compound value for struct. It may be caused by imprecise cast
logic.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@73279 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Analysis/RegionStore.cpp b/lib/Analysis/RegionStore.cpp
index 8ce37e6069..eae3aefe21 100644
--- a/lib/Analysis/RegionStore.cpp
+++ b/lib/Analysis/RegionStore.cpp
@@ -1198,8 +1198,10 @@ RegionStoreManager::BindStruct(const GRState* St, const TypedRegion* R, SVal V){
   if (V.isUnknown())
     return KillStruct(St, R);
 
-  if (isa<nonloc::SymbolVal>(V))
-    return setDefaultValue(St, R, V);
+  // We may get non-CompoundVal accidentally due to imprecise cast logic. Ignore
+  // them and make struct unknown.
+  if (!isa<nonloc::CompoundVal>(V))
+    return KillStruct(St, R);
 
   nonloc::CompoundVal& CV = cast<nonloc::CompoundVal>(V);
   nonloc::CompoundVal::iterator VI = CV.begin(), VE = CV.end();