1 //=- AnalysisBasedWarnings.cpp - Sema warnings based on libAnalysis -*- C++ -*-=//
2 //
3 //                     The LLVM Compiler Infrastructure
4 //
5 // This file is distributed under the University of Illinois Open Source
6 // License. See LICENSE.TXT for details.
7 //
8 //===----------------------------------------------------------------------===//
9 //
10 // This file defines analysis_warnings::[Policy,Executor].
11 // Together they are used by Sema to issue warnings based on inexpensive
12 // static analysis algorithms in libAnalysis.
13 //
14 //===----------------------------------------------------------------------===//
15 
16 #include "clang/Sema/AnalysisBasedWarnings.h"
17 #include "clang/Sema/SemaInternal.h"
18 #include "clang/Sema/ScopeInfo.h"
19 #include "clang/Basic/SourceManager.h"
20 #include "clang/Basic/SourceLocation.h"
21 #include "clang/Lex/Preprocessor.h"
22 #include "clang/AST/DeclObjC.h"
23 #include "clang/AST/DeclCXX.h"
24 #include "clang/AST/ExprObjC.h"
25 #include "clang/AST/ExprCXX.h"
26 #include "clang/AST/StmtObjC.h"
27 #include "clang/AST/StmtCXX.h"
28 #include "clang/AST/EvaluatedExprVisitor.h"
29 #include "clang/AST/StmtVisitor.h"
30 #include "clang/Analysis/AnalysisContext.h"
31 #include "clang/Analysis/CFG.h"
32 #include "clang/Analysis/Analyses/ReachableCode.h"
33 #include "clang/Analysis/Analyses/CFGReachabilityAnalysis.h"
34 #include "clang/Analysis/Analyses/ThreadSafety.h"
35 #include "clang/Analysis/CFGStmtMap.h"
36 #include "clang/Analysis/Analyses/UninitializedValues.h"
37 #include "llvm/ADT/BitVector.h"
38 #include "llvm/ADT/FoldingSet.h"
39 #include "llvm/ADT/ImmutableMap.h"
40 #include "llvm/ADT/PostOrderIterator.h"
41 #include "llvm/ADT/SmallVector.h"
42 #include "llvm/ADT/StringRef.h"
43 #include "llvm/Support/Casting.h"
44 #include <algorithm>
45 #include <vector>
46 
47 using namespace clang;
48 
49 //===----------------------------------------------------------------------===//
50 // Unreachable code analysis.
51 //===----------------------------------------------------------------------===//
52 
53 namespace {
54   class UnreachableCodeHandler : public reachable_code::Callback {
55     Sema &S;
56   public:
57     UnreachableCodeHandler(Sema &s) : S(s) {}
58 
59     void HandleUnreachable(SourceLocation L, SourceRange R1, SourceRange R2) {
60       S.Diag(L, diag::warn_unreachable) << R1 << R2;
61     }
62   };
63 }
64 
65 /// CheckUnreachable - Check for unreachable code.
66 static void CheckUnreachable(Sema &S, AnalysisDeclContext &AC) {
67   UnreachableCodeHandler UC(S);
68   reachable_code::FindUnreachableCode(AC, UC);
69 }
70 
71 //===----------------------------------------------------------------------===//
72 // Check for missing return value.
73 //===----------------------------------------------------------------------===//
74 
75 enum ControlFlowKind {
76   UnknownFallThrough,
77   NeverFallThrough,
78   MaybeFallThrough,
79   AlwaysFallThrough,
80   NeverFallThroughOrReturn
81 };
82 
83 /// CheckFallThrough - Check that we don't fall off the end of a
84 /// Statement that should return a value.
85 ///
86 /// \returns AlwaysFallThrough iff we always fall off the end of the statement,
87 /// MaybeFallThrough iff we might or might not fall off the end,
88 /// NeverFallThroughOrReturn iff we never fall off the end of the statement or
89 /// return.  We assume NeverFallThrough iff we never fall off the end of the
90 /// statement but we may return.  We assume that functions not marked noreturn
91 /// will return.
92 static ControlFlowKind CheckFallThrough(AnalysisDeclContext &AC) {
93   CFG *cfg = AC.getCFG();
94   if (cfg == 0) return UnknownFallThrough;
95 
96   // The CFG leaves in dead things, and we don't want the dead code paths to
97   // confuse us, so we mark all live things first.
98   llvm::BitVector live(cfg->getNumBlockIDs());
99   unsigned count = reachable_code::ScanReachableFromBlock(&cfg->getEntry(),
100                                                           live);
101 
102   bool AddEHEdges = AC.getAddEHEdges();
103   if (!AddEHEdges && count != cfg->getNumBlockIDs())
104     // When there are things remaining dead, and we didn't add EH edges
105     // from CallExprs to the catch clauses, we have to go back and
106     // mark them as live.
107     for (CFG::iterator I = cfg->begin(), E = cfg->end(); I != E; ++I) {
108       CFGBlock &b = **I;
109       if (!live[b.getBlockID()]) {
110         if (b.pred_begin() == b.pred_end()) {
111           if (b.getTerminator() && isa<CXXTryStmt>(b.getTerminator()))
112             // When not adding EH edges from calls, catch clauses
113             // can otherwise seem dead.  Avoid noting them as dead.
114             count += reachable_code::ScanReachableFromBlock(&b, live);
115           continue;
116         }
117       }
118     }
119 
120   // Now we know what is live, we check the live precessors of the exit block
121   // and look for fall through paths, being careful to ignore normal returns,
122   // and exceptional paths.
123   bool HasLiveReturn = false;
124   bool HasFakeEdge = false;
125   bool HasPlainEdge = false;
126   bool HasAbnormalEdge = false;
127 
128   // Ignore default cases that aren't likely to be reachable because all
129   // enums in a switch(X) have explicit case statements.
130   CFGBlock::FilterOptions FO;
131   FO.IgnoreDefaultsWithCoveredEnums = 1;
132 
133   for (CFGBlock::filtered_pred_iterator
134 	 I = cfg->getExit().filtered_pred_start_end(FO); I.hasMore(); ++I) {
135     const CFGBlock& B = **I;
136     if (!live[B.getBlockID()])
137       continue;
138 
139     // Skip blocks which contain an element marked as no-return. They don't
140     // represent actually viable edges into the exit block, so mark them as
141     // abnormal.
142     if (B.hasNoReturnElement()) {
143       HasAbnormalEdge = true;
144       continue;
145     }
146 
147     // Destructors can appear after the 'return' in the CFG.  This is
148     // normal.  We need to look pass the destructors for the return
149     // statement (if it exists).
150     CFGBlock::const_reverse_iterator ri = B.rbegin(), re = B.rend();
151 
152     for ( ; ri != re ; ++ri)
153       if (isa<CFGStmt>(*ri))
154         break;
155 
156     // No more CFGElements in the block?
157     if (ri == re) {
158       if (B.getTerminator() && isa<CXXTryStmt>(B.getTerminator())) {
159         HasAbnormalEdge = true;
160         continue;
161       }
162       // A labeled empty statement, or the entry block...
163       HasPlainEdge = true;
164       continue;
165     }
166 
167     CFGStmt CS = cast<CFGStmt>(*ri);
168     const Stmt *S = CS.getStmt();
169     if (isa<ReturnStmt>(S)) {
170       HasLiveReturn = true;
171       continue;
172     }
173     if (isa<ObjCAtThrowStmt>(S)) {
174       HasFakeEdge = true;
175       continue;
176     }
177     if (isa<CXXThrowExpr>(S)) {
178       HasFakeEdge = true;
179       continue;
180     }
181     if (const AsmStmt *AS = dyn_cast<AsmStmt>(S)) {
182       if (AS->isMSAsm()) {
183         HasFakeEdge = true;
184         HasLiveReturn = true;
185         continue;
186       }
187     }
188     if (isa<CXXTryStmt>(S)) {
189       HasAbnormalEdge = true;
190       continue;
191     }
192     if (std::find(B.succ_begin(), B.succ_end(), &cfg->getExit())
193         == B.succ_end()) {
194       HasAbnormalEdge = true;
195       continue;
196     }
197 
198     HasPlainEdge = true;
199   }
200   if (!HasPlainEdge) {
201     if (HasLiveReturn)
202       return NeverFallThrough;
203     return NeverFallThroughOrReturn;
204   }
205   if (HasAbnormalEdge || HasFakeEdge || HasLiveReturn)
206     return MaybeFallThrough;
207   // This says AlwaysFallThrough for calls to functions that are not marked
208   // noreturn, that don't return.  If people would like this warning to be more
209   // accurate, such functions should be marked as noreturn.
210   return AlwaysFallThrough;
211 }
212 
213 namespace {
214 
215 struct CheckFallThroughDiagnostics {
216   unsigned diag_MaybeFallThrough_HasNoReturn;
217   unsigned diag_MaybeFallThrough_ReturnsNonVoid;
218   unsigned diag_AlwaysFallThrough_HasNoReturn;
219   unsigned diag_AlwaysFallThrough_ReturnsNonVoid;
220   unsigned diag_NeverFallThroughOrReturn;
221   bool funMode;
222   SourceLocation FuncLoc;
223 
224   static CheckFallThroughDiagnostics MakeForFunction(const Decl *Func) {
225     CheckFallThroughDiagnostics D;
226     D.FuncLoc = Func->getLocation();
227     D.diag_MaybeFallThrough_HasNoReturn =
228       diag::warn_falloff_noreturn_function;
229     D.diag_MaybeFallThrough_ReturnsNonVoid =
230       diag::warn_maybe_falloff_nonvoid_function;
231     D.diag_AlwaysFallThrough_HasNoReturn =
232       diag::warn_falloff_noreturn_function;
233     D.diag_AlwaysFallThrough_ReturnsNonVoid =
234       diag::warn_falloff_nonvoid_function;
235 
236     // Don't suggest that virtual functions be marked "noreturn", since they
237     // might be overridden by non-noreturn functions.
238     bool isVirtualMethod = false;
239     if (const CXXMethodDecl *Method = dyn_cast<CXXMethodDecl>(Func))
240       isVirtualMethod = Method->isVirtual();
241 
242     // Don't suggest that template instantiations be marked "noreturn"
243     bool isTemplateInstantiation = false;
244     if (const FunctionDecl *Function = dyn_cast<FunctionDecl>(Func))
245       isTemplateInstantiation = Function->isTemplateInstantiation();
246 
247     if (!isVirtualMethod && !isTemplateInstantiation)
248       D.diag_NeverFallThroughOrReturn =
249         diag::warn_suggest_noreturn_function;
250     else
251       D.diag_NeverFallThroughOrReturn = 0;
252 
253     D.funMode = true;
254     return D;
255   }
256 
257   static CheckFallThroughDiagnostics MakeForBlock() {
258     CheckFallThroughDiagnostics D;
259     D.diag_MaybeFallThrough_HasNoReturn =
260       diag::err_noreturn_block_has_return_expr;
261     D.diag_MaybeFallThrough_ReturnsNonVoid =
262       diag::err_maybe_falloff_nonvoid_block;
263     D.diag_AlwaysFallThrough_HasNoReturn =
264       diag::err_noreturn_block_has_return_expr;
265     D.diag_AlwaysFallThrough_ReturnsNonVoid =
266       diag::err_falloff_nonvoid_block;
267     D.diag_NeverFallThroughOrReturn =
268       diag::warn_suggest_noreturn_block;
269     D.funMode = false;
270     return D;
271   }
272 
273   bool checkDiagnostics(DiagnosticsEngine &D, bool ReturnsVoid,
274                         bool HasNoReturn) const {
275     if (funMode) {
276       return (ReturnsVoid ||
277               D.getDiagnosticLevel(diag::warn_maybe_falloff_nonvoid_function,
278                                    FuncLoc) == DiagnosticsEngine::Ignored)
279         && (!HasNoReturn ||
280             D.getDiagnosticLevel(diag::warn_noreturn_function_has_return_expr,
281                                  FuncLoc) == DiagnosticsEngine::Ignored)
282         && (!ReturnsVoid ||
283             D.getDiagnosticLevel(diag::warn_suggest_noreturn_block, FuncLoc)
284               == DiagnosticsEngine::Ignored);
285     }
286 
287     // For blocks.
288     return  ReturnsVoid && !HasNoReturn
289             && (!ReturnsVoid ||
290                 D.getDiagnosticLevel(diag::warn_suggest_noreturn_block, FuncLoc)
291                   == DiagnosticsEngine::Ignored);
292   }
293 };
294 
295 }
296 
297 /// CheckFallThroughForFunctionDef - Check that we don't fall off the end of a
298 /// function that should return a value.  Check that we don't fall off the end
299 /// of a noreturn function.  We assume that functions and blocks not marked
300 /// noreturn will return.
301 static void CheckFallThroughForBody(Sema &S, const Decl *D, const Stmt *Body,
302                                     const BlockExpr *blkExpr,
303                                     const CheckFallThroughDiagnostics& CD,
304                                     AnalysisDeclContext &AC) {
305 
306   bool ReturnsVoid = false;
307   bool HasNoReturn = false;
308 
309   if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) {
310     ReturnsVoid = FD->getResultType()->isVoidType();
311     HasNoReturn = FD->hasAttr<NoReturnAttr>() ||
312        FD->getType()->getAs<FunctionType>()->getNoReturnAttr();
313   }
314   else if (const ObjCMethodDecl *MD = dyn_cast<ObjCMethodDecl>(D)) {
315     ReturnsVoid = MD->getResultType()->isVoidType();
316     HasNoReturn = MD->hasAttr<NoReturnAttr>();
317   }
318   else if (isa<BlockDecl>(D)) {
319     QualType BlockTy = blkExpr->getType();
320     if (const FunctionType *FT =
321           BlockTy->getPointeeType()->getAs<FunctionType>()) {
322       if (FT->getResultType()->isVoidType())
323         ReturnsVoid = true;
324       if (FT->getNoReturnAttr())
325         HasNoReturn = true;
326     }
327   }
328 
329   DiagnosticsEngine &Diags = S.getDiagnostics();
330 
331   // Short circuit for compilation speed.
332   if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
333       return;
334 
335   // FIXME: Function try block
336   if (const CompoundStmt *Compound = dyn_cast<CompoundStmt>(Body)) {
337     switch (CheckFallThrough(AC)) {
338       case UnknownFallThrough:
339         break;
340 
341       case MaybeFallThrough:
342         if (HasNoReturn)
343           S.Diag(Compound->getRBracLoc(),
344                  CD.diag_MaybeFallThrough_HasNoReturn);
345         else if (!ReturnsVoid)
346           S.Diag(Compound->getRBracLoc(),
347                  CD.diag_MaybeFallThrough_ReturnsNonVoid);
348         break;
349       case AlwaysFallThrough:
350         if (HasNoReturn)
351           S.Diag(Compound->getRBracLoc(),
352                  CD.diag_AlwaysFallThrough_HasNoReturn);
353         else if (!ReturnsVoid)
354           S.Diag(Compound->getRBracLoc(),
355                  CD.diag_AlwaysFallThrough_ReturnsNonVoid);
356         break;
357       case NeverFallThroughOrReturn:
358         if (ReturnsVoid && !HasNoReturn && CD.diag_NeverFallThroughOrReturn) {
359           if (const FunctionDecl *FD = dyn_cast<FunctionDecl>(D)) {
360             S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn)
361               << 0 << FD;
362           } else if (const ObjCMethodDecl *MD = dyn_cast<ObjCMethodDecl>(D)) {
363             S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn)
364               << 1 << MD;
365           } else {
366             S.Diag(Compound->getLBracLoc(), CD.diag_NeverFallThroughOrReturn);
367           }
368         }
369         break;
370       case NeverFallThrough:
371         break;
372     }
373   }
374 }
375 
376 //===----------------------------------------------------------------------===//
377 // -Wuninitialized
378 //===----------------------------------------------------------------------===//
379 
380 namespace {
381 /// ContainsReference - A visitor class to search for references to
382 /// a particular declaration (the needle) within any evaluated component of an
383 /// expression (recursively).
384 class ContainsReference : public EvaluatedExprVisitor<ContainsReference> {
385   bool FoundReference;
386   const DeclRefExpr *Needle;
387 
388 public:
389   ContainsReference(ASTContext &Context, const DeclRefExpr *Needle)
390     : EvaluatedExprVisitor<ContainsReference>(Context),
391       FoundReference(false), Needle(Needle) {}
392 
393   void VisitExpr(Expr *E) {
394     // Stop evaluating if we already have a reference.
395     if (FoundReference)
396       return;
397 
398     EvaluatedExprVisitor<ContainsReference>::VisitExpr(E);
399   }
400 
401   void VisitDeclRefExpr(DeclRefExpr *E) {
402     if (E == Needle)
403       FoundReference = true;
404     else
405       EvaluatedExprVisitor<ContainsReference>::VisitDeclRefExpr(E);
406   }
407 
408   bool doesContainReference() const { return FoundReference; }
409 };
410 }
411 
412 static bool SuggestInitializationFixit(Sema &S, const VarDecl *VD) {
413   // Don't issue a fixit if there is already an initializer.
414   if (VD->getInit())
415     return false;
416 
417   // Suggest possible initialization (if any).
418   const char *initialization = 0;
419   QualType VariableTy = VD->getType().getCanonicalType();
420 
421   if (VariableTy->isObjCObjectPointerType() ||
422       VariableTy->isBlockPointerType()) {
423     // Check if 'nil' is defined.
424     if (S.PP.getMacroInfo(&S.getASTContext().Idents.get("nil")))
425       initialization = " = nil";
426     else
427       initialization = " = 0";
428   }
429   else if (VariableTy->isRealFloatingType())
430     initialization = " = 0.0";
431   else if (VariableTy->isBooleanType() && S.Context.getLangOptions().CPlusPlus)
432     initialization = " = false";
433   else if (VariableTy->isEnumeralType())
434     return false;
435   else if (VariableTy->isPointerType() || VariableTy->isMemberPointerType()) {
436     if (S.Context.getLangOptions().CPlusPlus0x)
437       initialization = " = nullptr";
438     // Check if 'NULL' is defined.
439     else if (S.PP.getMacroInfo(&S.getASTContext().Idents.get("NULL")))
440       initialization = " = NULL";
441     else
442       initialization = " = 0";
443   }
444   else if (VariableTy->isScalarType())
445     initialization = " = 0";
446 
447   if (initialization) {
448     SourceLocation loc = S.PP.getLocForEndOfToken(VD->getLocEnd());
449     S.Diag(loc, diag::note_var_fixit_add_initialization) << VD->getDeclName()
450       << FixItHint::CreateInsertion(loc, initialization);
451     return true;
452   }
453   return false;
454 }
455 
456 /// DiagnoseUninitializedUse -- Helper function for diagnosing uses of an
457 /// uninitialized variable. This manages the different forms of diagnostic
458 /// emitted for particular types of uses. Returns true if the use was diagnosed
459 /// as a warning. If a pariticular use is one we omit warnings for, returns
460 /// false.
461 static bool DiagnoseUninitializedUse(Sema &S, const VarDecl *VD,
462                                      const Expr *E, bool isAlwaysUninit,
463                                      bool alwaysReportSelfInit = false) {
464   bool isSelfInit = false;
465 
466   if (const DeclRefExpr *DRE = dyn_cast<DeclRefExpr>(E)) {
467     if (isAlwaysUninit) {
468       // Inspect the initializer of the variable declaration which is
469       // being referenced prior to its initialization. We emit
470       // specialized diagnostics for self-initialization, and we
471       // specifically avoid warning about self references which take the
472       // form of:
473       //
474       //   int x = x;
475       //
476       // This is used to indicate to GCC that 'x' is intentionally left
477       // uninitialized. Proven code paths which access 'x' in
478       // an uninitialized state after this will still warn.
479       //
480       // TODO: Should we suppress maybe-uninitialized warnings for
481       // variables initialized in this way?
482       if (const Expr *Initializer = VD->getInit()) {
483         if (!alwaysReportSelfInit && DRE == Initializer->IgnoreParenImpCasts())
484           return false;
485 
486         ContainsReference CR(S.Context, DRE);
487         CR.Visit(const_cast<Expr*>(Initializer));
488         isSelfInit = CR.doesContainReference();
489       }
490       if (isSelfInit) {
491         S.Diag(DRE->getLocStart(),
492                diag::warn_uninit_self_reference_in_init)
493         << VD->getDeclName() << VD->getLocation() << DRE->getSourceRange();
494       } else {
495         S.Diag(DRE->getLocStart(), diag::warn_uninit_var)
496           << VD->getDeclName() << DRE->getSourceRange();
497       }
498     } else {
499       S.Diag(DRE->getLocStart(), diag::warn_maybe_uninit_var)
500         << VD->getDeclName() << DRE->getSourceRange();
501     }
502   } else {
503     const BlockExpr *BE = cast<BlockExpr>(E);
504     S.Diag(BE->getLocStart(),
505            isAlwaysUninit ? diag::warn_uninit_var_captured_by_block
506                           : diag::warn_maybe_uninit_var_captured_by_block)
507       << VD->getDeclName();
508   }
509 
510   // Report where the variable was declared when the use wasn't within
511   // the initializer of that declaration & we didn't already suggest
512   // an initialization fixit.
513   if (!isSelfInit && !SuggestInitializationFixit(S, VD))
514     S.Diag(VD->getLocStart(), diag::note_uninit_var_def)
515       << VD->getDeclName();
516 
517   return true;
518 }
519 
520 typedef std::pair<const Expr*, bool> UninitUse;
521 
522 namespace {
523 struct SLocSort {
524   bool operator()(const UninitUse &a, const UninitUse &b) {
525     SourceLocation aLoc = a.first->getLocStart();
526     SourceLocation bLoc = b.first->getLocStart();
527     return aLoc.getRawEncoding() < bLoc.getRawEncoding();
528   }
529 };
530 
531 class UninitValsDiagReporter : public UninitVariablesHandler {
532   Sema &S;
533   typedef SmallVector<UninitUse, 2> UsesVec;
534   typedef llvm::DenseMap<const VarDecl *, std::pair<UsesVec*, bool> > UsesMap;
535   UsesMap *uses;
536 
537 public:
538   UninitValsDiagReporter(Sema &S) : S(S), uses(0) {}
539   ~UninitValsDiagReporter() {
540     flushDiagnostics();
541   }
542 
543   std::pair<UsesVec*, bool> &getUses(const VarDecl *vd) {
544     if (!uses)
545       uses = new UsesMap();
546 
547     UsesMap::mapped_type &V = (*uses)[vd];
548     UsesVec *&vec = V.first;
549     if (!vec)
550       vec = new UsesVec();
551 
552     return V;
553   }
554 
555   void handleUseOfUninitVariable(const Expr *ex, const VarDecl *vd,
556                                  bool isAlwaysUninit) {
557     getUses(vd).first->push_back(std::make_pair(ex, isAlwaysUninit));
558   }
559 
560   void handleSelfInit(const VarDecl *vd) {
561     getUses(vd).second = true;
562   }
563 
564   void flushDiagnostics() {
565     if (!uses)
566       return;
567 
568     for (UsesMap::iterator i = uses->begin(), e = uses->end(); i != e; ++i) {
569       const VarDecl *vd = i->first;
570       const UsesMap::mapped_type &V = i->second;
571 
572       UsesVec *vec = V.first;
573       bool hasSelfInit = V.second;
574 
575       // Specially handle the case where we have uses of an uninitialized
576       // variable, but the root cause is an idiomatic self-init.  We want
577       // to report the diagnostic at the self-init since that is the root cause.
578       if (!vec->empty() && hasSelfInit && hasAlwaysUninitializedUse(vec))
579         DiagnoseUninitializedUse(S, vd, vd->getInit()->IgnoreParenCasts(),
580                                  /* isAlwaysUninit */ true,
581                                  /* alwaysReportSelfInit */ true);
582       else {
583         // Sort the uses by their SourceLocations.  While not strictly
584         // guaranteed to produce them in line/column order, this will provide
585         // a stable ordering.
586         std::sort(vec->begin(), vec->end(), SLocSort());
587 
588         for (UsesVec::iterator vi = vec->begin(), ve = vec->end(); vi != ve;
589              ++vi) {
590           if (DiagnoseUninitializedUse(S, vd, vi->first,
591                                         /*isAlwaysUninit=*/vi->second))
592             // Skip further diagnostics for this variable. We try to warn only
593             // on the first point at which a variable is used uninitialized.
594             break;
595         }
596       }
597 
598       // Release the uses vector.
599       delete vec;
600     }
601     delete uses;
602   }
603 
604 private:
605   static bool hasAlwaysUninitializedUse(const UsesVec* vec) {
606   for (UsesVec::const_iterator i = vec->begin(), e = vec->end(); i != e; ++i) {
607     if (i->second) {
608       return true;
609     }
610   }
611   return false;
612 }
613 };
614 }
615 
616 
617 //===----------------------------------------------------------------------===//
618 // -Wthread-safety
619 //===----------------------------------------------------------------------===//
620 namespace clang {
621 namespace thread_safety {
622 typedef std::pair<SourceLocation, PartialDiagnostic> DelayedDiag;
623 typedef llvm::SmallVector<DelayedDiag, 4> DiagList;
624 
625 struct SortDiagBySourceLocation {
626   Sema &S;
627   SortDiagBySourceLocation(Sema &S) : S(S) {}
628 
629   bool operator()(const DelayedDiag &left, const DelayedDiag &right) {
630     // Although this call will be slow, this is only called when outputting
631     // multiple warnings.
632     return S.getSourceManager().isBeforeInTranslationUnit(left.first,
633                                                           right.first);
634   }
635 };
636 
637 class ThreadSafetyReporter : public clang::thread_safety::ThreadSafetyHandler {
638   Sema &S;
639   DiagList Warnings;
640   SourceLocation FunLocation;
641 
642   // Helper functions
643   void warnLockMismatch(unsigned DiagID, Name LockName, SourceLocation Loc) {
644     // Gracefully handle rare cases when the analysis can't get a more
645     // precise source location.
646     if (!Loc.isValid())
647       Loc = FunLocation;
648     PartialDiagnostic Warning = S.PDiag(DiagID) << LockName;
649     Warnings.push_back(DelayedDiag(Loc, Warning));
650   }
651 
652  public:
653   ThreadSafetyReporter(Sema &S, SourceLocation FL)
654     : S(S), FunLocation(FL) {}
655 
656   /// \brief Emit all buffered diagnostics in order of sourcelocation.
657   /// We need to output diagnostics produced while iterating through
658   /// the lockset in deterministic order, so this function orders diagnostics
659   /// and outputs them.
660   void emitDiagnostics() {
661     SortDiagBySourceLocation SortDiagBySL(S);
662     sort(Warnings.begin(), Warnings.end(), SortDiagBySL);
663     for (DiagList::iterator I = Warnings.begin(), E = Warnings.end();
664          I != E; ++I)
665       S.Diag(I->first, I->second);
666   }
667 
668   void handleInvalidLockExp(SourceLocation Loc) {
669     PartialDiagnostic Warning = S.PDiag(diag::warn_cannot_resolve_lock) << Loc;
670     Warnings.push_back(DelayedDiag(Loc, Warning));
671   }
672   void handleUnmatchedUnlock(Name LockName, SourceLocation Loc) {
673     warnLockMismatch(diag::warn_unlock_but_no_lock, LockName, Loc);
674   }
675 
676   void handleDoubleLock(Name LockName, SourceLocation Loc) {
677     warnLockMismatch(diag::warn_double_lock, LockName, Loc);
678   }
679 
680   void handleMutexHeldEndOfScope(Name LockName, SourceLocation Loc,
681                                  LockErrorKind LEK){
682     unsigned DiagID = 0;
683     switch (LEK) {
684       case LEK_LockedSomePredecessors:
685         DiagID = diag::warn_lock_at_end_of_scope;
686         break;
687       case LEK_LockedSomeLoopIterations:
688         DiagID = diag::warn_expecting_lock_held_on_loop;
689         break;
690       case LEK_LockedAtEndOfFunction:
691         DiagID = diag::warn_no_unlock;
692         break;
693     }
694     warnLockMismatch(DiagID, LockName, Loc);
695   }
696 
697 
698   void handleExclusiveAndShared(Name LockName, SourceLocation Loc1,
699                                 SourceLocation Loc2) {
700     PartialDiagnostic Warning =
701       S.PDiag(diag::warn_lock_exclusive_and_shared) << LockName;
702     PartialDiagnostic Note =
703       S.PDiag(diag::note_lock_exclusive_and_shared) << LockName;
704     Warnings.push_back(DelayedDiag(Loc1, Warning));
705     Warnings.push_back(DelayedDiag(Loc2, Note));
706   }
707 
708   void handleNoMutexHeld(const NamedDecl *D, ProtectedOperationKind POK,
709                          AccessKind AK, SourceLocation Loc) {
710     assert((POK == POK_VarAccess || POK == POK_VarDereference)
711              && "Only works for variables");
712     unsigned DiagID = POK == POK_VarAccess?
713                         diag::warn_variable_requires_any_lock:
714                         diag::warn_var_deref_requires_any_lock;
715     PartialDiagnostic Warning = S.PDiag(DiagID)
716       << D->getName() << getLockKindFromAccessKind(AK);
717     Warnings.push_back(DelayedDiag(Loc, Warning));
718   }
719 
720   void handleMutexNotHeld(const NamedDecl *D, ProtectedOperationKind POK,
721                           Name LockName, LockKind LK, SourceLocation Loc) {
722     unsigned DiagID = 0;
723     switch (POK) {
724       case POK_VarAccess:
725         DiagID = diag::warn_variable_requires_lock;
726         break;
727       case POK_VarDereference:
728         DiagID = diag::warn_var_deref_requires_lock;
729         break;
730       case POK_FunctionCall:
731         DiagID = diag::warn_fun_requires_lock;
732         break;
733     }
734     PartialDiagnostic Warning = S.PDiag(DiagID)
735       << D->getName() << LockName << LK;
736     Warnings.push_back(DelayedDiag(Loc, Warning));
737   }
738 
739   void handleFunExcludesLock(Name FunName, Name LockName, SourceLocation Loc) {
740     PartialDiagnostic Warning =
741       S.PDiag(diag::warn_fun_excludes_mutex) << FunName << LockName;
742     Warnings.push_back(DelayedDiag(Loc, Warning));
743   }
744 };
745 }
746 }
747 
748 //===----------------------------------------------------------------------===//
749 // AnalysisBasedWarnings - Worker object used by Sema to execute analysis-based
750 //  warnings on a function, method, or block.
751 //===----------------------------------------------------------------------===//
752 
753 clang::sema::AnalysisBasedWarnings::Policy::Policy() {
754   enableCheckFallThrough = 1;
755   enableCheckUnreachable = 0;
756   enableThreadSafetyAnalysis = 0;
757 }
758 
759 clang::sema::AnalysisBasedWarnings::AnalysisBasedWarnings(Sema &s)
760   : S(s),
761     NumFunctionsAnalyzed(0),
762     NumFunctionsWithBadCFGs(0),
763     NumCFGBlocks(0),
764     MaxCFGBlocksPerFunction(0),
765     NumUninitAnalysisFunctions(0),
766     NumUninitAnalysisVariables(0),
767     MaxUninitAnalysisVariablesPerFunction(0),
768     NumUninitAnalysisBlockVisits(0),
769     MaxUninitAnalysisBlockVisitsPerFunction(0) {
770   DiagnosticsEngine &D = S.getDiagnostics();
771   DefaultPolicy.enableCheckUnreachable = (unsigned)
772     (D.getDiagnosticLevel(diag::warn_unreachable, SourceLocation()) !=
773         DiagnosticsEngine::Ignored);
774   DefaultPolicy.enableThreadSafetyAnalysis = (unsigned)
775     (D.getDiagnosticLevel(diag::warn_double_lock, SourceLocation()) !=
776      DiagnosticsEngine::Ignored);
777 
778 }
779 
780 static void flushDiagnostics(Sema &S, sema::FunctionScopeInfo *fscope) {
781   for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
782        i = fscope->PossiblyUnreachableDiags.begin(),
783        e = fscope->PossiblyUnreachableDiags.end();
784        i != e; ++i) {
785     const sema::PossiblyUnreachableDiag &D = *i;
786     S.Diag(D.Loc, D.PD);
787   }
788 }
789 
790 void clang::sema::
791 AnalysisBasedWarnings::IssueWarnings(sema::AnalysisBasedWarnings::Policy P,
792                                      sema::FunctionScopeInfo *fscope,
793                                      const Decl *D, const BlockExpr *blkExpr) {
794 
795   // We avoid doing analysis-based warnings when there are errors for
796   // two reasons:
797   // (1) The CFGs often can't be constructed (if the body is invalid), so
798   //     don't bother trying.
799   // (2) The code already has problems; running the analysis just takes more
800   //     time.
801   DiagnosticsEngine &Diags = S.getDiagnostics();
802 
803   // Do not do any analysis for declarations in system headers if we are
804   // going to just ignore them.
805   if (Diags.getSuppressSystemWarnings() &&
806       S.SourceMgr.isInSystemHeader(D->getLocation()))
807     return;
808 
809   // For code in dependent contexts, we'll do this at instantiation time.
810   if (cast<DeclContext>(D)->isDependentContext())
811     return;
812 
813   if (Diags.hasErrorOccurred() || Diags.hasFatalErrorOccurred()) {
814     // Flush out any possibly unreachable diagnostics.
815     flushDiagnostics(S, fscope);
816     return;
817   }
818 
819   const Stmt *Body = D->getBody();
820   assert(Body);
821 
822   AnalysisDeclContext AC(/* AnalysisDeclContextManager */ 0,  D, 0);
823 
824   // Don't generate EH edges for CallExprs as we'd like to avoid the n^2
825   // explosion for destrutors that can result and the compile time hit.
826   AC.getCFGBuildOptions().PruneTriviallyFalseEdges = true;
827   AC.getCFGBuildOptions().AddEHEdges = false;
828   AC.getCFGBuildOptions().AddInitializers = true;
829   AC.getCFGBuildOptions().AddImplicitDtors = true;
830 
831   // Force that certain expressions appear as CFGElements in the CFG.  This
832   // is used to speed up various analyses.
833   // FIXME: This isn't the right factoring.  This is here for initial
834   // prototyping, but we need a way for analyses to say what expressions they
835   // expect to always be CFGElements and then fill in the BuildOptions
836   // appropriately.  This is essentially a layering violation.
837   if (P.enableCheckUnreachable) {
838     // Unreachable code analysis requires a linearized CFG.
839     AC.getCFGBuildOptions().setAllAlwaysAdd();
840   }
841   else {
842     AC.getCFGBuildOptions()
843       .setAlwaysAdd(Stmt::BinaryOperatorClass)
844       .setAlwaysAdd(Stmt::BlockExprClass)
845       .setAlwaysAdd(Stmt::CStyleCastExprClass)
846       .setAlwaysAdd(Stmt::DeclRefExprClass)
847       .setAlwaysAdd(Stmt::ImplicitCastExprClass)
848       .setAlwaysAdd(Stmt::UnaryOperatorClass);
849   }
850 
851   // Construct the analysis context with the specified CFG build options.
852 
853   // Emit delayed diagnostics.
854   if (!fscope->PossiblyUnreachableDiags.empty()) {
855     bool analyzed = false;
856 
857     // Register the expressions with the CFGBuilder.
858     for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
859          i = fscope->PossiblyUnreachableDiags.begin(),
860          e = fscope->PossiblyUnreachableDiags.end();
861          i != e; ++i) {
862       if (const Stmt *stmt = i->stmt)
863         AC.registerForcedBlockExpression(stmt);
864     }
865 
866     if (AC.getCFG()) {
867       analyzed = true;
868       for (SmallVectorImpl<sema::PossiblyUnreachableDiag>::iterator
869             i = fscope->PossiblyUnreachableDiags.begin(),
870             e = fscope->PossiblyUnreachableDiags.end();
871             i != e; ++i)
872       {
873         const sema::PossiblyUnreachableDiag &D = *i;
874         bool processed = false;
875         if (const Stmt *stmt = i->stmt) {
876           const CFGBlock *block = AC.getBlockForRegisteredExpression(stmt);
877           assert(block);
878           if (CFGReverseBlockReachabilityAnalysis *cra = AC.getCFGReachablityAnalysis()) {
879             // Can this block be reached from the entrance?
880             if (cra->isReachable(&AC.getCFG()->getEntry(), block))
881               S.Diag(D.Loc, D.PD);
882             processed = true;
883           }
884         }
885         if (!processed) {
886           // Emit the warning anyway if we cannot map to a basic block.
887           S.Diag(D.Loc, D.PD);
888         }
889       }
890     }
891 
892     if (!analyzed)
893       flushDiagnostics(S, fscope);
894   }
895 
896 
897   // Warning: check missing 'return'
898   if (P.enableCheckFallThrough) {
899     const CheckFallThroughDiagnostics &CD =
900       (isa<BlockDecl>(D) ? CheckFallThroughDiagnostics::MakeForBlock()
901                          : CheckFallThroughDiagnostics::MakeForFunction(D));
902     CheckFallThroughForBody(S, D, Body, blkExpr, CD, AC);
903   }
904 
905   // Warning: check for unreachable code
906   if (P.enableCheckUnreachable) {
907     // Only check for unreachable code on non-template instantiations.
908     // Different template instantiations can effectively change the control-flow
909     // and it is very difficult to prove that a snippet of code in a template
910     // is unreachable for all instantiations.
911     bool isTemplateInstantiation = false;
912     if (const FunctionDecl *Function = dyn_cast<FunctionDecl>(D))
913       isTemplateInstantiation = Function->isTemplateInstantiation();
914     if (!isTemplateInstantiation)
915       CheckUnreachable(S, AC);
916   }
917 
918   // Check for thread safety violations
919   if (P.enableThreadSafetyAnalysis) {
920     SourceLocation FL = AC.getDecl()->getLocation();
921     thread_safety::ThreadSafetyReporter Reporter(S, FL);
922     thread_safety::runThreadSafetyAnalysis(AC, Reporter);
923     Reporter.emitDiagnostics();
924   }
925 
926   if (Diags.getDiagnosticLevel(diag::warn_uninit_var, D->getLocStart())
927       != DiagnosticsEngine::Ignored ||
928       Diags.getDiagnosticLevel(diag::warn_maybe_uninit_var, D->getLocStart())
929       != DiagnosticsEngine::Ignored) {
930     if (CFG *cfg = AC.getCFG()) {
931       UninitValsDiagReporter reporter(S);
932       UninitVariablesAnalysisStats stats;
933       std::memset(&stats, 0, sizeof(UninitVariablesAnalysisStats));
934       runUninitializedVariablesAnalysis(*cast<DeclContext>(D), *cfg, AC,
935                                         reporter, stats);
936 
937       if (S.CollectStats && stats.NumVariablesAnalyzed > 0) {
938         ++NumUninitAnalysisFunctions;
939         NumUninitAnalysisVariables += stats.NumVariablesAnalyzed;
940         NumUninitAnalysisBlockVisits += stats.NumBlockVisits;
941         MaxUninitAnalysisVariablesPerFunction =
942             std::max(MaxUninitAnalysisVariablesPerFunction,
943                      stats.NumVariablesAnalyzed);
944         MaxUninitAnalysisBlockVisitsPerFunction =
945             std::max(MaxUninitAnalysisBlockVisitsPerFunction,
946                      stats.NumBlockVisits);
947       }
948     }
949   }
950 
951   // Collect statistics about the CFG if it was built.
952   if (S.CollectStats && AC.isCFGBuilt()) {
953     ++NumFunctionsAnalyzed;
954     if (CFG *cfg = AC.getCFG()) {
955       // If we successfully built a CFG for this context, record some more
956       // detail information about it.
957       NumCFGBlocks += cfg->getNumBlockIDs();
958       MaxCFGBlocksPerFunction = std::max(MaxCFGBlocksPerFunction,
959                                          cfg->getNumBlockIDs());
960     } else {
961       ++NumFunctionsWithBadCFGs;
962     }
963   }
964 }
965 
966 void clang::sema::AnalysisBasedWarnings::PrintStats() const {
967   llvm::errs() << "\n*** Analysis Based Warnings Stats:\n";
968 
969   unsigned NumCFGsBuilt = NumFunctionsAnalyzed - NumFunctionsWithBadCFGs;
970   unsigned AvgCFGBlocksPerFunction =
971       !NumCFGsBuilt ? 0 : NumCFGBlocks/NumCFGsBuilt;
972   llvm::errs() << NumFunctionsAnalyzed << " functions analyzed ("
973                << NumFunctionsWithBadCFGs << " w/o CFGs).\n"
974                << "  " << NumCFGBlocks << " CFG blocks built.\n"
975                << "  " << AvgCFGBlocksPerFunction
976                << " average CFG blocks per function.\n"
977                << "  " << MaxCFGBlocksPerFunction
978                << " max CFG blocks per function.\n";
979 
980   unsigned AvgUninitVariablesPerFunction = !NumUninitAnalysisFunctions ? 0
981       : NumUninitAnalysisVariables/NumUninitAnalysisFunctions;
982   unsigned AvgUninitBlockVisitsPerFunction = !NumUninitAnalysisFunctions ? 0
983       : NumUninitAnalysisBlockVisits/NumUninitAnalysisFunctions;
984   llvm::errs() << NumUninitAnalysisFunctions
985                << " functions analyzed for uninitialiazed variables\n"
986                << "  " << NumUninitAnalysisVariables << " variables analyzed.\n"
987                << "  " << AvgUninitVariablesPerFunction
988                << " average variables per function.\n"
989                << "  " << MaxUninitAnalysisVariablesPerFunction
990                << " max variables per function.\n"
991                << "  " << NumUninitAnalysisBlockVisits << " block visits.\n"
992                << "  " << AvgUninitBlockVisitsPerFunction
993                << " average block visits per function.\n"
994                << "  " << MaxUninitAnalysisBlockVisitsPerFunction
995                << " max block visits per function.\n";
996 }
997