blob: 3f6f9433298088f6a6cb6f40cdcabbb6b1806944 [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 Kyrtzidisd2592a32010-12-22 18:53:44 +000015#include "ExprEngineInternalChecks.h"
Ted Kremenek21142582010-12-23 19:38:26 +000016#include "clang/StaticAnalyzer/BugReporter/BugType.h"
17#include "clang/StaticAnalyzer/Checkers/DereferenceChecker.h"
18#include "clang/StaticAnalyzer/PathSensitive/Checker.h"
19#include "clang/StaticAnalyzer/PathSensitive/ExprEngine.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 {
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000025class DereferenceChecker : public Checker {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000026 BuiltinBug *BT_null;
27 BuiltinBug *BT_undef;
28 llvm::SmallVector<ExplodedNode*, 2> ImplicitNullDerefNodes;
29public:
30 DereferenceChecker() : BT_null(0), BT_undef(0) {}
31 static void *getTag() { static int tag = 0; return &tag; }
Argyrios Kyrtzidisf7fbbda2011-01-11 19:45:13 +000032 void visitLocation(CheckerContext &C, const Stmt *S, SVal location,
33 bool isLoad);
Ted Kremenek452b84d2010-03-23 01:11:38 +000034
Ted Kremenekb4b817d2009-11-11 03:26:34 +000035 std::pair<ExplodedNode * const*, ExplodedNode * const*>
Ted Kremenek452b84d2010-03-23 01:11:38 +000036 getImplicitNodes() const {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000037 return std::make_pair(ImplicitNullDerefNodes.data(),
38 ImplicitNullDerefNodes.data() +
39 ImplicitNullDerefNodes.size());
40 }
Ted Kremenek646c3c32010-10-26 00:06:13 +000041 void AddDerefSource(llvm::raw_ostream &os,
42 llvm::SmallVectorImpl<SourceRange> &Ranges,
43 const Expr *Ex, bool loadedFrom = false);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000044};
45} // end anonymous namespace
46
Ted Kremenek9ef65372010-12-23 07:20:52 +000047void ento::RegisterDereferenceChecker(ExprEngine &Eng) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000048 Eng.registerCheck(new DereferenceChecker());
Ted Kremenekbc3a0212009-10-30 17:24:47 +000049}
50
Ted Kremenekb4b817d2009-11-11 03:26:34 +000051std::pair<ExplodedNode * const *, ExplodedNode * const *>
Ted Kremenek9ef65372010-12-23 07:20:52 +000052ento::GetImplicitNullDereferences(ExprEngine &Eng) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000053 DereferenceChecker *checker = Eng.getChecker<DereferenceChecker>();
54 if (!checker)
55 return std::make_pair((ExplodedNode * const *) 0,
56 (ExplodedNode * const *) 0);
57 return checker->getImplicitNodes();
58}
59
Ted Kremenek646c3c32010-10-26 00:06:13 +000060void DereferenceChecker::AddDerefSource(llvm::raw_ostream &os,
61 llvm::SmallVectorImpl<SourceRange> &Ranges,
62 const Expr *Ex,
63 bool loadedFrom) {
John McCallf6a16482010-12-04 03:47:34 +000064 Ex = Ex->IgnoreParenLValueCasts();
Ted Kremenek646c3c32010-10-26 00:06:13 +000065 switch (Ex->getStmtClass()) {
66 default:
67 return;
68 case Stmt::DeclRefExprClass: {
69 const DeclRefExpr *DR = cast<DeclRefExpr>(Ex);
70 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
71 os << " (" << (loadedFrom ? "loaded from" : "from")
72 << " variable '" << VD->getName() << "')";
73 Ranges.push_back(DR->getSourceRange());
74 }
75 return;
76 }
77 case Stmt::MemberExprClass: {
78 const MemberExpr *ME = cast<MemberExpr>(Ex);
79 os << " (" << (loadedFrom ? "loaded from" : "via")
80 << " field '" << ME->getMemberNameInfo() << "')";
81 SourceLocation L = ME->getMemberLoc();
82 Ranges.push_back(SourceRange(L, L));
83 break;
84 }
85 }
86}
87
Ted Kremenek342e9072010-12-20 21:22:47 +000088void DereferenceChecker::visitLocation(CheckerContext &C, const Stmt *S,
Argyrios Kyrtzidisf7fbbda2011-01-11 19:45:13 +000089 SVal l, bool isLoad) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000090 // Check for dereference of an undefined value.
91 if (l.isUndef()) {
Ted Kremenekd048c6e2010-12-20 21:19:09 +000092 if (ExplodedNode *N = C.generateSink()) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000093 if (!BT_undef)
94 BT_undef = new BuiltinBug("Dereference of undefined pointer value");
Ted Kremenek452b84d2010-03-23 01:11:38 +000095
Ted Kremenekb4b817d2009-11-11 03:26:34 +000096 EnhancedBugReport *report =
Benjamin Kramerd02e2322009-11-14 12:08:24 +000097 new EnhancedBugReport(*BT_undef, BT_undef->getDescription(), N);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000098 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
99 bugreporter::GetDerefExpr(N));
100 C.EmitReport(report);
101 }
102 return;
103 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000104
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000105 DefinedOrUnknownSVal location = cast<DefinedOrUnknownSVal>(l);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000106
107 // Check for null dereferences.
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000108 if (!isa<Loc>(location))
109 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000110
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000111 const GRState *state = C.getState();
112 const GRState *notNullState, *nullState;
Ted Kremenek28f47b92010-12-01 22:16:56 +0000113 llvm::tie(notNullState, nullState) = state->assume(location);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000114
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000115 // The explicit NULL case.
116 if (nullState) {
Ted Kremenek452b84d2010-03-23 01:11:38 +0000117 if (!notNullState) {
Ted Kremenek78d722f2009-11-21 01:50:48 +0000118 // Generate an error node.
Ted Kremenekd048c6e2010-12-20 21:19:09 +0000119 ExplodedNode *N = C.generateSink(nullState);
Ted Kremenek78d722f2009-11-21 01:50:48 +0000120 if (!N)
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000121 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000122
Ted Kremenek78d722f2009-11-21 01:50:48 +0000123 // We know that 'location' cannot be non-null. This is what
Ted Kremenek452b84d2010-03-23 01:11:38 +0000124 // we call an "explicit" null dereference.
Ted Kremenek78d722f2009-11-21 01:50:48 +0000125 if (!BT_null)
Ted Kremeneke576af22009-11-24 01:33:10 +0000126 BT_null = new BuiltinBug("Dereference of null pointer");
Ted Kremenek452b84d2010-03-23 01:11:38 +0000127
Ted Kremeneke576af22009-11-24 01:33:10 +0000128 llvm::SmallString<100> buf;
Ted Kremenek452b84d2010-03-23 01:11:38 +0000129 llvm::SmallVector<SourceRange, 2> Ranges;
Ted Kremenek892697d2010-12-16 07:46:53 +0000130
131 // Walk through lvalue casts to get the original expression
132 // that syntactically caused the load.
133 if (const Expr *expr = dyn_cast<Expr>(S))
134 S = expr->IgnoreParenLValueCasts();
Ted Kremeneke576af22009-11-24 01:33:10 +0000135
136 switch (S->getStmtClass()) {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000137 case Stmt::ArraySubscriptExprClass: {
138 llvm::raw_svector_ostream os(buf);
139 os << "Array access";
140 const ArraySubscriptExpr *AE = cast<ArraySubscriptExpr>(S);
141 AddDerefSource(os, Ranges, AE->getBase()->IgnoreParenCasts());
142 os << " results in a null pointer dereference";
143 break;
144 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000145 case Stmt::UnaryOperatorClass: {
Ted Kremenek646c3c32010-10-26 00:06:13 +0000146 llvm::raw_svector_ostream os(buf);
147 os << "Dereference of null pointer";
Ted Kremeneke576af22009-11-24 01:33:10 +0000148 const UnaryOperator *U = cast<UnaryOperator>(S);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000149 AddDerefSource(os, Ranges, U->getSubExpr()->IgnoreParens(), true);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000150 break;
151 }
152 case Stmt::MemberExprClass: {
153 const MemberExpr *M = cast<MemberExpr>(S);
Ted Kremenek646c3c32010-10-26 00:06:13 +0000154 if (M->isArrow()) {
155 llvm::raw_svector_ostream os(buf);
156 os << "Access to field '" << M->getMemberNameInfo()
157 << "' results in a dereference of a null pointer";
158 AddDerefSource(os, Ranges, M->getBase()->IgnoreParenCasts(), true);
159 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000160 break;
Ted Kremeneke576af22009-11-24 01:33:10 +0000161 }
Ted Kremenek0853c7f2010-09-10 20:20:49 +0000162 case Stmt::ObjCIvarRefExprClass: {
163 const ObjCIvarRefExpr *IV = cast<ObjCIvarRefExpr>(S);
164 if (const DeclRefExpr *DR =
165 dyn_cast<DeclRefExpr>(IV->getBase()->IgnoreParenCasts())) {
166 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
167 llvm::raw_svector_ostream os(buf);
168 os << "Instance variable access (via '" << VD->getName()
169 << "') results in a null pointer dereference";
170 }
171 }
172 Ranges.push_back(IV->getSourceRange());
173 break;
174 }
Ted Kremeneke576af22009-11-24 01:33:10 +0000175 default:
176 break;
177 }
Ted Kremenek2c791bd2009-11-06 00:44:32 +0000178
Ted Kremenek78d722f2009-11-21 01:50:48 +0000179 EnhancedBugReport *report =
Ted Kremeneke576af22009-11-24 01:33:10 +0000180 new EnhancedBugReport(*BT_null,
181 buf.empty() ? BT_null->getDescription():buf.str(),
182 N);
183
Ted Kremenek78d722f2009-11-21 01:50:48 +0000184 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
185 bugreporter::GetDerefExpr(N));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000186
187 for (llvm::SmallVectorImpl<SourceRange>::iterator
188 I = Ranges.begin(), E = Ranges.end(); I!=E; ++I)
189 report->addRange(*I);
190
Ted Kremenek78d722f2009-11-21 01:50:48 +0000191 C.EmitReport(report);
192 return;
193 }
194 else {
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000195 // Otherwise, we have the case where the location could either be
196 // null or not-null. Record the error node as an "implicit" null
Ted Kremenek452b84d2010-03-23 01:11:38 +0000197 // dereference.
Ted Kremenekd048c6e2010-12-20 21:19:09 +0000198 if (ExplodedNode *N = C.generateSink(nullState))
Ted Kremenek78d722f2009-11-21 01:50:48 +0000199 ImplicitNullDerefNodes.push_back(N);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000200 }
201 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000202
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000203 // From this point forward, we know that the location is not null.
Ted Kremenek19d67b52009-11-23 22:22:01 +0000204 C.addTransition(notNullState);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000205}