blob: 0b4ecb41d20f3816e0051a08e2713da7360d9aa0 [file] [log] [blame]
George Karpenkov9a542f72017-10-13 00:51:41 +00001//==- NonnullGlobalConstantsChecker.cpp ---------------------------*- C++ -*--//
George Karpenkovead01622017-10-11 18:39:40 +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//
George Karpenkov9a542f72017-10-13 00:51:41 +000010// This checker adds an assumption that constant globals of certain types* are
George Karpenkovead01622017-10-11 18:39:40 +000011// non-null, as otherwise they generally do not convey any useful information.
George Karpenkov9a542f72017-10-13 00:51:41 +000012// The assumption is useful, as many framework use e. g. global const strings,
George Karpenkovead01622017-10-11 18:39:40 +000013// and the analyzer might not be able to infer the global value if the
14// definition is in a separate translation unit.
George Karpenkov9a542f72017-10-13 00:51:41 +000015// The following types (and their typedef aliases) are considered to be
16// non-null:
George Karpenkovead01622017-10-11 18:39:40 +000017// - `char* const`
18// - `const CFStringRef` from CoreFoundation
19// - `NSString* const` from Foundation
George Karpenkov9a542f72017-10-13 00:51:41 +000020// - `CFBooleanRef` from Foundation
George Karpenkovead01622017-10-11 18:39:40 +000021//
22//===----------------------------------------------------------------------===//
23
24#include "ClangSACheckers.h"
25#include "clang/StaticAnalyzer/Core/BugReporter/BugType.h"
26#include "clang/StaticAnalyzer/Core/Checker.h"
27#include "clang/StaticAnalyzer/Core/CheckerManager.h"
28#include "clang/StaticAnalyzer/Core/PathSensitive/CheckerContext.h"
29#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
30
31using namespace clang;
32using namespace ento;
33
34namespace {
35
George Karpenkov9a542f72017-10-13 00:51:41 +000036class NonnullGlobalConstantsChecker : public Checker<check::Location> {
George Karpenkovead01622017-10-11 18:39:40 +000037 mutable IdentifierInfo *NSStringII = nullptr;
38 mutable IdentifierInfo *CFStringRefII = nullptr;
George Karpenkov9a542f72017-10-13 00:51:41 +000039 mutable IdentifierInfo *CFBooleanRefII = nullptr;
George Karpenkovead01622017-10-11 18:39:40 +000040
41public:
George Karpenkov9a542f72017-10-13 00:51:41 +000042 NonnullGlobalConstantsChecker() {}
George Karpenkovead01622017-10-11 18:39:40 +000043
44 void checkLocation(SVal l, bool isLoad, const Stmt *S,
45 CheckerContext &C) const;
46
47private:
48 void initIdentifierInfo(ASTContext &Ctx) const;
49
50 bool isGlobalConstString(SVal V) const;
51
George Karpenkov9a542f72017-10-13 00:51:41 +000052 bool isNonnullType(QualType Ty) const;
George Karpenkovead01622017-10-11 18:39:40 +000053};
54
55} // namespace
56
57/// Lazily initialize cache for required identifier informations.
George Karpenkov9a542f72017-10-13 00:51:41 +000058void NonnullGlobalConstantsChecker::initIdentifierInfo(ASTContext &Ctx) const {
George Karpenkovead01622017-10-11 18:39:40 +000059 if (NSStringII)
60 return;
61
62 NSStringII = &Ctx.Idents.get("NSString");
63 CFStringRefII = &Ctx.Idents.get("CFStringRef");
George Karpenkov9a542f72017-10-13 00:51:41 +000064 CFBooleanRefII = &Ctx.Idents.get("CFBooleanRef");
George Karpenkovead01622017-10-11 18:39:40 +000065}
66
67/// Add an assumption that const string-like globals are non-null.
George Karpenkov9a542f72017-10-13 00:51:41 +000068void NonnullGlobalConstantsChecker::checkLocation(SVal location, bool isLoad,
George Karpenkovead01622017-10-11 18:39:40 +000069 const Stmt *S,
70 CheckerContext &C) const {
71 initIdentifierInfo(C.getASTContext());
72 if (!isLoad || !location.isValid())
73 return;
74
75 ProgramStateRef State = C.getState();
76 SVal V = State->getSVal(location.castAs<Loc>());
77
78 if (isGlobalConstString(location)) {
79 Optional<DefinedOrUnknownSVal> Constr = V.getAs<DefinedOrUnknownSVal>();
80
81 if (Constr) {
82
83 // Assume that the variable is non-null.
84 ProgramStateRef OutputState = State->assume(*Constr, true);
85 C.addTransition(OutputState);
86 }
87 }
88}
89
90/// \param V loaded lvalue.
91/// \return whether {@code val} is a string-like const global.
George Karpenkov9a542f72017-10-13 00:51:41 +000092bool NonnullGlobalConstantsChecker::isGlobalConstString(SVal V) const {
George Karpenkovead01622017-10-11 18:39:40 +000093 Optional<loc::MemRegionVal> RegionVal = V.getAs<loc::MemRegionVal>();
94 if (!RegionVal)
95 return false;
96 auto *Region = dyn_cast<VarRegion>(RegionVal->getAsRegion());
97 if (!Region)
98 return false;
99 const VarDecl *Decl = Region->getDecl();
100
101 if (!Decl->hasGlobalStorage())
102 return false;
103
104 QualType Ty = Decl->getType();
105 bool HasConst = Ty.isConstQualified();
George Karpenkov9a542f72017-10-13 00:51:41 +0000106 if (isNonnullType(Ty) && HasConst)
George Karpenkovead01622017-10-11 18:39:40 +0000107 return true;
108
109 // Look through the typedefs.
110 while (auto *T = dyn_cast<TypedefType>(Ty)) {
111 Ty = T->getDecl()->getUnderlyingType();
112
113 // It is sufficient for any intermediate typedef
114 // to be classified const.
115 HasConst = HasConst || Ty.isConstQualified();
George Karpenkov9a542f72017-10-13 00:51:41 +0000116 if (isNonnullType(Ty) && HasConst)
George Karpenkovead01622017-10-11 18:39:40 +0000117 return true;
118 }
119 return false;
120}
121
George Karpenkov9a542f72017-10-13 00:51:41 +0000122/// \return whether {@code type} is extremely unlikely to be null
123bool NonnullGlobalConstantsChecker::isNonnullType(QualType Ty) const {
George Karpenkovead01622017-10-11 18:39:40 +0000124
125 if (Ty->isPointerType() && Ty->getPointeeType()->isCharType())
126 return true;
127
128 if (auto *T = dyn_cast<ObjCObjectPointerType>(Ty)) {
George Karpenkov734cad82017-10-11 19:13:15 +0000129 return T->getInterfaceDecl() &&
130 T->getInterfaceDecl()->getIdentifier() == NSStringII;
George Karpenkovead01622017-10-11 18:39:40 +0000131 } else if (auto *T = dyn_cast<TypedefType>(Ty)) {
George Karpenkov9a542f72017-10-13 00:51:41 +0000132 IdentifierInfo* II = T->getDecl()->getIdentifier();
133 return II == CFStringRefII || II == CFBooleanRefII;
George Karpenkovead01622017-10-11 18:39:40 +0000134 }
135 return false;
136}
137
George Karpenkov9a542f72017-10-13 00:51:41 +0000138void ento::registerNonnullGlobalConstantsChecker(CheckerManager &Mgr) {
139 Mgr.registerChecker<NonnullGlobalConstantsChecker>();
George Karpenkovead01622017-10-11 18:39:40 +0000140}