12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394 |
- //== ObjCAtSyncChecker.cpp - nil mutex checker for @synchronized -*- C++ -*--=//
- //
- // The LLVM Compiler Infrastructure
- //
- // This file is distributed under the University of Illinois Open Source
- // License. See LICENSE.TXT for details.
- //
- //===----------------------------------------------------------------------===//
- //
- // This defines ObjCAtSyncChecker, a builtin check that checks for null pointers
- // used as mutexes for @synchronized.
- //
- //===----------------------------------------------------------------------===//
- #include "ClangSACheckers.h"
- #include "clang/AST/StmtObjC.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"
- #include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
- using namespace clang;
- using namespace ento;
- namespace {
- class ObjCAtSyncChecker
- : public Checker< check::PreStmt<ObjCAtSynchronizedStmt> > {
- mutable std::unique_ptr<BuiltinBug> BT_null;
- mutable std::unique_ptr<BuiltinBug> BT_undef;
- public:
- void checkPreStmt(const ObjCAtSynchronizedStmt *S, CheckerContext &C) const;
- };
- } // end anonymous namespace
- void ObjCAtSyncChecker::checkPreStmt(const ObjCAtSynchronizedStmt *S,
- CheckerContext &C) const {
- const Expr *Ex = S->getSynchExpr();
- ProgramStateRef state = C.getState();
- SVal V = C.getSVal(Ex);
- // Uninitialized value used for the mutex?
- if (V.getAs<UndefinedVal>()) {
- if (ExplodedNode *N = C.generateErrorNode()) {
- if (!BT_undef)
- BT_undef.reset(new BuiltinBug(this, "Uninitialized value used as mutex "
- "for @synchronized"));
- auto report =
- llvm::make_unique<BugReport>(*BT_undef, BT_undef->getDescription(), N);
- bugreporter::trackNullOrUndefValue(N, Ex, *report);
- C.emitReport(std::move(report));
- }
- return;
- }
- if (V.isUnknown())
- return;
- // Check for null mutexes.
- ProgramStateRef notNullState, nullState;
- std::tie(notNullState, nullState) = state->assume(V.castAs<DefinedSVal>());
- if (nullState) {
- if (!notNullState) {
- // Generate an error node. This isn't a sink since
- // a null mutex just means no synchronization occurs.
- if (ExplodedNode *N = C.generateNonFatalErrorNode(nullState)) {
- if (!BT_null)
- BT_null.reset(new BuiltinBug(
- this, "Nil value used as mutex for @synchronized() "
- "(no synchronization will occur)"));
- auto report =
- llvm::make_unique<BugReport>(*BT_null, BT_null->getDescription(), N);
- bugreporter::trackNullOrUndefValue(N, Ex, *report);
- C.emitReport(std::move(report));
- return;
- }
- }
- // Don't add a transition for 'nullState'. If the value is
- // under-constrained to be null or non-null, assume it is non-null
- // afterwards.
- }
- if (notNullState)
- C.addTransition(notNullState);
- }
- void ento::registerObjCAtSyncChecker(CheckerManager &mgr) {
- if (mgr.getLangOpts().ObjC2)
- mgr.registerChecker<ObjCAtSyncChecker>();
- }
|