diff options
author | Chih-Hung Hsieh <chh@google.com> | 2017-08-16 18:02:49 +0000 |
---|---|---|
committer | Chih-Hung Hsieh <chh@google.com> | 2017-08-16 18:02:49 +0000 |
commit | 41d29b15e89e864719a78ff6cfb994c72d9e569d (patch) | |
tree | 6ce4fee4d721acbd9b836ef2e351ea16aa062b33 /clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp | |
parent | 3be4ecb15b9e51ee959be424a7ce9e006859e4e3 (diff) | |
download | bcm5719-llvm-41d29b15e89e864719a78ff6cfb994c72d9e569d.tar.gz bcm5719-llvm-41d29b15e89e864719a78ff6cfb994c72d9e569d.zip |
[clang-tidy] Add a close-on-exec check on epoll_create() in Android module.
Summary:
epoll_create() is better to be replaced by epoll_create1() with EPOLL_CLOEXEC
flag to avoid file descriptor leakage.
Differential Revision: https://reviews.llvm.org/D35367
llvm-svn: 311029
Diffstat (limited to 'clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp')
-rw-r--r-- | clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp b/clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp index 800c27ac734..aec1007756b 100644 --- a/clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp +++ b/clang-tools-extra/clang-tidy/android/AndroidTidyModule.cpp @@ -14,6 +14,7 @@ #include "CloexecAcceptCheck.h" #include "CloexecCreatCheck.h" #include "CloexecEpollCreate1Check.h" +#include "CloexecEpollCreateCheck.h" #include "CloexecDupCheck.h" #include "CloexecFopenCheck.h" #include "CloexecInotifyInit1Check.h" @@ -37,6 +38,8 @@ public: CheckFactories.registerCheck<CloexecCreatCheck>("android-cloexec-creat"); CheckFactories.registerCheck<CloexecEpollCreate1Check>( "android-cloexec-epoll-create1"); + CheckFactories.registerCheck<CloexecEpollCreateCheck>( + "android-cloexec-epoll-create"); CheckFactories.registerCheck<CloexecDupCheck>("android-cloexec-dup"); CheckFactories.registerCheck<CloexecFopenCheck>("android-cloexec-fopen"); CheckFactories.registerCheck<CloexecInotifyInitCheck>( |