blob: aad3b0f5f2773e44439835fbc8ab3f75ab700e7c [file] [log] [blame]
Ted Kremenek18c7cee2009-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
Jordy Rose087611e2011-09-02 08:02:59 +000012// (in its current form) could be written as a flow-insensitive check, in
Ted Kremenek18c7cee2009-11-03 08:03:59 +000013// 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 Kyrtzidis9d4d4f92011-02-16 01:40:52 +000018#include "ClangSACheckers.h"
Argyrios Kyrtzidis6a5674f2011-03-01 01:16:21 +000019#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidis507ff532011-02-17 21:39:17 +000020#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Ted Kremenekf8cbac42011-02-10 01:03:03 +000021#include "clang/StaticAnalyzer/Core/BugReporter/BugReporter.h"
Anna Zaks6b1c2122011-10-26 21:06:39 +000022#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Jordan Rose4f7df9b2012-07-26 21:39:41 +000023#include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
Jordy Rose087611e2011-09-02 08:02:59 +000024#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenekf8cbac42011-02-10 01:03:03 +000025#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
Ted Kremenek18c7cee2009-11-03 08:03:59 +000026#include "clang/AST/DeclObjC.h"
27#include "clang/AST/Decl.h"
28
29using namespace clang;
Ted Kremenek98857c92010-12-23 07:20:52 +000030using namespace ento;
Ted Kremenek18c7cee2009-11-03 08:03:59 +000031
32namespace {
Kovarththanan Rajaratnam65c65662009-11-28 06:07:30 +000033class NSAutoreleasePoolChecker
Argyrios Kyrtzidis6a5674f2011-03-01 01:16:21 +000034 : public Checker<check::PreObjCMessage> {
Dylan Noblesmithe2778992012-02-05 02:12:40 +000035 mutable OwningPtr<BugType> BT;
Argyrios Kyrtzidisaad83722011-02-23 07:19:18 +000036 mutable Selector releaseS;
Ted Kremenek18c7cee2009-11-03 08:03:59 +000037
38public:
Jordan Rose547060b2012-07-02 19:28:04 +000039 void checkPreObjCMessage(const ObjCMethodCall &msg, CheckerContext &C) const;
Ted Kremenek18c7cee2009-11-03 08:03:59 +000040};
41
42} // end anonymous namespace
43
Jordan Rose547060b2012-07-02 19:28:04 +000044void NSAutoreleasePoolChecker::checkPreObjCMessage(const ObjCMethodCall &msg,
Argyrios Kyrtzidisaad83722011-02-23 07:19:18 +000045 CheckerContext &C) const {
Jordan Rose547060b2012-07-02 19:28:04 +000046 if (!msg.isInstanceMessage())
Ted Kremenek18c7cee2009-11-03 08:03:59 +000047 return;
Jordan Rose547060b2012-07-02 19:28:04 +000048
49 const ObjCInterfaceDecl *OD = msg.getReceiverInterface();
Ted Kremenek18c7cee2009-11-03 08:03:59 +000050 if (!OD)
51 return;
Jordan Rose547060b2012-07-02 19:28:04 +000052 if (!OD->getIdentifier()->isStr("NSAutoreleasePool"))
Ted Kremenek18c7cee2009-11-03 08:03:59 +000053 return;
Argyrios Kyrtzidisaad83722011-02-23 07:19:18 +000054
55 if (releaseS.isNull())
56 releaseS = GetNullarySelector("release", C.getASTContext());
Ted Kremenek18c7cee2009-11-03 08:03:59 +000057 // Sending 'release' message?
Argyrios Kyrtzidis37ab7262011-01-25 00:03:53 +000058 if (msg.getSelector() != releaseS)
Ted Kremenek18c7cee2009-11-03 08:03:59 +000059 return;
Anna Zaks6b1c2122011-10-26 21:06:39 +000060
61 if (!BT)
62 BT.reset(new BugType("Use -drain instead of -release",
63 "API Upgrade (Apple)"));
64
65 ExplodedNode *N = C.addTransition();
66 if (!N) {
67 assert(0);
68 return;
69 }
70
71 BugReport *Report = new BugReport(*BT, "Use -drain instead of -release when "
72 "using NSAutoreleasePool and garbage collection", N);
73 Report->addRange(msg.getSourceRange());
74 C.EmitReport(Report);
Ted Kremenek18c7cee2009-11-03 08:03:59 +000075}
Argyrios Kyrtzidisaad83722011-02-23 07:19:18 +000076
77void ento::registerNSAutoreleasePoolChecker(CheckerManager &mgr) {
David Blaikiebbafb8a2012-03-11 07:00:24 +000078 if (mgr.getLangOpts().getGC() != LangOptions::NonGC)
Argyrios Kyrtzidisaad83722011-02-23 07:19:18 +000079 mgr.registerChecker<NSAutoreleasePoolChecker>();
80}