blob: 6676fe5e7a36606397c14c21f96f9f46767b1dd7 [file] [log] [blame]
Zhongxing Xuab280992010-05-25 04:59:19 +00001//=== CastSizeChecker.cpp ---------------------------------------*- C++ -*-===//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
10// CastSizeChecker checks when casting a malloc'ed symbolic region to type T,
11// whether the size of the symbolic region is a multiple of the size of T.
12//
13//===----------------------------------------------------------------------===//
14#include "clang/AST/CharUnits.h"
15#include "clang/Checker/BugReporter/BugType.h"
16#include "clang/Checker/PathSensitive/CheckerVisitor.h"
17#include "GRExprEngineInternalChecks.h"
18
19using namespace clang;
20
21namespace {
22class CastSizeChecker : public CheckerVisitor<CastSizeChecker> {
23 BuiltinBug *BT;
24public:
25 CastSizeChecker() : BT(0) {}
26 static void *getTag();
27 void PreVisitCastExpr(CheckerContext &C, const CastExpr *B);
28};
29}
30
31void *CastSizeChecker::getTag() {
32 static int x;
33 return &x;
34}
35
36void CastSizeChecker::PreVisitCastExpr(CheckerContext &C, const CastExpr *CE) {
37 const Expr *E = CE->getSubExpr();
38 ASTContext &Ctx = C.getASTContext();
39 QualType ToTy = Ctx.getCanonicalType(CE->getType());
40 PointerType *ToPTy = dyn_cast<PointerType>(ToTy.getTypePtr());
41
42 if (!ToPTy)
43 return;
44
45 QualType ToPointeeTy = ToPTy->getPointeeType();
46
Ted Kremenekf69cf182010-09-01 20:35:38 +000047 // Only perform the check if 'ToPointeeTy' is a complete type.
48 if (ToPointeeTy->isIncompleteType())
49 return;
50
Jordy Rose32f26562010-07-04 00:00:41 +000051 const GRState *state = C.getState();
52 const MemRegion *R = state->getSVal(E).getAsRegion();
Zhongxing Xuab280992010-05-25 04:59:19 +000053 if (R == 0)
54 return;
55
56 const SymbolicRegion *SR = dyn_cast<SymbolicRegion>(R);
57 if (SR == 0)
58 return;
59
Jordy Rose32f26562010-07-04 00:00:41 +000060 ValueManager &ValMgr = C.getValueManager();
61 SVal Extent = SR->getExtent(ValMgr);
62
63 SValuator &SVator = ValMgr.getSValuator();
64 const llvm::APSInt *ExtentInt = SVator.getKnownValue(state, Extent);
65 if (!ExtentInt)
Zhongxing Xuab280992010-05-25 04:59:19 +000066 return;
67
Jordy Rose32f26562010-07-04 00:00:41 +000068 CharUnits RegionSize = CharUnits::fromQuantity(ExtentInt->getSExtValue());
Zhongxing Xuab280992010-05-25 04:59:19 +000069 CharUnits TypeSize = C.getASTContext().getTypeSizeInChars(ToPointeeTy);
Jordy Rosec580f2e2010-06-20 04:30:57 +000070
Jordy Rose32f26562010-07-04 00:00:41 +000071 // Ignore void, and a few other un-sizeable types.
Jordy Rosec580f2e2010-06-20 04:30:57 +000072 if (TypeSize.isZero())
73 return;
74
Zhongxing Xuab280992010-05-25 04:59:19 +000075 if (RegionSize % TypeSize != 0) {
76 if (ExplodedNode *N = C.GenerateSink()) {
77 if (!BT)
78 BT = new BuiltinBug("Cast region with wrong size.",
79 "Cast a region whose size is not a multiple of the"
80 " destination type size.");
81 RangedBugReport *R = new RangedBugReport(*BT, BT->getDescription(), N);
82 R->addRange(CE->getSourceRange());
83 C.EmitReport(R);
84 }
85 }
86}
87
88
89void clang::RegisterCastSizeChecker(GRExprEngine &Eng) {
90 Eng.registerCheck(new CastSizeChecker());
91}