| //== BasicObjCFoundationChecks.cpp - Simple Apple-Foundation checks -*- C++ -*-- |
| // |
| // The LLVM Compiler Infrastructure |
| // |
| // This file is distributed under the University of Illinois Open Source |
| // License. See LICENSE.TXT for details. |
| // |
| //===----------------------------------------------------------------------===// |
| // |
| // This file defines BasicObjCFoundationChecks, a class that encapsulates |
| // a set of simple checks to run on Objective-C code using Apple's Foundation |
| // classes. |
| // |
| //===----------------------------------------------------------------------===// |
| |
| #include "BasicObjCFoundationChecks.h" |
| |
| #include "clang/Analysis/PathSensitive/ExplodedGraph.h" |
| #include "clang/Analysis/PathSensitive/GRExprEngine.h" |
| #include "clang/Analysis/PathSensitive/GRSimpleAPICheck.h" |
| #include "clang/Analysis/PathSensitive/ValueState.h" |
| #include "clang/Analysis/PathSensitive/BugReporter.h" |
| #include "clang/Analysis/PathDiagnostic.h" |
| #include "clang/AST/Expr.h" |
| #include "clang/AST/ASTContext.h" |
| #include "llvm/Support/Compiler.h" |
| |
| #include <vector> |
| #include <sstream> |
| |
| using namespace clang; |
| |
| static ObjCInterfaceType* GetReceiverType(ObjCMessageExpr* ME) { |
| Expr* Receiver = ME->getReceiver(); |
| |
| if (!Receiver) |
| return NULL; |
| |
| // FIXME: Cleanup |
| QualType X = Receiver->getType(); |
| Type* TP = X.getTypePtr(); |
| assert (TP->isPointerType()); |
| |
| const PointerType* T = TP->getAsPointerType(); |
| |
| return dyn_cast<ObjCInterfaceType>(T->getPointeeType().getTypePtr()); |
| } |
| |
| static const char* GetReceiverNameType(ObjCMessageExpr* ME) { |
| ObjCInterfaceType* ReceiverType = GetReceiverType(ME); |
| return ReceiverType ? ReceiverType->getDecl()->getIdentifier()->getName() |
| : NULL; |
| } |
| |
| namespace { |
| |
| class VISIBILITY_HIDDEN NilArg : public BugType { |
| public: |
| virtual ~NilArg() {} |
| |
| virtual const char* getName() const { |
| return "nil argument"; |
| } |
| |
| class Report : public BugReport { |
| std::string Msg; |
| const char* s; |
| SourceRange R; |
| public: |
| |
| Report(NilArg& Desc, ObjCMessageExpr* ME, unsigned Arg) : BugReport(Desc) { |
| |
| Expr* E = ME->getArg(Arg); |
| R = E->getSourceRange(); |
| |
| std::ostringstream os; |
| |
| os << "Argument to '" << GetReceiverNameType(ME) << "' method '" |
| << ME->getSelector().getName() << "' cannot be nil."; |
| |
| Msg = os.str(); |
| s = Msg.c_str(); |
| } |
| |
| virtual ~Report() {} |
| |
| virtual const char* getDescription() const { return s; } |
| |
| virtual void getRanges(const SourceRange*& B, const SourceRange*& E) const { |
| B = &R; |
| E = B+1; |
| } |
| |
| virtual PathDiagnosticPiece* getEndPath(ASTContext& Ctx, |
| ExplodedNode<ValueState> *N) const { |
| |
| Stmt* S = cast<PostStmt>(N->getLocation()).getStmt(); |
| |
| if (!S) |
| return NULL; |
| |
| FullSourceLoc L(S->getLocStart(), Ctx.getSourceManager()); |
| PathDiagnosticPiece* P = new PathDiagnosticPiece(L, s); |
| |
| P->addRange(R); |
| |
| return P; |
| } |
| }; |
| }; |
| |
| |
| class VISIBILITY_HIDDEN BasicObjCFoundationChecks : public GRSimpleAPICheck { |
| NilArg Desc; |
| ASTContext &Ctx; |
| ValueStateManager* VMgr; |
| |
| typedef std::vector<std::pair<NodeTy*,BugReport*> > ErrorsTy; |
| ErrorsTy Errors; |
| |
| RVal GetRVal(ValueState* St, Expr* E) { return VMgr->GetRVal(St, E); } |
| |
| bool isNSString(ObjCInterfaceType* T, const char* suffix); |
| bool AuditNSString(NodeTy* N, ObjCMessageExpr* ME); |
| |
| void Warn(NodeTy* N, Expr* E, const std::string& s); |
| void WarnNilArg(NodeTy* N, Expr* E); |
| |
| bool CheckNilArg(NodeTy* N, unsigned Arg); |
| |
| public: |
| BasicObjCFoundationChecks(ASTContext& ctx, ValueStateManager* vmgr) |
| : Ctx(ctx), VMgr(vmgr) {} |
| |
| virtual ~BasicObjCFoundationChecks() { |
| for (ErrorsTy::iterator I = Errors.begin(), E = Errors.end(); I!=E; ++I) |
| delete I->second; |
| } |
| |
| virtual bool Audit(ExplodedNode<ValueState>* N); |
| |
| virtual void EmitWarnings(BugReporter& BR); |
| |
| private: |
| |
| void AddError(NodeTy* N, BugReport* R) { |
| Errors.push_back(std::make_pair(N, R)); |
| } |
| |
| void WarnNilArg(NodeTy* N, ObjCMessageExpr* ME, unsigned Arg) { |
| AddError(N, new NilArg::Report(Desc, ME, Arg)); |
| } |
| }; |
| |
| } // end anonymous namespace |
| |
| |
| GRSimpleAPICheck* |
| clang::CreateBasicObjCFoundationChecks(ASTContext& Ctx, |
| ValueStateManager* VMgr) { |
| |
| return new BasicObjCFoundationChecks(Ctx, VMgr); |
| } |
| |
| |
| |
| bool BasicObjCFoundationChecks::Audit(ExplodedNode<ValueState>* N) { |
| |
| ObjCMessageExpr* ME = |
| cast<ObjCMessageExpr>(cast<PostStmt>(N->getLocation()).getStmt()); |
| |
| ObjCInterfaceType* ReceiverType = GetReceiverType(ME); |
| |
| if (!ReceiverType) |
| return NULL; |
| |
| const char* name = ReceiverType->getDecl()->getIdentifier()->getName(); |
| |
| if (!name) |
| return false; |
| |
| if (name[0] != 'N' || name[1] != 'S') |
| return false; |
| |
| name += 2; |
| |
| // FIXME: Make all of this faster. |
| |
| if (isNSString(ReceiverType, name)) |
| return AuditNSString(N, ME); |
| |
| return false; |
| } |
| |
| static inline bool isNil(RVal X) { |
| return isa<lval::ConcreteInt>(X); |
| } |
| |
| //===----------------------------------------------------------------------===// |
| // Error reporting. |
| //===----------------------------------------------------------------------===// |
| |
| |
| void BasicObjCFoundationChecks::EmitWarnings(BugReporter& BR) { |
| |
| for (ErrorsTy::iterator I=Errors.begin(), E=Errors.end(); I!=E; ++I) |
| |
| BR.EmitPathWarning(*I->second, I->first); |
| } |
| |
| bool BasicObjCFoundationChecks::CheckNilArg(NodeTy* N, unsigned Arg) { |
| ObjCMessageExpr* ME = |
| cast<ObjCMessageExpr>(cast<PostStmt>(N->getLocation()).getStmt()); |
| |
| Expr * E = ME->getArg(Arg); |
| |
| if (isNil(GetRVal(N->getState(), E))) { |
| WarnNilArg(N, ME, Arg); |
| return true; |
| } |
| |
| return false; |
| } |
| |
| //===----------------------------------------------------------------------===// |
| // NSString checking. |
| //===----------------------------------------------------------------------===// |
| |
| bool BasicObjCFoundationChecks::isNSString(ObjCInterfaceType* T, |
| const char* suffix) { |
| |
| return !strcmp("String", suffix) || !strcmp("MutableString", suffix); |
| } |
| |
| bool BasicObjCFoundationChecks::AuditNSString(NodeTy* N, |
| ObjCMessageExpr* ME) { |
| |
| Selector S = ME->getSelector(); |
| |
| if (S.isUnarySelector()) |
| return false; |
| |
| // FIXME: This is going to be really slow doing these checks with |
| // lexical comparisons. |
| |
| std::string name = S.getName(); |
| assert (!name.empty()); |
| const char* cstr = &name[0]; |
| unsigned len = name.size(); |
| |
| switch (len) { |
| default: |
| break; |
| case 8: |
| if (!strcmp(cstr, "compare:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 15: |
| // FIXME: Checking for initWithFormat: will not work in most cases |
| // yet because [NSString alloc] returns id, not NSString*. We will |
| // need support for tracking expected-type information in the analyzer |
| // to find these errors. |
| if (!strcmp(cstr, "initWithFormat:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 16: |
| if (!strcmp(cstr, "compare:options:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 22: |
| if (!strcmp(cstr, "compare:options:range:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 23: |
| |
| if (!strcmp(cstr, "caseInsensitiveCompare:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 29: |
| if (!strcmp(cstr, "compare:options:range:locale:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| |
| case 37: |
| if (!strcmp(cstr, "componentsSeparatedByCharactersInSet:")) |
| return CheckNilArg(N, 0); |
| |
| break; |
| } |
| |
| return false; |
| } |