diff options
author | Zhongxing Xu <xuzhongxing@gmail.com> | 2009-11-03 06:46:03 +0000 |
---|---|---|
committer | Zhongxing Xu <xuzhongxing@gmail.com> | 2009-11-03 06:46:03 +0000 |
commit | ab162e18737e12f4dc4877eab8c756761c9d6b1e (patch) | |
tree | e5d7962f2c4cac0720276a383265c36e49e567b0 /clang/lib/Analysis/GRExprEngineInternalChecks.cpp | |
parent | 064c5fef11305ec3c67e30d6df2a18b1f93a7059 (diff) | |
download | bcm5719-llvm-ab162e18737e12f4dc4877eab8c756761c9d6b1e.tar.gz bcm5719-llvm-ab162e18737e12f4dc4877eab8c756761c9d6b1e.zip |
Pull UndefinedArgChecker into its own files.
llvm-svn: 85875
Diffstat (limited to 'clang/lib/Analysis/GRExprEngineInternalChecks.cpp')
-rw-r--r-- | clang/lib/Analysis/GRExprEngineInternalChecks.cpp | 39 |
1 files changed, 2 insertions, 37 deletions
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<CheckUndefinedArg> { - - 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<CheckAttrNonNull>(new CheckAttrNonNull()); - registerCheck<CheckUndefinedArg>(new CheckUndefinedArg()); + registerCheck<UndefinedArgChecker>(new UndefinedArgChecker()); registerCheck<BadCallChecker>(new BadCallChecker()); registerCheck<DivZeroChecker>(new DivZeroChecker()); registerCheck<UndefDerefChecker>(new UndefDerefChecker()); |