Skip to content

Commit

Permalink
Remove clang-tidy-vs from clang-tools-extra (PR41791)
Browse files Browse the repository at this point in the history
The clang-tidy-vs visual studio plugin in clang-tools-extra contains a
security vulnerability in the YamlDotNet package [1]. I posted to cfe-dev [2],
asking if there was anyone who was interested in updating the the plugin
to address the vulnerability. Reid mentioned that Zach (the original committer),
said that there's another plugin (Clang Power Tools) that provides clang-tidy support,
with additional extra features, so it would be ok to remove clang-tidy-vs.

This commit removes the plugin to address the security vulnerability, and adds
a section to the release notes that mentions that the plugin was removed, and
suggests to use Clang Power Tools.

Fixes PR 41791.

[1]: https://nvd.nist.gov/vuln/detail/CVE-2018-1000210
[2]: http://lists.llvm.org/pipermail/cfe-dev/2019-August/063196.html

Differential Revision: https://reviews.llvm.org/D66813

llvm-svn: 370096
  • Loading branch information
hyp committed Aug 27, 2019
1 parent 2f2feeb commit 559ae14
Show file tree
Hide file tree
Showing 39 changed files with 8 additions and 3,507 deletions.
1 change: 0 additions & 1 deletion clang-tools-extra/CMakeLists.txt
Expand Up @@ -4,7 +4,6 @@ add_subdirectory(clang-apply-replacements)
add_subdirectory(clang-reorder-fields)
add_subdirectory(modularize)
add_subdirectory(clang-tidy)
add_subdirectory(clang-tidy-vs)

add_subdirectory(clang-change-namespace)
add_subdirectory(clang-doc)
Expand Down
7 changes: 0 additions & 7 deletions clang-tools-extra/clang-tidy-vs/.gitignore

This file was deleted.

28 changes: 0 additions & 28 deletions clang-tools-extra/clang-tidy-vs/CMakeLists.txt

This file was deleted.

22 changes: 0 additions & 22 deletions clang-tools-extra/clang-tidy-vs/ClangTidy.sln

This file was deleted.

70 changes: 0 additions & 70 deletions clang-tools-extra/clang-tidy-vs/ClangTidy/CategoryVerb.cs

This file was deleted.

67 changes: 0 additions & 67 deletions clang-tools-extra/clang-tidy-vs/ClangTidy/CheckDatabase.cs

This file was deleted.

0 comments on commit 559ae14

Please sign in to comment.