annotate clang-tools-extra/clang-tidy/fuchsia/RestrictSystemIncludesCheck.cpp @ 160:dbfec6499728

...
author anatofuz
date Wed, 18 Mar 2020 19:11:03 +0900
parents 1d019706d866
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
150
anatofuz
parents:
diff changeset
1 //===--- RestrictSystemIncludesCheck.cpp - clang-tidy----------------------===//
anatofuz
parents:
diff changeset
2 //
anatofuz
parents:
diff changeset
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
anatofuz
parents:
diff changeset
4 // See https://llvm.org/LICENSE.txt for license information.
anatofuz
parents:
diff changeset
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
anatofuz
parents:
diff changeset
6 //
anatofuz
parents:
diff changeset
7 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
8
anatofuz
parents:
diff changeset
9 #include "RestrictSystemIncludesCheck.h"
anatofuz
parents:
diff changeset
10 #include "clang/Frontend/CompilerInstance.h"
anatofuz
parents:
diff changeset
11 #include "clang/Lex/HeaderSearch.h"
anatofuz
parents:
diff changeset
12 #include "clang/Lex/PPCallbacks.h"
anatofuz
parents:
diff changeset
13 #include "clang/Lex/Preprocessor.h"
anatofuz
parents:
diff changeset
14 #include "llvm/ADT/DenseMap.h"
anatofuz
parents:
diff changeset
15 #include "llvm/ADT/SmallVector.h"
anatofuz
parents:
diff changeset
16 #include "llvm/Support/Path.h"
anatofuz
parents:
diff changeset
17 #include <cstring>
anatofuz
parents:
diff changeset
18
anatofuz
parents:
diff changeset
19 namespace clang {
anatofuz
parents:
diff changeset
20 namespace tidy {
anatofuz
parents:
diff changeset
21 namespace fuchsia {
anatofuz
parents:
diff changeset
22
anatofuz
parents:
diff changeset
23 class RestrictedIncludesPPCallbacks : public PPCallbacks {
anatofuz
parents:
diff changeset
24 public:
anatofuz
parents:
diff changeset
25 explicit RestrictedIncludesPPCallbacks(RestrictSystemIncludesCheck &Check,
anatofuz
parents:
diff changeset
26 const SourceManager &SM)
anatofuz
parents:
diff changeset
27 : Check(Check), SM(SM) {}
anatofuz
parents:
diff changeset
28
anatofuz
parents:
diff changeset
29 void InclusionDirective(SourceLocation HashLoc, const Token &IncludeTok,
anatofuz
parents:
diff changeset
30 StringRef FileName, bool IsAngled,
anatofuz
parents:
diff changeset
31 CharSourceRange FilenameRange, const FileEntry *File,
anatofuz
parents:
diff changeset
32 StringRef SearchPath, StringRef RelativePath,
anatofuz
parents:
diff changeset
33 const Module *Imported,
anatofuz
parents:
diff changeset
34 SrcMgr::CharacteristicKind FileType) override;
anatofuz
parents:
diff changeset
35 void EndOfMainFile() override;
anatofuz
parents:
diff changeset
36
anatofuz
parents:
diff changeset
37 private:
anatofuz
parents:
diff changeset
38 struct IncludeDirective {
anatofuz
parents:
diff changeset
39 IncludeDirective() = default;
anatofuz
parents:
diff changeset
40 IncludeDirective(SourceLocation Loc, CharSourceRange Range,
anatofuz
parents:
diff changeset
41 StringRef Filename, StringRef FullPath, bool IsInMainFile)
anatofuz
parents:
diff changeset
42 : Loc(Loc), Range(Range), IncludeFile(Filename), IncludePath(FullPath),
anatofuz
parents:
diff changeset
43 IsInMainFile(IsInMainFile) {}
anatofuz
parents:
diff changeset
44
anatofuz
parents:
diff changeset
45 SourceLocation Loc; // '#' location in the include directive
anatofuz
parents:
diff changeset
46 CharSourceRange Range; // SourceRange for the file name
anatofuz
parents:
diff changeset
47 std::string IncludeFile; // Filename as a string
anatofuz
parents:
diff changeset
48 std::string IncludePath; // Full file path as a string
anatofuz
parents:
diff changeset
49 bool IsInMainFile; // Whether or not the include is in the main file
anatofuz
parents:
diff changeset
50 };
anatofuz
parents:
diff changeset
51
anatofuz
parents:
diff changeset
52 using FileIncludes = llvm::SmallVector<IncludeDirective, 8>;
anatofuz
parents:
diff changeset
53 llvm::SmallDenseMap<FileID, FileIncludes> IncludeDirectives;
anatofuz
parents:
diff changeset
54
anatofuz
parents:
diff changeset
55 RestrictSystemIncludesCheck &Check;
anatofuz
parents:
diff changeset
56 const SourceManager &SM;
anatofuz
parents:
diff changeset
57 };
anatofuz
parents:
diff changeset
58
anatofuz
parents:
diff changeset
59 void RestrictedIncludesPPCallbacks::InclusionDirective(
anatofuz
parents:
diff changeset
60 SourceLocation HashLoc, const Token &IncludeTok, StringRef FileName,
anatofuz
parents:
diff changeset
61 bool IsAngled, CharSourceRange FilenameRange, const FileEntry *File,
anatofuz
parents:
diff changeset
62 StringRef SearchPath, StringRef RelativePath, const Module *Imported,
anatofuz
parents:
diff changeset
63 SrcMgr::CharacteristicKind FileType) {
anatofuz
parents:
diff changeset
64 if (!Check.contains(FileName) && SrcMgr::isSystem(FileType)) {
anatofuz
parents:
diff changeset
65 SmallString<256> FullPath;
anatofuz
parents:
diff changeset
66 llvm::sys::path::append(FullPath, SearchPath);
anatofuz
parents:
diff changeset
67 llvm::sys::path::append(FullPath, RelativePath);
anatofuz
parents:
diff changeset
68 // Bucket the allowed include directives by the id of the file they were
anatofuz
parents:
diff changeset
69 // declared in.
anatofuz
parents:
diff changeset
70 IncludeDirectives[SM.getFileID(HashLoc)].emplace_back(
anatofuz
parents:
diff changeset
71 HashLoc, FilenameRange, FileName, FullPath.str(),
anatofuz
parents:
diff changeset
72 SM.isInMainFile(HashLoc));
anatofuz
parents:
diff changeset
73 }
anatofuz
parents:
diff changeset
74 }
anatofuz
parents:
diff changeset
75
anatofuz
parents:
diff changeset
76 void RestrictedIncludesPPCallbacks::EndOfMainFile() {
anatofuz
parents:
diff changeset
77 for (const auto &Bucket : IncludeDirectives) {
anatofuz
parents:
diff changeset
78 const FileIncludes &FileDirectives = Bucket.second;
anatofuz
parents:
diff changeset
79
anatofuz
parents:
diff changeset
80 // Emit fixits for all restricted includes.
anatofuz
parents:
diff changeset
81 for (const auto &Include : FileDirectives) {
anatofuz
parents:
diff changeset
82 // Fetch the length of the include statement from the start to just after
anatofuz
parents:
diff changeset
83 // the newline, for finding the end (including the newline).
anatofuz
parents:
diff changeset
84 unsigned ToLen = std::strcspn(SM.getCharacterData(Include.Loc), "\n") + 1;
anatofuz
parents:
diff changeset
85 CharSourceRange ToRange = CharSourceRange::getCharRange(
anatofuz
parents:
diff changeset
86 Include.Loc, Include.Loc.getLocWithOffset(ToLen));
anatofuz
parents:
diff changeset
87
anatofuz
parents:
diff changeset
88 if (!Include.IsInMainFile) {
anatofuz
parents:
diff changeset
89 auto D = Check.diag(
anatofuz
parents:
diff changeset
90 Include.Loc,
anatofuz
parents:
diff changeset
91 "system include %0 not allowed, transitively included from %1");
anatofuz
parents:
diff changeset
92 D << Include.IncludeFile << SM.getFilename(Include.Loc);
anatofuz
parents:
diff changeset
93 D << FixItHint::CreateRemoval(ToRange);
anatofuz
parents:
diff changeset
94 continue;
anatofuz
parents:
diff changeset
95 }
anatofuz
parents:
diff changeset
96 auto D = Check.diag(Include.Loc, "system include %0 not allowed");
anatofuz
parents:
diff changeset
97 D << Include.IncludeFile;
anatofuz
parents:
diff changeset
98 D << FixItHint::CreateRemoval(ToRange);
anatofuz
parents:
diff changeset
99 }
anatofuz
parents:
diff changeset
100 }
anatofuz
parents:
diff changeset
101 }
anatofuz
parents:
diff changeset
102
anatofuz
parents:
diff changeset
103 void RestrictSystemIncludesCheck::registerPPCallbacks(
anatofuz
parents:
diff changeset
104 const SourceManager &SM, Preprocessor *PP, Preprocessor *ModuleExpanderPP) {
anatofuz
parents:
diff changeset
105 PP->addPPCallbacks(
anatofuz
parents:
diff changeset
106 std::make_unique<RestrictedIncludesPPCallbacks>(*this, SM));
anatofuz
parents:
diff changeset
107 }
anatofuz
parents:
diff changeset
108
anatofuz
parents:
diff changeset
109 void RestrictSystemIncludesCheck::storeOptions(
anatofuz
parents:
diff changeset
110 ClangTidyOptions::OptionMap &Opts) {
anatofuz
parents:
diff changeset
111 Options.store(Opts, "Includes", AllowedIncludes);
anatofuz
parents:
diff changeset
112 }
anatofuz
parents:
diff changeset
113
anatofuz
parents:
diff changeset
114 } // namespace fuchsia
anatofuz
parents:
diff changeset
115 } // namespace tidy
anatofuz
parents:
diff changeset
116 } // namespace clang