From ab162e18737e12f4dc4877eab8c756761c9d6b1e Mon Sep 17 00:00:00 2001 From: Zhongxing Xu Date: Tue, 3 Nov 2009 06:46:03 +0000 Subject: Pull UndefinedArgChecker into its own files. llvm-svn: 85875 --- clang/lib/Analysis/GRExprEngineInternalChecks.cpp | 39 ++--------------------- 1 file changed, 2 insertions(+), 37 deletions(-) (limited to 'clang/lib/Analysis/GRExprEngineInternalChecks.cpp') diff --git a/clang/lib/Analysis/GRExprEngineInternalChecks.cpp b/clang/lib/Analysis/GRExprEngineInternalChecks.cpp index 630e617a095..9bf7672e4c5 100644 --- a/clang/lib/Analysis/GRExprEngineInternalChecks.cpp +++ b/clang/lib/Analysis/GRExprEngineInternalChecks.cpp @@ -19,6 +19,7 @@ #include "clang/Analysis/PathSensitive/Checkers/UndefDerefChecker.h" #include "clang/Analysis/PathSensitive/Checkers/DivZeroChecker.h" #include "clang/Analysis/PathSensitive/Checkers/BadCallChecker.h" +#include "clang/Analysis/PathSensitive/Checkers/UndefinedArgChecker.h" #include "clang/Analysis/PathDiagnostic.h" #include "clang/Basic/SourceManager.h" #include "llvm/Support/Compiler.h" @@ -596,42 +597,6 @@ public: } }; -// Undefined arguments checking. - -class VISIBILITY_HIDDEN CheckUndefinedArg - : public CheckerVisitor { - - BadArg *BT; - -public: - CheckUndefinedArg() : BT(0) {} - ~CheckUndefinedArg() {} - - static void *getTag() { - static int x = 0; - return &x; - } - - void PreVisitCallExpr(CheckerContext &C, const CallExpr *CE); -}; - -void CheckUndefinedArg::PreVisitCallExpr(CheckerContext &C, const CallExpr *CE){ - for (CallExpr::const_arg_iterator I = CE->arg_begin(), E = CE->arg_end(); - I != E; ++I) { - if (C.getState()->getSVal(*I).isUndef()) { - if (ExplodedNode *ErrorNode = C.GenerateNode(CE, true)) { - if (!BT) - BT = new BadArg(); - // Generate a report for this bug. - ArgReport *Report = new ArgReport(*BT, BT->getDescription().c_str(), - ErrorNode, *I); - Report->addRange((*I)->getSourceRange()); - C.EmitReport(Report); - } - } - } -} - } // end clang namespace //===----------------------------------------------------------------------===// @@ -661,7 +626,7 @@ void GRExprEngine::RegisterInternalChecks() { // automatically. Note that the check itself is owned by the GRExprEngine // object. registerCheck(new CheckAttrNonNull()); - registerCheck(new CheckUndefinedArg()); + registerCheck(new UndefinedArgChecker()); registerCheck(new BadCallChecker()); registerCheck(new DivZeroChecker()); registerCheck(new UndefDerefChecker()); -- cgit v1.2.3