From 0cd8012b73d45e03509a25494441704a401c7021 Mon Sep 17 00:00:00 2001 From: Yurii Zubrytskyi Date: Thu, 9 Apr 2020 23:08:31 -0700 Subject: [PATCH] Get rid of two runtime static destructors Bug: 153704006 Test: compiles Change-Id: Icb44c2d2c1e26b13a4d03a4996bffe016a141286 --- services/incremental/BinderIncrementalService.cpp | 5 +++-- services/incremental/IncrementalService.cpp | 5 +++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/services/incremental/BinderIncrementalService.cpp b/services/incremental/BinderIncrementalService.cpp index 2dbbc5ac68060..aabc58c3a2977 100644 --- a/services/incremental/BinderIncrementalService.cpp +++ b/services/incremental/BinderIncrementalService.cpp @@ -17,6 +17,7 @@ #include "BinderIncrementalService.h" #include +#include #include #include #include @@ -93,8 +94,8 @@ BinderIncrementalService* BinderIncrementalService::start() { } status_t BinderIncrementalService::dump(int fd, const Vector&) { - static const String16 kDump("android.permission.DUMP"); - if (!PermissionCache::checkCallingPermission(kDump)) { + static const android::base::NoDestructor kDump("android.permission.DUMP"); + if (!PermissionCache::checkCallingPermission(*kDump)) { return PERMISSION_DENIED; } mImpl.onDump(fd); diff --git a/services/incremental/IncrementalService.cpp b/services/incremental/IncrementalService.cpp index 92366e51eb47a..4f64cad1f9d9e 100644 --- a/services/incremental/IncrementalService.cpp +++ b/services/incremental/IncrementalService.cpp @@ -20,6 +20,7 @@ #include #include +#include #include #include #include @@ -701,8 +702,8 @@ IncrementalService::IfsMountPtr IncrementalService::getIfs(StorageId storage) co const IncrementalService::IfsMountPtr& IncrementalService::getIfsLocked(StorageId storage) const { auto it = mMounts.find(storage); if (it == mMounts.end()) { - static const IfsMountPtr kEmpty = {}; - return kEmpty; + static const android::base::NoDestructor kEmpty{}; + return *kEmpty; } return it->second; }