"llvm/lib/git@repo.hca.bsc.es:lalbano/llvm-bpevl.git" did not exist on "14cb0bdf2b6ca0b7befbb07fe9f73dad5786f59b"
Newer
Older
Ted Kremenek
committed
//=- NSAutoreleasePoolChecker.cpp --------------------------------*- C++ -*-==//
//
// The LLVM Compiler Infrastructure
//
// This file is distributed under the University of Illinois Open Source
// License. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
//
// This file defines a NSAutoreleasePoolChecker, a small checker that warns
// about subpar uses of NSAutoreleasePool. Note that while the check itself
// (in its current form) could be written as a flow-insensitive check, in
Ted Kremenek
committed
// can be potentially enhanced in the future with flow-sensitive information.
// It is also a good example of the CheckerVisitor interface.
//
//===----------------------------------------------------------------------===//
#include "ClangSACheckers.h"
#include "clang/StaticAnalyzer/Core/Checker.h"
#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Ted Kremenek
committed
#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenek
committed
#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
#include "clang/StaticAnalyzer/Core/PathSensitive/ObjCMessage.h"
Ted Kremenek
committed
#include "clang/AST/DeclObjC.h"
#include "clang/AST/Decl.h"
using namespace clang;
Ted Kremenek
committed
namespace {
Kovarththanan Rajaratnam
committed
class NSAutoreleasePoolChecker
: public Checker<check::PreObjCMessage> {
mutable OwningPtr<BugType> BT;
Ted Kremenek
committed
public:
void checkPreObjCMessage(ObjCMessage msg, CheckerContext &C) const;
Ted Kremenek
committed
};
} // end anonymous namespace
void NSAutoreleasePoolChecker::checkPreObjCMessage(ObjCMessage msg,
CheckerContext &C) const {
Ted Kremenek
committed
Argyrios Kyrtzidis
committed
const Expr *receiver = msg.getInstanceReceiver();
Ted Kremenek
committed
if (!receiver)
return;
// FIXME: Enhance with value-tracking information instead of consulting
// the type of the expression.
const ObjCObjectPointerType* PT =
receiver->getType()->getAs<ObjCObjectPointerType>();
Ted Kremenek
committed
if (!PT)
return;
Ted Kremenek
committed
const ObjCInterfaceDecl *OD = PT->getInterfaceDecl();
Ted Kremenek
committed
if (!OD)
return;
if (!OD->getIdentifier()->getName().equals("NSAutoreleasePool"))
return;
if (releaseS.isNull())
releaseS = GetNullarySelector("release", C.getASTContext());
Ted Kremenek
committed
// Sending 'release' message?
Argyrios Kyrtzidis
committed
if (msg.getSelector() != releaseS)
Ted Kremenek
committed
return;
if (!BT)
BT.reset(new BugType("Use -drain instead of -release",
"API Upgrade (Apple)"));
ExplodedNode *N = C.addTransition();
if (!N) {
assert(0);
return;
}
BugReport *Report = new BugReport(*BT, "Use -drain instead of -release when "
"using NSAutoreleasePool and garbage collection", N);
Report->addRange(msg.getSourceRange());
C.EmitReport(Report);
Ted Kremenek
committed
}
void ento::registerNSAutoreleasePoolChecker(CheckerManager &mgr) {
David Blaikie
committed
if (mgr.getLangOpts().getGC() != LangOptions::NonGC)
mgr.registerChecker<NSAutoreleasePoolChecker>();
}