blob: 3f3cacc8b557ee6c1c9c6a7025270f9257484f48 [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"
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000016#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000017#include "clang/StaticAnalyzer/Core/CheckerManager.h"
18#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Ted Kremenek9b663712011-02-10 01:03:03 +000019#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Ted Kremenekbc3a0212009-10-30 17:24:47 +000020
21using namespace clang;
Ted Kremenek9ef65372010-12-23 07:20:52 +000022using namespace ento;
Ted Kremenekbc3a0212009-10-30 17:24:47 +000023
Ted Kremenekb4b817d2009-11-11 03:26:34 +000024namespace {
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000025class DereferenceChecker
Argyrios Kyrtzidisec8605f2011-03-01 01:16:21 +000026 : public Checker< check::Location,
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000027 EventDispatcher<ImplicitNullDerefEvent> > {
28 mutable llvm::OwningPtr<BuiltinBug> BT_null;
29 mutable llvm::OwningPtr<BuiltinBug> BT_undef;
Ted Kremenek452b84d2010-03-23 01:11:38 +000030
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000031public:
32 void checkLocation(SVal location, bool isLoad, CheckerContext &C) const;
33
Chris Lattner5f9e2722011-07-23 10:55:15 +000034 static void AddDerefSource(raw_ostream &os,
35 SmallVectorImpl<SourceRange> &Ranges,
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000036 const Expr *Ex, bool loadedFrom = false);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000037};
38} // end anonymous namespace
39
Chris Lattner5f9e2722011-07-23 10:55:15 +000040void DereferenceChecker::AddDerefSource(raw_ostream &os,
41 SmallVectorImpl<SourceRange> &Ranges,
Ted Kremenek646c3c32010-10-26 00:06:13 +000042 const Expr *Ex,
43 bool loadedFrom) {
John McCallf6a16482010-12-04 03:47:34 +000044 Ex = Ex->IgnoreParenLValueCasts();
Ted Kremenek646c3c32010-10-26 00:06:13 +000045 switch (Ex->getStmtClass()) {
46 default:
47 return;
48 case Stmt::DeclRefExprClass: {
49 const DeclRefExpr *DR = cast<DeclRefExpr>(Ex);
50 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
51 os << " (" << (loadedFrom ? "loaded from" : "from")
52 << " variable '" << VD->getName() << "')";
53 Ranges.push_back(DR->getSourceRange());
54 }
55 return;
56 }
57 case Stmt::MemberExprClass: {
58 const MemberExpr *ME = cast<MemberExpr>(Ex);
59 os << " (" << (loadedFrom ? "loaded from" : "via")
60 << " field '" << ME->getMemberNameInfo() << "')";
61 SourceLocation L = ME->getMemberLoc();
62 Ranges.push_back(SourceRange(L, L));
63 break;
64 }
65 }
66}
67
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000068void DereferenceChecker::checkLocation(SVal l, bool isLoad,
69 CheckerContext &C) const {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000070 // Check for dereference of an undefined value.
71 if (l.isUndef()) {
Ted Kremenekd048c6e2010-12-20 21:19:09 +000072 if (ExplodedNode *N = C.generateSink()) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000073 if (!BT_undef)
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000074 BT_undef.reset(new BuiltinBug("Dereference of undefined pointer value"));
Ted Kremenek452b84d2010-03-23 01:11:38 +000075
Ted Kremenekb4b817d2009-11-11 03:26:34 +000076 EnhancedBugReport *report =
Benjamin Kramerd02e2322009-11-14 12:08:24 +000077 new EnhancedBugReport(*BT_undef, BT_undef->getDescription(), N);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000078 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
79 bugreporter::GetDerefExpr(N));
80 C.EmitReport(report);
81 }
82 return;
83 }
Ted Kremenek452b84d2010-03-23 01:11:38 +000084
Ted Kremenekb4b817d2009-11-11 03:26:34 +000085 DefinedOrUnknownSVal location = cast<DefinedOrUnknownSVal>(l);
Ted Kremenek452b84d2010-03-23 01:11:38 +000086
87 // Check for null dereferences.
Ted Kremenekb4b817d2009-11-11 03:26:34 +000088 if (!isa<Loc>(location))
89 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +000090
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +000091 const Stmt *S = C.getStmt();
Ted Kremenekb4b817d2009-11-11 03:26:34 +000092 const GRState *state = C.getState();
93 const GRState *notNullState, *nullState;
Ted Kremenek28f47b92010-12-01 22:16:56 +000094 llvm::tie(notNullState, nullState) = state->assume(location);
Ted Kremenek452b84d2010-03-23 01:11:38 +000095
Ted Kremenekb4b817d2009-11-11 03:26:34 +000096 // The explicit NULL case.
97 if (nullState) {
Ted Kremenek452b84d2010-03-23 01:11:38 +000098 if (!notNullState) {
Ted Kremenek78d722f2009-11-21 01:50:48 +000099 // Generate an error node.
Ted Kremenekd048c6e2010-12-20 21:19:09 +0000100 ExplodedNode *N = C.generateSink(nullState);
Ted Kremenek78d722f2009-11-21 01:50:48 +0000101 if (!N)
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000102 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000103
Ted Kremenek78d722f2009-11-21 01:50:48 +0000104 // We know that 'location' cannot be non-null. This is what
Ted Kremenek452b84d2010-03-23 01:11:38 +0000105 // we call an "explicit" null dereference.
Ted Kremenek78d722f2009-11-21 01:50:48 +0000106 if (!BT_null)
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000107 BT_null.reset(new BuiltinBug("Dereference of null pointer"));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000108
Ted Kremeneke576af22009-11-24 01:33:10 +0000109 llvm::SmallString<100> buf;
Chris Lattner5f9e2722011-07-23 10:55:15 +0000110 SmallVector<SourceRange, 2> Ranges;
Ted Kremenek892697d2010-12-16 07:46:53 +0000111
112 // Walk through lvalue casts to get the original expression
113 // that syntactically caused the load.
114 if (const Expr *expr = dyn_cast<Expr>(S))
115 S = expr->IgnoreParenLValueCasts();
Ted Kremeneke576af22009-11-24 01:33:10 +0000116
117 switch (S->getStmtClass()) {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000118 case Stmt::ArraySubscriptExprClass: {
119 llvm::raw_svector_ostream os(buf);
120 os << "Array access";
121 const ArraySubscriptExpr *AE = cast<ArraySubscriptExpr>(S);
122 AddDerefSource(os, Ranges, AE->getBase()->IgnoreParenCasts());
123 os << " results in a null pointer dereference";
124 break;
125 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000126 case Stmt::UnaryOperatorClass: {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000127 llvm::raw_svector_ostream os(buf);
128 os << "Dereference of null pointer";
Ted Kremeneke576af22009-11-24 01:33:10 +0000129 const UnaryOperator *U = cast<UnaryOperator>(S);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000130 AddDerefSource(os, Ranges, U->getSubExpr()->IgnoreParens(), true);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000131 break;
132 }
133 case Stmt::MemberExprClass: {
134 const MemberExpr *M = cast<MemberExpr>(S);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000135 if (M->isArrow()) {
136 llvm::raw_svector_ostream os(buf);
137 os << "Access to field '" << M->getMemberNameInfo()
138 << "' results in a dereference of a null pointer";
139 AddDerefSource(os, Ranges, M->getBase()->IgnoreParenCasts(), true);
140 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000141 break;
Ted Kremeneke576af22009-11-24 01:33:10 +0000142 }
Ted Kremenek0853c7f2010-09-10 20:20:49 +0000143 case Stmt::ObjCIvarRefExprClass: {
144 const ObjCIvarRefExpr *IV = cast<ObjCIvarRefExpr>(S);
145 if (const DeclRefExpr *DR =
146 dyn_cast<DeclRefExpr>(IV->getBase()->IgnoreParenCasts())) {
147 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
148 llvm::raw_svector_ostream os(buf);
149 os << "Instance variable access (via '" << VD->getName()
150 << "') results in a null pointer dereference";
151 }
152 }
153 Ranges.push_back(IV->getSourceRange());
154 break;
155 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000156 default:
157 break;
158 }
Ted Kremenek2c791bd2009-11-06 00:44:32 +0000159
Ted Kremenek78d722f2009-11-21 01:50:48 +0000160 EnhancedBugReport *report =
Ted Kremeneke576af22009-11-24 01:33:10 +0000161 new EnhancedBugReport(*BT_null,
162 buf.empty() ? BT_null->getDescription():buf.str(),
163 N);
164
Ted Kremenek78d722f2009-11-21 01:50:48 +0000165 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
166 bugreporter::GetDerefExpr(N));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000167
Chris Lattner5f9e2722011-07-23 10:55:15 +0000168 for (SmallVectorImpl<SourceRange>::iterator
Ted Kremenek452b84d2010-03-23 01:11:38 +0000169 I = Ranges.begin(), E = Ranges.end(); I!=E; ++I)
170 report->addRange(*I);
171
Ted Kremenek78d722f2009-11-21 01:50:48 +0000172 C.EmitReport(report);
173 return;
174 }
175 else {
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000176 // Otherwise, we have the case where the location could either be
177 // null or not-null. Record the error node as an "implicit" null
Ted Kremenek452b84d2010-03-23 01:11:38 +0000178 // dereference.
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000179 if (ExplodedNode *N = C.generateSink(nullState)) {
180 ImplicitNullDerefEvent event = { l, isLoad, N, &C.getBugReporter() };
181 dispatchEvent(event);
182 }
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000183 }
184 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000185
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000186 // From this point forward, we know that the location is not null.
Ted Kremenek19d67b52009-11-23 22:22:01 +0000187 C.addTransition(notNullState);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000188}
Argyrios Kyrtzidisb3d74da2011-02-28 17:36:18 +0000189
190void ento::registerDereferenceChecker(CheckerManager &mgr) {
191 mgr.registerChecker<DereferenceChecker>();
192}