blob: 73f8087fd3c01de1358daf85d3401e1fba0ee315 [file] [log] [blame]
Anna Zaksef893392013-03-09 03:23:14 +00001//===--- NonNullParamChecker.cpp - Undefined arguments checker -*- C++ -*--===//
Zhongxing Xu9b9d7312009-11-03 07:35:33 +00002//
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//
Anna Zaksef893392013-03-09 03:23:14 +000010// This defines NonNullParamChecker, which checks for arguments expected not to
11// be null due to:
12// - the corresponding parameters being declared to have nonnull attribute
13// - the corresponding parameters being references; since the call would form
14// a reference to a null pointer
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000015//
16//===----------------------------------------------------------------------===//
17
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +000018#include "ClangSACheckers.h"
Benjamin Kramerea70eb32012-12-01 15:09:41 +000019#include "clang/AST/Attr.h"
Chandler Carruth3a022472012-12-04 09:13:33 +000020#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
Argyrios Kyrtzidis6a5674f2011-03-01 01:16:21 +000021#include "clang/StaticAnalyzer/Core/Checker.h"
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +000022#include "clang/StaticAnalyzer/Core/CheckerManager.h"
Jordan Rose4f7df9b2012-07-26 21:39:41 +000023#include "clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h"
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +000024#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000025
26using namespace clang;
Ted Kremenek98857c92010-12-23 07:20:52 +000027using namespace ento;
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000028
Ted Kremenek43253152009-11-11 05:50:44 +000029namespace {
Anna Zaksef893392013-03-09 03:23:14 +000030class NonNullParamChecker
Jordan Rose682b3162012-07-02 19:28:21 +000031 : public Checker< check::PreCall > {
Ahmed Charlesb8984322014-03-07 20:03:18 +000032 mutable std::unique_ptr<BugType> BTAttrNonNull;
33 mutable std::unique_ptr<BugType> BTNullRefArg;
34
Ted Kremenek43253152009-11-11 05:50:44 +000035public:
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +000036
Jordan Rose682b3162012-07-02 19:28:21 +000037 void checkPreCall(const CallEvent &Call, CheckerContext &C) const;
Anna Zaks9e0da9e02013-03-07 03:02:36 +000038
Aaron Ballman8d3a7a52015-06-23 13:15:32 +000039 std::unique_ptr<BugReport>
40 genReportNullAttrNonNull(const ExplodedNode *ErrorN, const Expr *ArgE) const;
41 std::unique_ptr<BugReport>
42 genReportReferenceToNullPointer(const ExplodedNode *ErrorN,
43 const Expr *ArgE) const;
Ted Kremenek43253152009-11-11 05:50:44 +000044};
45} // end anonymous namespace
46
Anna Zaksef893392013-03-09 03:23:14 +000047void NonNullParamChecker::checkPreCall(const CallEvent &Call,
Ted Kremenekf0ae7d02014-01-17 07:15:35 +000048 CheckerContext &C) const {
Jordan Rose682b3162012-07-02 19:28:21 +000049 const Decl *FD = Call.getDecl();
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000050 if (!FD)
51 return;
52
Jordan Rose679659f2014-10-13 19:38:02 +000053 // Merge all non-null attributes
54 unsigned NumArgs = Call.getNumArgs();
55 llvm::SmallBitVector AttrNonNull(NumArgs);
56 for (const auto *NonNull : FD->specific_attrs<NonNullAttr>()) {
57 if (!NonNull->args_size()) {
58 AttrNonNull.set(0, NumArgs);
59 break;
60 }
61 for (unsigned Val : NonNull->args()) {
62 if (Val >= NumArgs)
63 continue;
64 AttrNonNull.set(Val);
65 }
66 }
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000067
Jordan Rose682b3162012-07-02 19:28:21 +000068 ProgramStateRef state = C.getState();
69
Anna Zaks9e0da9e02013-03-07 03:02:36 +000070 CallEvent::param_type_iterator TyI = Call.param_type_begin(),
71 TyE = Call.param_type_end();
72
Jordan Rose679659f2014-10-13 19:38:02 +000073 for (unsigned idx = 0; idx < NumArgs; ++idx) {
Anna Zaks9e0da9e02013-03-07 03:02:36 +000074
75 // Check if the parameter is a reference. We want to report when reference
76 // to a null pointer is passed as a paramter.
77 bool haveRefTypeParam = false;
78 if (TyI != TyE) {
79 haveRefTypeParam = (*TyI)->isReferenceType();
80 TyI++;
81 }
82
Jordan Rose679659f2014-10-13 19:38:02 +000083 bool haveAttrNonNull = AttrNonNull[idx];
Ted Kremenekf0ae7d02014-01-17 07:15:35 +000084 if (!haveAttrNonNull) {
85 // Check if the parameter is also marked 'nonnull'.
86 ArrayRef<ParmVarDecl*> parms = Call.parameters();
87 if (idx < parms.size())
88 haveAttrNonNull = parms[idx]->hasAttr<NonNullAttr>();
89 }
Anna Zaks9e0da9e02013-03-07 03:02:36 +000090
91 if (!haveRefTypeParam && !haveAttrNonNull)
Zhongxing Xu9b9d7312009-11-03 07:35:33 +000092 continue;
93
Anna Zaks9e0da9e02013-03-07 03:02:36 +000094 // If the value is unknown or undefined, we can't perform this check.
95 const Expr *ArgE = Call.getArgExpr(idx);
Jordan Rose682b3162012-07-02 19:28:21 +000096 SVal V = Call.getArgSVal(idx);
David Blaikie05785d12013-02-20 22:23:23 +000097 Optional<DefinedSVal> DV = V.getAs<DefinedSVal>();
Jordy Rose3d858882010-06-21 20:08:28 +000098 if (!DV)
99 continue;
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000100
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000101 // Process the case when the argument is not a location.
102 assert(!haveRefTypeParam || DV->getAs<Loc>());
103
104 if (haveAttrNonNull && !DV->getAs<Loc>()) {
Ted Kremenekdcf85a82010-11-09 02:11:43 +0000105 // If the argument is a union type, we want to handle a potential
Jordan Rose682b3162012-07-02 19:28:21 +0000106 // transparent_union GCC extension.
Jordan Rose682b3162012-07-02 19:28:21 +0000107 if (!ArgE)
108 continue;
109
110 QualType T = ArgE->getType();
Ted Kremenekdcf85a82010-11-09 02:11:43 +0000111 const RecordType *UT = T->getAsUnionType();
112 if (!UT || !UT->getDecl()->hasAttr<TransparentUnionAttr>())
113 continue;
Jordan Rose682b3162012-07-02 19:28:21 +0000114
David Blaikie05785d12013-02-20 22:23:23 +0000115 if (Optional<nonloc::CompoundVal> CSV =
David Blaikie2fdacbc2013-02-20 05:52:05 +0000116 DV->getAs<nonloc::CompoundVal>()) {
Ted Kremenekdcf85a82010-11-09 02:11:43 +0000117 nonloc::CompoundVal::iterator CSV_I = CSV->begin();
118 assert(CSV_I != CSV->end());
119 V = *CSV_I;
David Blaikie2fdacbc2013-02-20 05:52:05 +0000120 DV = V.getAs<DefinedSVal>();
Ted Kremenekdcf85a82010-11-09 02:11:43 +0000121 assert(++CSV_I == CSV->end());
Jordan Rosee359d012014-02-26 01:20:19 +0000122 // FIXME: Handle (some_union){ some_other_union_val }, which turns into
123 // a LazyCompoundVal inside a CompoundVal.
124 if (!V.getAs<Loc>())
Anna Zaks23c85ed2013-03-06 20:26:02 +0000125 continue;
126 // Retrieve the corresponding expression.
127 if (const CompoundLiteralExpr *CE = dyn_cast<CompoundLiteralExpr>(ArgE))
128 if (const InitListExpr *IE =
129 dyn_cast<InitListExpr>(CE->getInitializer()))
130 ArgE = dyn_cast<Expr>(*(IE->begin()));
131
Jordan Rose682b3162012-07-02 19:28:21 +0000132 } else {
Ted Kremenekdcf85a82010-11-09 02:11:43 +0000133 // FIXME: Handle LazyCompoundVals?
134 continue;
135 }
136 }
137
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000138 ConstraintManager &CM = C.getConstraintManager();
Ted Kremenek49b1e382012-01-26 21:29:00 +0000139 ProgramStateRef stateNotNull, stateNull;
Benjamin Kramer867ea1d2014-03-02 13:01:17 +0000140 std::tie(stateNotNull, stateNull) = CM.assumeDual(state, *DV);
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000141
142 if (stateNull && !stateNotNull) {
143 // Generate an error node. Check for a null node in case
144 // we cache out.
Ted Kremenek750b7ac2010-12-20 21:19:09 +0000145 if (ExplodedNode *errorNode = C.generateSink(stateNull)) {
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000146
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000147 std::unique_ptr<BugReport> R;
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000148 if (haveAttrNonNull)
149 R = genReportNullAttrNonNull(errorNode, ArgE);
150 else if (haveRefTypeParam)
151 R = genReportReferenceToNullPointer(errorNode, ArgE);
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000152
153 // Highlight the range of the argument that was null.
Jordan Rose682b3162012-07-02 19:28:21 +0000154 R->addRange(Call.getArgSourceRange(idx));
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000155
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000156 // Emit the bug report.
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000157 C.emitReport(std::move(R));
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000158 }
159
160 // Always return. Either we cached out or we just emitted an error.
161 return;
162 }
163
164 // If a pointer value passed the check we should assume that it is
165 // indeed not null from this point forward.
166 assert(stateNotNull);
167 state = stateNotNull;
168 }
169
170 // If we reach here all of the arguments passed the nonnull check.
171 // If 'state' has been updated generated a new node.
Anna Zaksda4c8d62011-10-26 21:06:34 +0000172 C.addTransition(state);
Zhongxing Xu9b9d7312009-11-03 07:35:33 +0000173}
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +0000174
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000175std::unique_ptr<BugReport>
176NonNullParamChecker::genReportNullAttrNonNull(const ExplodedNode *ErrorNode,
177 const Expr *ArgE) const {
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000178 // Lazily allocate the BugType object if it hasn't already been
179 // created. Ownership is transferred to the BugReporter object once
180 // the BugReport is passed to 'EmitWarning'.
181 if (!BTAttrNonNull)
182 BTAttrNonNull.reset(new BugType(
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000183 this, "Argument with 'nonnull' attribute passed null", "API"));
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000184
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000185 auto R = llvm::make_unique<BugReport>(
186 *BTAttrNonNull,
187 "Null pointer passed as an argument to a 'nonnull' parameter", ErrorNode);
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000188 if (ArgE)
189 bugreporter::trackNullOrUndefValue(ErrorNode, ArgE, *R);
190
191 return R;
192}
193
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000194std::unique_ptr<BugReport> NonNullParamChecker::genReportReferenceToNullPointer(
195 const ExplodedNode *ErrorNode, const Expr *ArgE) const {
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000196 if (!BTNullRefArg)
Alexander Kornienko4aca9b12014-02-11 21:49:21 +0000197 BTNullRefArg.reset(new BuiltinBug(this, "Dereference of null pointer"));
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000198
Aaron Ballman8d3a7a52015-06-23 13:15:32 +0000199 auto R = llvm::make_unique<BugReport>(
200 *BTNullRefArg, "Forming reference to null pointer", ErrorNode);
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000201 if (ArgE) {
202 const Expr *ArgEDeref = bugreporter::getDerefExpr(ArgE);
Craig Topper0dbb7832014-05-27 02:45:47 +0000203 if (!ArgEDeref)
Anna Zaks9e0da9e02013-03-07 03:02:36 +0000204 ArgEDeref = ArgE;
205 bugreporter::trackNullOrUndefValue(ErrorNode,
206 ArgEDeref,
207 *R);
208 }
209 return R;
210
211}
212
Anna Zaksef893392013-03-09 03:23:14 +0000213void ento::registerNonNullParamChecker(CheckerManager &mgr) {
214 mgr.registerChecker<NonNullParamChecker>();
Argyrios Kyrtzidis6fff2e32011-02-28 01:28:01 +0000215}