diff options
| -rw-r--r-- | libs/binder/Android.bp | 13 | ||||
| -rw-r--r-- | libs/binder/ndk/Android.bp | 15 | ||||
| -rw-r--r-- | libs/binder/ndk/include_cpp/android/binder_interface_utils.h | 3 |
3 files changed, 28 insertions, 3 deletions
diff --git a/libs/binder/Android.bp b/libs/binder/Android.bp index 1a819f88d2..e754d74655 100644 --- a/libs/binder/Android.bp +++ b/libs/binder/Android.bp @@ -181,15 +181,26 @@ cc_library { "--header-filter=^.*frameworks/native/libs/binder/.*.h$", ], tidy_checks_as_errors: [ - "*", + // Explicitly list the checks that should not occur in this module. + "abseil-*", + "android-*", + "bugprone-*", + "cert-*", + "clang-analyzer-*", "-clang-analyzer-core.CallAndMessage", "-clang-analyzer-core.uninitialized.Assign", "-clang-analyzer-unix.Malloc", "-clang-analyzer-deadcode.DeadStores", "-clang-analyzer-optin.cplusplus.UninitializedObject", + "google-*", + "-google-readability-*", + "-google-runtime-references", + "misc-*", "-misc-no-recursion", "-misc-redundant-expression", "-misc-unused-using-decls", + "performance*", + "portability*", ], } diff --git a/libs/binder/ndk/Android.bp b/libs/binder/ndk/Android.bp index d7b6fcef46..82f388284c 100644 --- a/libs/binder/ndk/Android.bp +++ b/libs/binder/ndk/Android.bp @@ -104,15 +104,28 @@ cc_library { "--header-filter=^.*frameworks/native/libs/binder/.*.h$", ], tidy_checks_as_errors: [ - "*", + // Explicitly list the checks that should not occur in this module. + "abseil-*", + "android-*", + "bugprone-*", + "cert-*", + "clang-analyzer-*", "-clang-analyzer-core.CallAndMessage", "-clang-analyzer-core.uninitialized.Assign", "-clang-analyzer-unix.Malloc", "-clang-analyzer-deadcode.DeadStores", "-clang-analyzer-optin.cplusplus.UninitializedObject", + "google-*", + "-google-readability-*", + "-google-runtime-references", + "misc-*", "-misc-no-recursion", + "-misc-non-private-member-variables-in-classes", "-misc-redundant-expression", + "-misc-unused-parameters", "-misc-unused-using-decls", + "performance*", + "portability*", ], } diff --git a/libs/binder/ndk/include_cpp/android/binder_interface_utils.h b/libs/binder/ndk/include_cpp/android/binder_interface_utils.h index a4f444192d..a1102e2833 100644 --- a/libs/binder/ndk/include_cpp/android/binder_interface_utils.h +++ b/libs/binder/ndk/include_cpp/android/binder_interface_utils.h @@ -83,7 +83,8 @@ class SharedRefBase { template <class T, class... Args> static std::shared_ptr<T> make(Args&&... args) { T* t = new T(std::forward<Args>(args)...); - return t->template ref<T>(); + // warning: Potential leak of memory pointed to by 't' [clang-analyzer-unix.Malloc] + return t->template ref<T>(); // NOLINT(clang-analyzer-unix.Malloc) } static void operator delete(void* p) { std::free(p); } |