blob: beb4a81743b7b92f3208fed05df4d65da866f8a9 [file] [log] [blame]
Ted Kremenek54cb7cc2009-11-03 08:03:59 +00001//=- NSAutoreleasePoolChecker.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// This file defines a NSAutoreleasePoolChecker, a small checker that warns
11// about subpar uses of NSAutoreleasePool. Note that while the check itself
12// (in it's current form) could be written as a flow-insensitive check, in
13// can be potentially enhanced in the future with flow-sensitive information.
14// It is also a good example of the CheckerVisitor interface.
15//
16//===----------------------------------------------------------------------===//
17
Argyrios Kyrtzidis98cabba2010-12-22 18:51:49 +000018#include "clang/GR/BugReporter/BugReporter.h"
Argyrios Kyrtzidisd2592a32010-12-22 18:53:44 +000019#include "clang/GR/PathSensitive/ExprEngine.h"
Argyrios Kyrtzidis98cabba2010-12-22 18:51:49 +000020#include "clang/GR/PathSensitive/CheckerVisitor.h"
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000021#include "BasicObjCFoundationChecks.h"
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000022#include "clang/AST/DeclObjC.h"
23#include "clang/AST/Decl.h"
24
25using namespace clang;
Argyrios Kyrtzidis5a4f98f2010-12-22 18:53:20 +000026using namespace GR;
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000027
28namespace {
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000029class NSAutoreleasePoolChecker
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000030 : public CheckerVisitor<NSAutoreleasePoolChecker> {
31
32 Selector releaseS;
33
34public:
35 NSAutoreleasePoolChecker(Selector release_s) : releaseS(release_s) {}
36
37 static void *getTag() {
38 static int x = 0;
39 return &x;
40 }
41
42 void PreVisitObjCMessageExpr(CheckerContext &C, const ObjCMessageExpr *ME);
43};
44
45} // end anonymous namespace
46
47
Argyrios Kyrtzidisd2592a32010-12-22 18:53:44 +000048void GR::RegisterNSAutoreleasePoolChecks(ExprEngine &Eng) {
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000049 ASTContext &Ctx = Eng.getContext();
50 if (Ctx.getLangOptions().getGCMode() != LangOptions::NonGC) {
51 Eng.registerCheck(new NSAutoreleasePoolChecker(GetNullarySelector("release",
52 Ctx)));
53 }
54}
55
56void
57NSAutoreleasePoolChecker::PreVisitObjCMessageExpr(CheckerContext &C,
58 const ObjCMessageExpr *ME) {
59
Douglas Gregor04badcf2010-04-21 00:45:42 +000060 const Expr *receiver = ME->getInstanceReceiver();
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000061 if (!receiver)
62 return;
63
64 // FIXME: Enhance with value-tracking information instead of consulting
65 // the type of the expression.
66 const ObjCObjectPointerType* PT =
67 receiver->getType()->getAs<ObjCObjectPointerType>();
Ted Kremenek71a5e282009-11-20 00:12:36 +000068
69 if (!PT)
70 return;
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000071 const ObjCInterfaceDecl* OD = PT->getInterfaceDecl();
72 if (!OD)
73 return;
74 if (!OD->getIdentifier()->getName().equals("NSAutoreleasePool"))
75 return;
76
77 // Sending 'release' message?
78 if (ME->getSelector() != releaseS)
79 return;
80
81 SourceRange R = ME->getSourceRange();
82
83 C.getBugReporter().EmitBasicReport("Use -drain instead of -release",
84 "API Upgrade (Apple)",
85 "Use -drain instead of -release when using NSAutoreleasePool "
86 "and garbage collection", ME->getLocStart(), &R, 1);
87}