123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869 |
- //== ReturnUndefChecker.cpp -------------------------------------*- 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 ReturnUndefChecker, which is a path-sensitive
- // check which looks for undefined or garbage values being returned to the
- // caller.
- //
- //===----------------------------------------------------------------------===//
- #include "InternalChecks.h"
- #include "clang/StaticAnalyzer/BugReporter/BugType.h"
- #include "clang/StaticAnalyzer/PathSensitive/CheckerVisitor.h"
- #include "clang/StaticAnalyzer/PathSensitive/ExprEngine.h"
- using namespace clang;
- using namespace ento;
- namespace {
- class ReturnUndefChecker :
- public CheckerVisitor<ReturnUndefChecker> {
- BuiltinBug *BT;
- public:
- ReturnUndefChecker() : BT(0) {}
- static void *getTag();
- void PreVisitReturnStmt(CheckerContext &C, const ReturnStmt *RS);
- };
- }
- void ento::RegisterReturnUndefChecker(ExprEngine &Eng) {
- Eng.registerCheck(new ReturnUndefChecker());
- }
- void *ReturnUndefChecker::getTag() {
- static int x = 0; return &x;
- }
- void ReturnUndefChecker::PreVisitReturnStmt(CheckerContext &C,
- const ReturnStmt *RS) {
-
- const Expr *RetE = RS->getRetValue();
- if (!RetE)
- return;
-
- if (!C.getState()->getSVal(RetE).isUndef())
- return;
-
- ExplodedNode *N = C.generateSink();
- if (!N)
- return;
-
- if (!BT)
- BT = new BuiltinBug("Garbage return value",
- "Undefined or garbage value returned to caller");
-
- EnhancedBugReport *report =
- new EnhancedBugReport(*BT, BT->getDescription(), N);
- report->addRange(RetE->getSourceRange());
- report->addVisitorCreator(bugreporter::registerTrackNullOrUndefValue, RetE);
- C.EmitReport(report);
- }
|