diff options
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Checker/RegionStore.cpp | 4 | ||||
-rw-r--r-- | lib/Checker/Store.cpp | 11 |
2 files changed, 6 insertions, 9 deletions
diff --git a/lib/Checker/RegionStore.cpp b/lib/Checker/RegionStore.cpp index 595fb6f19b..19945a7b3c 100644 --- a/lib/Checker/RegionStore.cpp +++ b/lib/Checker/RegionStore.cpp @@ -1087,6 +1087,10 @@ SVal RegionStoreManager::RetrieveElement(Store store, return ValMgr.makeIntVal(c, T); } } + + // Check for loads from a code text region. For such loads, just give up. + if (const CodeTextRegion *cR = dyn_cast<CodeTextRegion>(superR)) + return UnknownVal(); // Handle the case where we are indexing into a larger scalar object. // For example, this handles: diff --git a/lib/Checker/Store.cpp b/lib/Checker/Store.cpp index 7c80eed0ea..1cb5cd70ca 100644 --- a/lib/Checker/Store.cpp +++ b/lib/Checker/Store.cpp @@ -101,17 +101,10 @@ const MemRegion *StoreManager::CastRegion(const MemRegion *R, QualType CastToTy) assert(0 && "Invalid region cast"); break; } - + case MemRegion::FunctionTextRegionKind: case MemRegion::BlockTextRegionKind: - case MemRegion::BlockDataRegionKind: { - // CodeTextRegion should be cast to only a function or block pointer type, - // although they can in practice be casted to anything, e.g, void*, char*, - // etc. - // Just return the region. - return R; - } - + case MemRegion::BlockDataRegionKind: case MemRegion::StringRegionKind: // FIXME: Need to handle arbitrary downcasts. case MemRegion::SymbolicRegionKind: |