diff options
| author | Jordy Rose <jediknil@belkadan.com> | 2010-06-20 04:30:57 +0000 |
|---|---|---|
| committer | Jordy Rose <jediknil@belkadan.com> | 2010-06-20 04:30:57 +0000 |
| commit | c580f2e189810ae655c889536644470575bc551a (patch) | |
| tree | d4e5e454088a4a220b05d69836b09186c58e1c3c /lib/Checker/CastSizeChecker.cpp | |
| parent | 8f9359f5ae1227f3b489d1d261225d8180b64ed3 (diff) | |
Casting to void* or any other pointer-to-sizeless type (e.g. function pointers) causes a divide-by-zero error. Simple fix: check if the pointee type size is 0 and bail out early if it is.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@106401 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Checker/CastSizeChecker.cpp')
| -rw-r--r-- | lib/Checker/CastSizeChecker.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/Checker/CastSizeChecker.cpp b/lib/Checker/CastSizeChecker.cpp index 754d775a65..59ea9e0e84 100644 --- a/lib/Checker/CastSizeChecker.cpp +++ b/lib/Checker/CastSizeChecker.cpp @@ -63,6 +63,11 @@ void CastSizeChecker::PreVisitCastExpr(CheckerContext &C, const CastExpr *CE) { CharUnits RegionSize = CharUnits::fromQuantity(CI->getValue().getSExtValue()); CharUnits TypeSize = C.getASTContext().getTypeSizeInChars(ToPointeeTy); + + // void, and a few other un-sizeable types + if (TypeSize.isZero()) + return; + if (RegionSize % TypeSize != 0) { if (ExplodedNode *N = C.GenerateSink()) { if (!BT) |