123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147 |
- //=== CastSizeChecker.cpp ---------------------------------------*- C++ -*-===//
- //
- // The LLVM Compiler Infrastructure
- //
- // This file is distributed under the University of Illinois Open Source
- // License. See LICENSE.TXT for details.
- //
- //===----------------------------------------------------------------------===//
- //
- // CastSizeChecker checks when casting a malloc'ed symbolic region to type T,
- // whether the size of the symbolic region is a multiple of the size of T.
- //
- //===----------------------------------------------------------------------===//
- #include "ClangSACheckers.h"
- #include "clang/AST/CharUnits.h"
- #include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
- #include "clang/StaticAnalyzer/Core/Checker.h"
- #include "clang/StaticAnalyzer/Core/CheckerManager.h"
- #include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
- using namespace clang;
- using namespace ento;
- namespace {
- class CastSizeChecker : public Checker< check::PreStmt<CastExpr> > {
- mutable std::unique_ptr<BuiltinBug> BT;
- public:
- void checkPreStmt(const CastExpr *CE, CheckerContext &C) const;
- };
- }
- /// Check if we are casting to a struct with a flexible array at the end.
- /// \code
- /// struct foo {
- /// size_t len;
- /// struct bar data[];
- /// };
- /// \endcode
- /// or
- /// \code
- /// struct foo {
- /// size_t len;
- /// struct bar data[0];
- /// }
- /// \endcode
- /// In these cases it is also valid to allocate size of struct foo + a multiple
- /// of struct bar.
- static bool evenFlexibleArraySize(ASTContext &Ctx, CharUnits RegionSize,
- CharUnits TypeSize, QualType ToPointeeTy) {
- const RecordType *RT = ToPointeeTy->getAs<RecordType>();
- if (!RT)
- return false;
- const RecordDecl *RD = RT->getDecl();
- RecordDecl::field_iterator Iter(RD->field_begin());
- RecordDecl::field_iterator End(RD->field_end());
- const FieldDecl *Last = nullptr;
- for (; Iter != End; ++Iter)
- Last = *Iter;
- assert(Last && "empty structs should already be handled");
- const Type *ElemType = Last->getType()->getArrayElementTypeNoTypeQual();
- CharUnits FlexSize;
- if (const ConstantArrayType *ArrayTy =
- Ctx.getAsConstantArrayType(Last->getType())) {
- FlexSize = Ctx.getTypeSizeInChars(ElemType);
- if (ArrayTy->getSize() == 1 && TypeSize > FlexSize)
- TypeSize -= FlexSize;
- else if (ArrayTy->getSize() != 0)
- return false;
- } else if (RD->hasFlexibleArrayMember()) {
- FlexSize = Ctx.getTypeSizeInChars(ElemType);
- } else {
- return false;
- }
- if (FlexSize.isZero())
- return false;
- CharUnits Left = RegionSize - TypeSize;
- if (Left.isNegative())
- return false;
- if (Left % FlexSize == 0)
- return true;
- return false;
- }
- void CastSizeChecker::checkPreStmt(const CastExpr *CE,CheckerContext &C) const {
- const Expr *E = CE->getSubExpr();
- ASTContext &Ctx = C.getASTContext();
- QualType ToTy = Ctx.getCanonicalType(CE->getType());
- const PointerType *ToPTy = dyn_cast<PointerType>(ToTy.getTypePtr());
- if (!ToPTy)
- return;
- QualType ToPointeeTy = ToPTy->getPointeeType();
- // Only perform the check if 'ToPointeeTy' is a complete type.
- if (ToPointeeTy->isIncompleteType())
- return;
- ProgramStateRef state = C.getState();
- const MemRegion *R = state->getSVal(E, C.getLocationContext()).getAsRegion();
- if (!R)
- return;
- const SymbolicRegion *SR = dyn_cast<SymbolicRegion>(R);
- if (!SR)
- return;
- SValBuilder &svalBuilder = C.getSValBuilder();
- SVal extent = SR->getExtent(svalBuilder);
- const llvm::APSInt *extentInt = svalBuilder.getKnownValue(state, extent);
- if (!extentInt)
- return;
- CharUnits regionSize = CharUnits::fromQuantity(extentInt->getSExtValue());
- CharUnits typeSize = C.getASTContext().getTypeSizeInChars(ToPointeeTy);
- // Ignore void, and a few other un-sizeable types.
- if (typeSize.isZero())
- return;
- if (regionSize % typeSize == 0)
- return;
- if (evenFlexibleArraySize(Ctx, regionSize, typeSize, ToPointeeTy))
- return;
- if (ExplodedNode *errorNode = C.generateSink()) {
- if (!BT)
- BT.reset(new BuiltinBug(this, "Cast region with wrong size.",
- "Cast a region whose size is not a multiple"
- " of the destination type size."));
- auto R = llvm::make_unique<BugReport>(*BT, BT->getDescription(), errorNode);
- R->addRange(CE->getSourceRange());
- C.emitReport(std::move(R));
- }
- }
- void ento::registerCastSizeChecker(CheckerManager &mgr) {
- mgr.registerChecker<CastSizeChecker>();
- }
|