blob: e63d2b2082a1a736b7b835ce41e676a19da5638c [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 Kyrtzidis0b1ba622011-02-16 01:40:52 +000018#include "ClangSACheckers.h"
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000019#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidis695fb502011-02-17 21:39:17 +000020#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000021#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenek9b663712011-02-10 01:03:03 +000022#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
23#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000024#include "clang/AST/DeclObjC.h"
25#include "clang/AST/Decl.h"
26
27using namespace clang;
Ted Kremenek9ef65372010-12-23 07:20:52 +000028using namespace ento;
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000029
30namespace {
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000031class NSAutoreleasePoolChecker
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000032 : public Checker<check::PreObjCMessage> {
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000033
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000034 mutable Selector releaseS;
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000035
36public:
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000037 void checkPreObjCMessage(ObjCMessage msg, CheckerContext &C) const;
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000038};
39
40} // end anonymous namespace
41
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000042void NSAutoreleasePoolChecker::checkPreObjCMessage(ObjCMessage msg,
43 CheckerContext &C) const {
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000044
Argyrios Kyrtzidis432424d2011-01-25 00:03:53 +000045 const Expr *receiver = msg.getInstanceReceiver();
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000046 if (!receiver)
47 return;
48
49 // FIXME: Enhance with value-tracking information instead of consulting
50 // the type of the expression.
51 const ObjCObjectPointerType* PT =
52 receiver->getType()->getAs<ObjCObjectPointerType>();
Ted Kremenek71a5e282009-11-20 00:12:36 +000053
54 if (!PT)
55 return;
Ted Kremenek9c378f72011-08-12 23:37:29 +000056 const ObjCInterfaceDecl *OD = PT->getInterfaceDecl();
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000057 if (!OD)
58 return;
59 if (!OD->getIdentifier()->getName().equals("NSAutoreleasePool"))
60 return;
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000061
62 if (releaseS.isNull())
63 releaseS = GetNullarySelector("release", C.getASTContext());
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000064 // Sending 'release' message?
Argyrios Kyrtzidis432424d2011-01-25 00:03:53 +000065 if (msg.getSelector() != releaseS)
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000066 return;
67
Argyrios Kyrtzidis432424d2011-01-25 00:03:53 +000068 SourceRange R = msg.getSourceRange();
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000069
70 C.getBugReporter().EmitBasicReport("Use -drain instead of -release",
71 "API Upgrade (Apple)",
72 "Use -drain instead of -release when using NSAutoreleasePool "
Argyrios Kyrtzidis432424d2011-01-25 00:03:53 +000073 "and garbage collection", R.getBegin(), &R, 1);
Ted Kremenek54cb7cc2009-11-03 08:03:59 +000074}
Argyrios Kyrtzidis45d9b4e2011-02-23 07:19:18 +000075
76void ento::registerNSAutoreleasePoolChecker(CheckerManager &mgr) {
77 if (mgr.getLangOptions().getGCMode() != LangOptions::NonGC)
78 mgr.registerChecker<NSAutoreleasePoolChecker>();
79}