2008-04-03 12:42:52 +08:00
|
|
|
// BugReporter.cpp - Generate PathDiagnostics for Bugs ------------*- 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 BugReporter, a utility class for generating
|
|
|
|
// PathDiagnostics for analyses based on GRSimpleVals.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
|
|
|
#include "clang/Analysis/PathSensitive/BugReporter.h"
|
2008-04-10 05:41:14 +08:00
|
|
|
#include "clang/Analysis/PathSensitive/GRExprEngine.h"
|
2008-04-03 12:42:52 +08:00
|
|
|
#include "clang/Basic/SourceManager.h"
|
|
|
|
#include "clang/Basic/SourceLocation.h"
|
|
|
|
#include "clang/AST/ASTContext.h"
|
|
|
|
#include "clang/AST/CFG.h"
|
|
|
|
#include "clang/AST/Expr.h"
|
|
|
|
#include "clang/Analysis/ProgramPoint.h"
|
|
|
|
#include "clang/Analysis/PathDiagnostic.h"
|
|
|
|
#include <sstream>
|
|
|
|
|
|
|
|
using namespace clang;
|
|
|
|
|
|
|
|
BugReporter::~BugReporter() {}
|
2008-04-10 05:41:14 +08:00
|
|
|
BugType::~BugType() {}
|
|
|
|
BugReport::~BugReport() {}
|
2008-04-12 02:40:29 +08:00
|
|
|
RangedBugReport::~RangedBugReport() {}
|
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
ExplodedGraph<ValueState>& BugReporter::getGraph() { return Eng.getGraph(); }
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
static inline Stmt* GetStmt(const ProgramPoint& P) {
|
|
|
|
if (const PostStmt* PS = dyn_cast<PostStmt>(&P)) {
|
|
|
|
return PS->getStmt();
|
|
|
|
}
|
|
|
|
else if (const BlockEdge* BE = dyn_cast<BlockEdge>(&P)) {
|
|
|
|
return BE->getSrc()->getTerminator();
|
|
|
|
}
|
|
|
|
else if (const BlockEntrance* BE = dyn_cast<BlockEntrance>(&P)) {
|
|
|
|
return BE->getFirstStmt();
|
|
|
|
}
|
|
|
|
|
|
|
|
assert (false && "Unsupported ProgramPoint.");
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2008-04-08 07:35:17 +08:00
|
|
|
static inline Stmt* GetStmt(const CFGBlock* B) {
|
|
|
|
assert (!B->empty());
|
|
|
|
return (*B)[0];
|
|
|
|
}
|
|
|
|
|
2008-04-15 01:39:48 +08:00
|
|
|
Stmt* BugReport::getStmt() const {
|
|
|
|
return N ? GetStmt(N->getLocation()) : NULL;
|
|
|
|
}
|
2008-04-18 07:44:37 +08:00
|
|
|
|
|
|
|
static inline ExplodedNode<ValueState>*
|
|
|
|
GetNextNode(ExplodedNode<ValueState>* N) {
|
|
|
|
return N->pred_empty() ? NULL : *(N->pred_begin());
|
|
|
|
}
|
2008-04-03 12:42:52 +08:00
|
|
|
|
2008-04-15 01:39:48 +08:00
|
|
|
|
2008-04-18 07:44:37 +08:00
|
|
|
static Stmt* GetLastStmt(ExplodedNode<ValueState>* N) {
|
|
|
|
assert (isa<BlockEntrance>(N->getLocation()));
|
|
|
|
|
|
|
|
for (N = GetNextNode(N); N; N = GetNextNode(N)) {
|
|
|
|
|
|
|
|
ProgramPoint P = N->getLocation();
|
|
|
|
|
|
|
|
if (PostStmt* PS = dyn_cast<PostStmt>(&P))
|
|
|
|
return PS->getStmt();
|
|
|
|
}
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
PathDiagnosticPiece*
|
|
|
|
BugReport::getEndPath(BugReporter& BR,
|
|
|
|
ExplodedNode<ValueState>* EndPathNode) const {
|
|
|
|
|
|
|
|
ProgramPoint ProgP = EndPathNode->getLocation();
|
|
|
|
Stmt *S = NULL;
|
|
|
|
|
|
|
|
if (BlockEntrance* BE = dyn_cast<BlockEntrance>(&ProgP))
|
|
|
|
if (BE->getBlock() == &BR.getCFG().getExit())
|
|
|
|
S = GetLastStmt(EndPathNode);
|
|
|
|
if (!S)
|
|
|
|
S = GetStmt(ProgP);
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
if (!S)
|
|
|
|
return NULL;
|
|
|
|
|
2008-04-18 07:44:37 +08:00
|
|
|
FullSourceLoc L(S->getLocStart(), BR.getContext().getSourceManager());
|
2008-04-10 05:41:14 +08:00
|
|
|
|
|
|
|
PathDiagnosticPiece* P =
|
|
|
|
new PathDiagnosticPiece(L, getDescription());
|
2008-04-03 12:42:52 +08:00
|
|
|
|
2008-04-04 02:00:37 +08:00
|
|
|
const SourceRange *Beg, *End;
|
|
|
|
getRanges(Beg, End);
|
|
|
|
|
|
|
|
if (Beg == End) {
|
|
|
|
if (Expr* E = dyn_cast<Expr>(S))
|
|
|
|
P->addRange(E->getSourceRange());
|
|
|
|
}
|
|
|
|
else {
|
|
|
|
assert (Beg < End);
|
|
|
|
for (; Beg != End; ++Beg)
|
|
|
|
P->addRange(*Beg);
|
|
|
|
}
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
return P;
|
|
|
|
}
|
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
void BugReport::getRanges(const SourceRange*& beg,
|
2008-04-15 01:39:48 +08:00
|
|
|
const SourceRange*& end) const {
|
2008-04-04 01:57:38 +08:00
|
|
|
beg = NULL;
|
|
|
|
end = NULL;
|
|
|
|
}
|
|
|
|
|
2008-04-15 01:39:48 +08:00
|
|
|
FullSourceLoc BugReport::getLocation(SourceManager& Mgr) {
|
|
|
|
|
|
|
|
if (!N)
|
|
|
|
return FullSourceLoc();
|
|
|
|
|
|
|
|
Stmt* S = GetStmt(N->getLocation());
|
|
|
|
|
|
|
|
if (!S)
|
|
|
|
return FullSourceLoc();
|
|
|
|
|
|
|
|
return FullSourceLoc(S->getLocStart(), Mgr);
|
|
|
|
}
|
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
PathDiagnosticPiece* BugReport::VisitNode(ExplodedNode<ValueState>* N,
|
|
|
|
ExplodedNode<ValueState>* PrevN,
|
|
|
|
ExplodedGraph<ValueState>& G,
|
2008-04-18 11:39:05 +08:00
|
|
|
BugReporter& BR) {
|
2008-04-10 05:41:14 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
void BugReporter::GeneratePathDiagnostic(PathDiagnostic& PD,
|
2008-04-15 01:39:48 +08:00
|
|
|
BugReport& R) {
|
2008-04-10 05:41:14 +08:00
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
ExplodedNode<ValueState>* N = R.getEndNode();
|
|
|
|
|
|
|
|
if (!N)
|
|
|
|
return;
|
2008-04-15 01:39:48 +08:00
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
llvm::OwningPtr<ExplodedGraph<ValueState> > GTrim(getGraph().Trim(&N, &N+1));
|
2008-04-18 07:44:37 +08:00
|
|
|
|
2008-04-03 12:59:14 +08:00
|
|
|
// Find the sink in the trimmed graph.
|
|
|
|
// FIXME: Should we eventually have a sink iterator?
|
|
|
|
|
|
|
|
ExplodedNode<ValueState>* NewN = 0;
|
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
for (ExplodedGraph<ValueState>::node_iterator
|
2008-04-18 07:44:37 +08:00
|
|
|
I = GTrim->nodes_begin(), E = GTrim->nodes_end(); I != E; ++I) {
|
2008-04-03 12:59:14 +08:00
|
|
|
|
|
|
|
if (I->isSink()) {
|
|
|
|
NewN = &*I;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
assert (NewN);
|
|
|
|
assert (NewN->getLocation() == N->getLocation());
|
|
|
|
|
|
|
|
N = NewN;
|
2008-04-18 07:44:37 +08:00
|
|
|
|
|
|
|
if (PathDiagnosticPiece* Piece = R.getEndPath(*this, N))
|
|
|
|
PD.push_back(Piece);
|
|
|
|
else
|
|
|
|
return;
|
2008-04-09 08:20:43 +08:00
|
|
|
|
|
|
|
ExplodedNode<ValueState>* NextNode = N->pred_empty()
|
|
|
|
? NULL : *(N->pred_begin());
|
|
|
|
|
2008-04-18 07:44:37 +08:00
|
|
|
SourceManager& SMgr = Ctx.getSourceManager();
|
|
|
|
|
2008-04-09 08:20:43 +08:00
|
|
|
while (NextNode) {
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
ExplodedNode<ValueState>* LastNode = N;
|
2008-04-09 08:20:43 +08:00
|
|
|
N = NextNode;
|
2008-04-18 07:44:37 +08:00
|
|
|
NextNode = GetNextNode(N);
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
ProgramPoint P = N->getLocation();
|
|
|
|
|
|
|
|
if (const BlockEdge* BE = dyn_cast<BlockEdge>(&P)) {
|
|
|
|
|
|
|
|
CFGBlock* Src = BE->getSrc();
|
|
|
|
CFGBlock* Dst = BE->getDst();
|
|
|
|
|
|
|
|
Stmt* T = Src->getTerminator();
|
|
|
|
|
|
|
|
if (!T)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
FullSourceLoc L(T->getLocStart(), SMgr);
|
|
|
|
|
|
|
|
switch (T->getStmtClass()) {
|
|
|
|
default:
|
|
|
|
break;
|
|
|
|
|
|
|
|
case Stmt::GotoStmtClass:
|
|
|
|
case Stmt::IndirectGotoStmtClass: {
|
|
|
|
|
|
|
|
Stmt* S = GetStmt(LastNode->getLocation());
|
|
|
|
|
|
|
|
if (!S)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
|
|
|
|
os << "Control jumps to line "
|
2008-04-08 07:35:17 +08:00
|
|
|
<< SMgr.getLogicalLineNumber(S->getLocStart()) << ".\n";
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, os.str()));
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
case Stmt::SwitchStmtClass: {
|
|
|
|
|
|
|
|
// Figure out what case arm we took.
|
|
|
|
|
|
|
|
Stmt* S = Dst->getLabel();
|
|
|
|
|
|
|
|
if (!S)
|
|
|
|
continue;
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
|
|
|
|
switch (S->getStmtClass()) {
|
|
|
|
default:
|
|
|
|
continue;
|
|
|
|
|
|
|
|
case Stmt::DefaultStmtClass: {
|
|
|
|
|
|
|
|
os << "Control jumps to the 'default' case at line "
|
2008-04-09 08:20:43 +08:00
|
|
|
<< SMgr.getLogicalLineNumber(S->getLocStart()) << ".\n";
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
|
|
|
case Stmt::CaseStmtClass: {
|
|
|
|
|
|
|
|
os << "Control jumps to 'case ";
|
|
|
|
|
|
|
|
Expr* CondE = cast<SwitchStmt>(T)->getCond();
|
|
|
|
unsigned bits = Ctx.getTypeSize(CondE->getType());
|
|
|
|
|
|
|
|
llvm::APSInt V1(bits, false);
|
|
|
|
|
|
|
|
CaseStmt* Case = cast<CaseStmt>(S);
|
|
|
|
|
|
|
|
if (!Case->getLHS()->isIntegerConstantExpr(V1, Ctx, 0, true)) {
|
|
|
|
assert (false &&
|
|
|
|
"Case condition must evaluate to an integer constant.");
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
os << V1.toString();
|
|
|
|
|
|
|
|
// Get the RHS of the case, if it exists.
|
|
|
|
|
|
|
|
if (Expr* E = Case->getRHS()) {
|
|
|
|
|
|
|
|
llvm::APSInt V2(bits, false);
|
|
|
|
|
|
|
|
if (!E->isIntegerConstantExpr(V2, Ctx, 0, true)) {
|
|
|
|
assert (false &&
|
|
|
|
"Case condition (RHS) must evaluate to an integer constant.");
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
os << " .. " << V2.toString();
|
|
|
|
}
|
|
|
|
|
|
|
|
os << ":' at line "
|
2008-04-08 07:35:17 +08:00
|
|
|
<< SMgr.getLogicalLineNumber(S->getLocStart()) << ".\n";
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
break;
|
|
|
|
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, os.str()));
|
|
|
|
break;
|
|
|
|
}
|
2008-04-08 07:35:17 +08:00
|
|
|
|
|
|
|
case Stmt::ConditionalOperatorClass: {
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
os << "'?' condition evaluates to ";
|
|
|
|
|
|
|
|
if (*(Src->succ_begin()+1) == Dst)
|
|
|
|
os << "false.";
|
|
|
|
else
|
|
|
|
os << "true.";
|
|
|
|
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, os.str()));
|
|
|
|
|
|
|
|
break;
|
|
|
|
}
|
2008-04-03 12:42:52 +08:00
|
|
|
|
2008-04-08 07:35:17 +08:00
|
|
|
case Stmt::DoStmtClass: {
|
|
|
|
|
|
|
|
if (*(Src->succ_begin()) == Dst) {
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
|
|
|
|
os << "Loop condition is true. Execution continues on line "
|
|
|
|
<< SMgr.getLogicalLineNumber(GetStmt(Dst)->getLocStart()) << '.';
|
|
|
|
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, os.str()));
|
|
|
|
}
|
|
|
|
else
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L,
|
|
|
|
"Loop condition is false. Exiting loop."));
|
|
|
|
|
|
|
|
break;
|
|
|
|
}
|
2008-04-03 12:42:52 +08:00
|
|
|
|
|
|
|
case Stmt::WhileStmtClass:
|
2008-04-08 07:35:17 +08:00
|
|
|
case Stmt::ForStmtClass: {
|
|
|
|
|
|
|
|
if (*(Src->succ_begin()+1) == Dst) {
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
|
|
|
|
os << "Loop condition is false. Execution continues on line "
|
|
|
|
<< SMgr.getLogicalLineNumber(GetStmt(Dst)->getLocStart()) << '.';
|
|
|
|
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, os.str()));
|
|
|
|
}
|
|
|
|
else
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L,
|
|
|
|
"Loop condition is true. Entering loop body."));
|
|
|
|
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
|
2008-04-03 12:42:52 +08:00
|
|
|
case Stmt::IfStmtClass: {
|
|
|
|
|
|
|
|
if (*(Src->succ_begin()+1) == Dst)
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, "Taking false branch."));
|
|
|
|
else
|
|
|
|
PD.push_front(new PathDiagnosticPiece(L, "Taking true branch."));
|
|
|
|
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
2008-04-09 08:20:43 +08:00
|
|
|
}
|
|
|
|
else
|
2008-04-18 11:39:05 +08:00
|
|
|
if (PathDiagnosticPiece* piece = R.VisitNode(N, NextNode, *GTrim, *this))
|
2008-04-10 05:41:14 +08:00
|
|
|
PD.push_front(piece);
|
2008-04-03 12:42:52 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2008-04-19 04:54:29 +08:00
|
|
|
bool BugTypeCacheLocation::isCached(BugReport& R) {
|
|
|
|
|
|
|
|
ExplodedNode<ValueState>* N = R.getEndNode();
|
2008-04-03 12:42:52 +08:00
|
|
|
|
2008-04-18 10:24:50 +08:00
|
|
|
if (!N)
|
|
|
|
return false;
|
2008-04-19 04:54:29 +08:00
|
|
|
|
|
|
|
// Cache the location of the error. Don't emit the same
|
2008-04-03 12:42:52 +08:00
|
|
|
// warning for the same error type that occurs at the same program
|
|
|
|
// location but along a different path.
|
|
|
|
|
|
|
|
void* p = N->getLocation().getRawData();
|
|
|
|
|
|
|
|
if (CachedErrors.count(p))
|
|
|
|
return true;
|
|
|
|
|
2008-04-19 04:54:29 +08:00
|
|
|
CachedErrors.insert(p);
|
2008-04-03 12:42:52 +08:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
void BugReporter::EmitWarning(BugReport& R) {
|
|
|
|
|
2008-04-19 04:54:29 +08:00
|
|
|
if (R.getBugType().isCached(R))
|
2008-04-03 12:42:52 +08:00
|
|
|
return;
|
2008-04-18 09:56:37 +08:00
|
|
|
|
2008-04-10 05:41:14 +08:00
|
|
|
PathDiagnostic D(R.getName());
|
2008-04-15 01:39:48 +08:00
|
|
|
GeneratePathDiagnostic(D, R);
|
2008-04-18 09:56:37 +08:00
|
|
|
|
|
|
|
// Emit a full diagnostic for the path if we have a PathDiagnosticClient.
|
2008-04-03 15:33:55 +08:00
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
if (PD && !D.empty()) {
|
2008-04-10 05:41:14 +08:00
|
|
|
PD->HandlePathDiagnostic(D);
|
2008-04-18 09:56:37 +08:00
|
|
|
return;
|
|
|
|
}
|
2008-04-15 01:39:48 +08:00
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
// We don't have a PathDiagnosticClient, but we can still emit a single
|
|
|
|
// line diagnostic. Determine the location.
|
2008-04-03 12:42:52 +08:00
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
FullSourceLoc L = D.empty() ? R.getLocation(Ctx.getSourceManager())
|
|
|
|
: D.back()->getLocation();
|
2008-04-15 02:06:42 +08:00
|
|
|
|
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
// Determine the range.
|
2008-04-15 02:06:42 +08:00
|
|
|
|
2008-04-18 09:56:37 +08:00
|
|
|
const SourceRange *Beg, *End;
|
|
|
|
|
|
|
|
if (!D.empty()) {
|
|
|
|
Beg = D.back()->ranges_begin();
|
|
|
|
End = D.back()->ranges_end();
|
2008-04-15 02:06:42 +08:00
|
|
|
}
|
2008-04-18 09:56:37 +08:00
|
|
|
else
|
|
|
|
R.getRanges(Beg, End);
|
|
|
|
|
|
|
|
// Compute the message.
|
|
|
|
|
|
|
|
std::ostringstream os;
|
|
|
|
os << "[CHECKER] ";
|
|
|
|
|
|
|
|
if (D.empty())
|
|
|
|
os << R.getDescription();
|
|
|
|
else
|
|
|
|
os << D.back()->getString();
|
|
|
|
|
|
|
|
unsigned ErrorDiag = Diag.getCustomDiagID(Diagnostic::Warning,
|
|
|
|
os.str().c_str());
|
2008-04-19 06:11:59 +08:00
|
|
|
|
|
|
|
if (PD)
|
|
|
|
Diag.Report(PD, L, ErrorDiag, NULL, 0, Beg, End - Beg);
|
|
|
|
else
|
|
|
|
Diag.Report(L, ErrorDiag, NULL, 0, Beg, End - Beg);
|
2008-04-03 12:42:52 +08:00
|
|
|
}
|