blob: 9b56c9fe137b2c8adf85009f19d52abc13ee3439 [file] [log] [blame]
Ted Kremenekbc3a0212009-10-30 17:24:47 +00001//== NullDerefChecker.cpp - Null dereference checker ------------*- 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//
Argyrios Kyrtzidisd2592a32010-12-22 18:53:44 +000010// This defines NullDerefChecker, a builtin check in ExprEngine that performs
Ted Kremenekbc3a0212009-10-30 17:24:47 +000011// checks for null pointers at loads and stores.
12//
13//===----------------------------------------------------------------------===//
14
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000015#include "ClangSACheckers.h"
Benjamin Kramerc35fb7d2012-01-28 12:06:22 +000016#include "clang/AST/ExprObjC.h"
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000017#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000018#include "clang/StaticAnalyzer/Core/CheckerManager.h"
19#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenek9b663712011-02-10 01:03:03 +000020#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Benjamin Kramer8fe83e12012-02-04 13:45:25 +000021#include "llvm/ADT/SmallString.h"
Ted Kremenekbc3a0212009-10-30 17:24:47 +000022
23using namespace clang;
Ted Kremenek9ef65372010-12-23 07:20:52 +000024using namespace ento;
Ted Kremenekbc3a0212009-10-30 17:24:47 +000025
Ted Kremenekb4b817d2009-11-11 03:26:34 +000026namespace {
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000027class DereferenceChecker
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000028 : public Checker< check::Location,
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000029 EventDispatcher<ImplicitNullDerefEvent> > {
Dylan Noblesmith6f42b622012-02-05 02:12:40 +000030 mutable OwningPtr<BuiltinBug> BT_null;
31 mutable OwningPtr<BuiltinBug> BT_undef;
Ted Kremenek452b84d2010-03-23 01:11:38 +000032
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000033public:
Anna Zaks390909c2011-10-06 00:43:15 +000034 void checkLocation(SVal location, bool isLoad, const Stmt* S,
35 CheckerContext &C) const;
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000036
Ted Kremenek76aadc32012-03-09 01:13:14 +000037 static const MemRegion *AddDerefSource(raw_ostream &os,
Chris Lattner5f9e2722011-07-23 10:55:15 +000038 SmallVectorImpl<SourceRange> &Ranges,
Ted Kremenek76aadc32012-03-09 01:13:14 +000039 const Expr *Ex, const ProgramState *state,
40 const LocationContext *LCtx,
41 bool loadedFrom = false);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000042};
43} // end anonymous namespace
44
Ted Kremenek76aadc32012-03-09 01:13:14 +000045const MemRegion *
46DereferenceChecker::AddDerefSource(raw_ostream &os,
47 SmallVectorImpl<SourceRange> &Ranges,
48 const Expr *Ex,
49 const ProgramState *state,
50 const LocationContext *LCtx,
51 bool loadedFrom) {
John McCallf6a16482010-12-04 03:47:34 +000052 Ex = Ex->IgnoreParenLValueCasts();
Ted Kremenek76aadc32012-03-09 01:13:14 +000053 const MemRegion *sourceR = 0;
Ted Kremenek646c3c32010-10-26 00:06:13 +000054 switch (Ex->getStmtClass()) {
55 default:
Ted Kremenek76aadc32012-03-09 01:13:14 +000056 break;
Ted Kremenek646c3c32010-10-26 00:06:13 +000057 case Stmt::DeclRefExprClass: {
58 const DeclRefExpr *DR = cast<DeclRefExpr>(Ex);
59 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
60 os << " (" << (loadedFrom ? "loaded from" : "from")
61 << " variable '" << VD->getName() << "')";
62 Ranges.push_back(DR->getSourceRange());
Ted Kremenek76aadc32012-03-09 01:13:14 +000063 sourceR = state->getLValue(VD, LCtx).getAsRegion();
Ted Kremenek646c3c32010-10-26 00:06:13 +000064 }
Ted Kremenek76aadc32012-03-09 01:13:14 +000065 break;
Ted Kremenek646c3c32010-10-26 00:06:13 +000066 }
67 case Stmt::MemberExprClass: {
68 const MemberExpr *ME = cast<MemberExpr>(Ex);
69 os << " (" << (loadedFrom ? "loaded from" : "via")
70 << " field '" << ME->getMemberNameInfo() << "')";
71 SourceLocation L = ME->getMemberLoc();
72 Ranges.push_back(SourceRange(L, L));
73 break;
74 }
75 }
Ted Kremenek76aadc32012-03-09 01:13:14 +000076 return sourceR;
Ted Kremenek646c3c32010-10-26 00:06:13 +000077}
78
Anna Zaks390909c2011-10-06 00:43:15 +000079void DereferenceChecker::checkLocation(SVal l, bool isLoad, const Stmt* S,
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000080 CheckerContext &C) const {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000081 // Check for dereference of an undefined value.
82 if (l.isUndef()) {
Ted Kremenekd048c6e2010-12-20 21:19:09 +000083 if (ExplodedNode *N = C.generateSink()) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000084 if (!BT_undef)
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000085 BT_undef.reset(new BuiltinBug("Dereference of undefined pointer value"));
Ted Kremenek452b84d2010-03-23 01:11:38 +000086
Anna Zakse172e8b2011-08-17 23:00:25 +000087 BugReport *report =
88 new BugReport(*BT_undef, BT_undef->getDescription(), N);
Anna Zaks50bbc162011-08-19 22:33:38 +000089 report->addVisitor(bugreporter::getTrackNullOrUndefValueVisitor(N,
Ted Kremenek76aadc32012-03-09 01:13:14 +000090 bugreporter::GetDerefExpr(N), report));
Ted Kremeneked7948b2012-05-31 06:03:17 +000091 report->disablePathPruning();
Ted Kremenekb4b817d2009-11-11 03:26:34 +000092 C.EmitReport(report);
93 }
94 return;
95 }
Ted Kremenek452b84d2010-03-23 01:11:38 +000096
Ted Kremenekb4b817d2009-11-11 03:26:34 +000097 DefinedOrUnknownSVal location = cast<DefinedOrUnknownSVal>(l);
Ted Kremenek452b84d2010-03-23 01:11:38 +000098
99 // Check for null dereferences.
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000100 if (!isa<Loc>(location))
101 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000102
Ted Kremenek8bef8232012-01-26 21:29:00 +0000103 ProgramStateRef state = C.getState();
Ted Kremenek76aadc32012-03-09 01:13:14 +0000104 const LocationContext *LCtx = C.getLocationContext();
Ted Kremenek8bef8232012-01-26 21:29:00 +0000105 ProgramStateRef notNullState, nullState;
Ted Kremenek28f47b92010-12-01 22:16:56 +0000106 llvm::tie(notNullState, nullState) = state->assume(location);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000107
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000108 // The explicit NULL case.
109 if (nullState) {
Ted Kremenek452b84d2010-03-23 01:11:38 +0000110 if (!notNullState) {
Ted Kremenek78d722f2009-11-21 01:50:48 +0000111 // Generate an error node.
Ted Kremenekd048c6e2010-12-20 21:19:09 +0000112 ExplodedNode *N = C.generateSink(nullState);
Ted Kremenek78d722f2009-11-21 01:50:48 +0000113 if (!N)
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000114 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000115
Ted Kremenek78d722f2009-11-21 01:50:48 +0000116 // We know that 'location' cannot be non-null. This is what
Ted Kremenek452b84d2010-03-23 01:11:38 +0000117 // we call an "explicit" null dereference.
Ted Kremenek78d722f2009-11-21 01:50:48 +0000118 if (!BT_null)
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000119 BT_null.reset(new BuiltinBug("Dereference of null pointer"));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000120
Dylan Noblesmithf7ccbad2012-02-05 02:13:05 +0000121 SmallString<100> buf;
Chris Lattner5f9e2722011-07-23 10:55:15 +0000122 SmallVector<SourceRange, 2> Ranges;
Ted Kremenek892697d2010-12-16 07:46:53 +0000123
124 // Walk through lvalue casts to get the original expression
125 // that syntactically caused the load.
126 if (const Expr *expr = dyn_cast<Expr>(S))
127 S = expr->IgnoreParenLValueCasts();
Ted Kremenek76aadc32012-03-09 01:13:14 +0000128
129 const MemRegion *sourceR = 0;
Ted Kremeneke576af22009-11-24 01:33:10 +0000130
131 switch (S->getStmtClass()) {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000132 case Stmt::ArraySubscriptExprClass: {
133 llvm::raw_svector_ostream os(buf);
134 os << "Array access";
135 const ArraySubscriptExpr *AE = cast<ArraySubscriptExpr>(S);
Ted Kremenek76aadc32012-03-09 01:13:14 +0000136 sourceR =
137 AddDerefSource(os, Ranges, AE->getBase()->IgnoreParenCasts(),
138 state.getPtr(), LCtx);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000139 os << " results in a null pointer dereference";
140 break;
141 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000142 case Stmt::UnaryOperatorClass: {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000143 llvm::raw_svector_ostream os(buf);
144 os << "Dereference of null pointer";
Ted Kremeneke576af22009-11-24 01:33:10 +0000145 const UnaryOperator *U = cast<UnaryOperator>(S);
Ted Kremenek76aadc32012-03-09 01:13:14 +0000146 sourceR =
147 AddDerefSource(os, Ranges, U->getSubExpr()->IgnoreParens(),
148 state.getPtr(), LCtx, true);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000149 break;
150 }
151 case Stmt::MemberExprClass: {
152 const MemberExpr *M = cast<MemberExpr>(S);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000153 if (M->isArrow()) {
154 llvm::raw_svector_ostream os(buf);
155 os << "Access to field '" << M->getMemberNameInfo()
156 << "' results in a dereference of a null pointer";
Ted Kremenek76aadc32012-03-09 01:13:14 +0000157 sourceR =
158 AddDerefSource(os, Ranges, M->getBase()->IgnoreParenCasts(),
159 state.getPtr(), LCtx, true);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000160 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000161 break;
Ted Kremeneke576af22009-11-24 01:33:10 +0000162 }
Ted Kremenek0853c7f2010-09-10 20:20:49 +0000163 case Stmt::ObjCIvarRefExprClass: {
164 const ObjCIvarRefExpr *IV = cast<ObjCIvarRefExpr>(S);
165 if (const DeclRefExpr *DR =
166 dyn_cast<DeclRefExpr>(IV->getBase()->IgnoreParenCasts())) {
167 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
168 llvm::raw_svector_ostream os(buf);
169 os << "Instance variable access (via '" << VD->getName()
170 << "') results in a null pointer dereference";
171 }
172 }
173 Ranges.push_back(IV->getSourceRange());
174 break;
175 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000176 default:
177 break;
178 }
Ted Kremenek2c791bd2009-11-06 00:44:32 +0000179
Anna Zakse172e8b2011-08-17 23:00:25 +0000180 BugReport *report =
181 new BugReport(*BT_null,
Ted Kremeneke576af22009-11-24 01:33:10 +0000182 buf.empty() ? BT_null->getDescription():buf.str(),
183 N);
184
Anna Zaks50bbc162011-08-19 22:33:38 +0000185 report->addVisitor(bugreporter::getTrackNullOrUndefValueVisitor(N,
Ted Kremenek76aadc32012-03-09 01:13:14 +0000186 bugreporter::GetDerefExpr(N), report));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000187
Chris Lattner5f9e2722011-07-23 10:55:15 +0000188 for (SmallVectorImpl<SourceRange>::iterator
Ted Kremenek452b84d2010-03-23 01:11:38 +0000189 I = Ranges.begin(), E = Ranges.end(); I!=E; ++I)
190 report->addRange(*I);
191
Ted Kremenek76aadc32012-03-09 01:13:14 +0000192 if (sourceR) {
193 report->markInteresting(sourceR);
194 report->markInteresting(state->getRawSVal(loc::MemRegionVal(sourceR)));
195 }
196
Ted Kremenek78d722f2009-11-21 01:50:48 +0000197 C.EmitReport(report);
198 return;
199 }
200 else {
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000201 // Otherwise, we have the case where the location could either be
202 // null or not-null. Record the error node as an "implicit" null
Ted Kremenek452b84d2010-03-23 01:11:38 +0000203 // dereference.
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000204 if (ExplodedNode *N = C.generateSink(nullState)) {
205 ImplicitNullDerefEvent event = { l, isLoad, N, &C.getBugReporter() };
206 dispatchEvent(event);
207 }
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000208 }
209 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000210
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000211 // From this point forward, we know that the location is not null.
Anna Zaks0bd6b112011-10-26 21:06:34 +0000212 C.addTransition(notNullState);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000213}
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000214
215void ento::registerDereferenceChecker(CheckerManager &mgr) {
216 mgr.registerChecker<DereferenceChecker>();
217}