blob: af74c79558d59dcf0813f1bfe2db06258c3fd7a5 [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//
10// This defines NullDerefChecker, a builtin check in GRExprEngine that performs
11// checks for null pointers at loads and stores.
12//
13//===----------------------------------------------------------------------===//
14
Benjamin Kramer5e2d2c22010-03-27 21:19:47 +000015#include "GRExprEngineInternalChecks.h"
16#include "clang/Checker/BugReporter/BugType.h"
Ted Kremenekf12d3192010-01-26 22:56:15 +000017#include "clang/Checker/Checkers/DereferenceChecker.h"
Ted Kremenek1309f9a2010-01-25 04:41:41 +000018#include "clang/Checker/PathSensitive/Checker.h"
19#include "clang/Checker/PathSensitive/GRExprEngine.h"
Ted Kremenekbc3a0212009-10-30 17:24:47 +000020
21using namespace clang;
22
Ted Kremenekb4b817d2009-11-11 03:26:34 +000023namespace {
Kovarththanan Rajaratnamba5fb5a2009-11-28 06:07:30 +000024class DereferenceChecker : public Checker {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000025 BuiltinBug *BT_null;
26 BuiltinBug *BT_undef;
27 llvm::SmallVector<ExplodedNode*, 2> ImplicitNullDerefNodes;
28public:
29 DereferenceChecker() : BT_null(0), BT_undef(0) {}
30 static void *getTag() { static int tag = 0; return &tag; }
31 void VisitLocation(CheckerContext &C, const Stmt *S, SVal location);
Ted Kremenek452b84d2010-03-23 01:11:38 +000032
Ted Kremenekb4b817d2009-11-11 03:26:34 +000033 std::pair<ExplodedNode * const*, ExplodedNode * const*>
Ted Kremenek452b84d2010-03-23 01:11:38 +000034 getImplicitNodes() const {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000035 return std::make_pair(ImplicitNullDerefNodes.data(),
36 ImplicitNullDerefNodes.data() +
37 ImplicitNullDerefNodes.size());
38 }
39};
40} // end anonymous namespace
41
42void clang::RegisterDereferenceChecker(GRExprEngine &Eng) {
43 Eng.registerCheck(new DereferenceChecker());
Ted Kremenekbc3a0212009-10-30 17:24:47 +000044}
45
Ted Kremenekb4b817d2009-11-11 03:26:34 +000046std::pair<ExplodedNode * const *, ExplodedNode * const *>
47clang::GetImplicitNullDereferences(GRExprEngine &Eng) {
48 DereferenceChecker *checker = Eng.getChecker<DereferenceChecker>();
49 if (!checker)
50 return std::make_pair((ExplodedNode * const *) 0,
51 (ExplodedNode * const *) 0);
52 return checker->getImplicitNodes();
53}
54
55void DereferenceChecker::VisitLocation(CheckerContext &C, const Stmt *S,
56 SVal l) {
57 // Check for dereference of an undefined value.
58 if (l.isUndef()) {
Ted Kremenek19d67b52009-11-23 22:22:01 +000059 if (ExplodedNode *N = C.GenerateSink()) {
Ted Kremenekb4b817d2009-11-11 03:26:34 +000060 if (!BT_undef)
61 BT_undef = new BuiltinBug("Dereference of undefined pointer value");
Ted Kremenek452b84d2010-03-23 01:11:38 +000062
Ted Kremenekb4b817d2009-11-11 03:26:34 +000063 EnhancedBugReport *report =
Benjamin Kramerd02e2322009-11-14 12:08:24 +000064 new EnhancedBugReport(*BT_undef, BT_undef->getDescription(), N);
Ted Kremenekb4b817d2009-11-11 03:26:34 +000065 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
66 bugreporter::GetDerefExpr(N));
67 C.EmitReport(report);
68 }
69 return;
70 }
Ted Kremenek452b84d2010-03-23 01:11:38 +000071
Ted Kremenekb4b817d2009-11-11 03:26:34 +000072 DefinedOrUnknownSVal location = cast<DefinedOrUnknownSVal>(l);
Ted Kremenek452b84d2010-03-23 01:11:38 +000073
74 // Check for null dereferences.
Ted Kremenekb4b817d2009-11-11 03:26:34 +000075 if (!isa<Loc>(location))
76 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +000077
Ted Kremenekb4b817d2009-11-11 03:26:34 +000078 const GRState *state = C.getState();
79 const GRState *notNullState, *nullState;
80 llvm::tie(notNullState, nullState) = state->Assume(location);
Ted Kremenek452b84d2010-03-23 01:11:38 +000081
Ted Kremenekb4b817d2009-11-11 03:26:34 +000082 // The explicit NULL case.
83 if (nullState) {
Ted Kremenek452b84d2010-03-23 01:11:38 +000084 if (!notNullState) {
Ted Kremenek78d722f2009-11-21 01:50:48 +000085 // Generate an error node.
Ted Kremenek19d67b52009-11-23 22:22:01 +000086 ExplodedNode *N = C.GenerateSink(nullState);
Ted Kremenek78d722f2009-11-21 01:50:48 +000087 if (!N)
Ted Kremenekb4b817d2009-11-11 03:26:34 +000088 return;
Ted Kremenek452b84d2010-03-23 01:11:38 +000089
Ted Kremenek78d722f2009-11-21 01:50:48 +000090 // We know that 'location' cannot be non-null. This is what
Ted Kremenek452b84d2010-03-23 01:11:38 +000091 // we call an "explicit" null dereference.
Ted Kremenek78d722f2009-11-21 01:50:48 +000092 if (!BT_null)
Ted Kremeneke576af22009-11-24 01:33:10 +000093 BT_null = new BuiltinBug("Dereference of null pointer");
Ted Kremenek452b84d2010-03-23 01:11:38 +000094
Ted Kremeneke576af22009-11-24 01:33:10 +000095 llvm::SmallString<100> buf;
Ted Kremenek452b84d2010-03-23 01:11:38 +000096 llvm::SmallVector<SourceRange, 2> Ranges;
Ted Kremeneke576af22009-11-24 01:33:10 +000097
98 switch (S->getStmtClass()) {
99 case Stmt::UnaryOperatorClass: {
100 const UnaryOperator *U = cast<UnaryOperator>(S);
101 const Expr *SU = U->getSubExpr()->IgnoreParens();
102 if (const DeclRefExpr *DR = dyn_cast<DeclRefExpr>(SU)) {
103 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
104 llvm::raw_svector_ostream os(buf);
Ted Kremenek452b84d2010-03-23 01:11:38 +0000105 os << "Dereference of null pointer (loaded from variable '"
106 << VD->getName() << "')";
107 Ranges.push_back(DR->getSourceRange());
Ted Kremeneke576af22009-11-24 01:33:10 +0000108 }
109 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000110 break;
111 }
112 case Stmt::MemberExprClass: {
113 const MemberExpr *M = cast<MemberExpr>(S);
114 if (M->isArrow())
115 if (DeclRefExpr *DR =
116 dyn_cast<DeclRefExpr>(M->getBase()->IgnoreParenCasts())) {
117 if (const VarDecl *VD = dyn_cast<VarDecl>(DR->getDecl())) {
118 llvm::raw_svector_ostream os(buf);
119 os << "Field access results in a dereference of a null pointer "
120 "(loaded from variable '" << VD->getName() << "')";
121 Ranges.push_back(M->getBase()->getSourceRange());
122 }
123 }
124 break;
Ted Kremeneke576af22009-11-24 01:33:10 +0000125 }
126 default:
127 break;
128 }
Ted Kremenek2c791bd2009-11-06 00:44:32 +0000129
Ted Kremenek78d722f2009-11-21 01:50:48 +0000130 EnhancedBugReport *report =
Ted Kremeneke576af22009-11-24 01:33:10 +0000131 new EnhancedBugReport(*BT_null,
132 buf.empty() ? BT_null->getDescription():buf.str(),
133 N);
134
Ted Kremenek78d722f2009-11-21 01:50:48 +0000135 report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue,
136 bugreporter::GetDerefExpr(N));
Ted Kremenek452b84d2010-03-23 01:11:38 +0000137
138 for (llvm::SmallVectorImpl<SourceRange>::iterator
139 I = Ranges.begin(), E = Ranges.end(); I!=E; ++I)
140 report->addRange(*I);
141
Ted Kremenek78d722f2009-11-21 01:50:48 +0000142 C.EmitReport(report);
143 return;
144 }
145 else {
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000146 // Otherwise, we have the case where the location could either be
147 // null or not-null. Record the error node as an "implicit" null
Ted Kremenek452b84d2010-03-23 01:11:38 +0000148 // dereference.
Ted Kremenek19d67b52009-11-23 22:22:01 +0000149 if (ExplodedNode *N = C.GenerateSink(nullState))
Ted Kremenek78d722f2009-11-21 01:50:48 +0000150 ImplicitNullDerefNodes.push_back(N);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000151 }
152 }
Ted Kremenek452b84d2010-03-23 01:11:38 +0000153
Ted Kremenekb4b817d2009-11-11 03:26:34 +0000154 // From this point forward, we know that the location is not null.
Ted Kremenek19d67b52009-11-23 22:22:01 +0000155 C.addTransition(notNullState);
Ted Kremenekbc3a0212009-10-30 17:24:47 +0000156}