diff --git a/frameworks/js/napi/accesstoken/src/napi_atmanager.cpp b/frameworks/js/napi/accesstoken/src/napi_atmanager.cpp
index 134967a0036566e078f8a09df6b23d39b6c4a36f..794297df13961ef7d1311a192deb3c62717a4cb3 100644
--- a/frameworks/js/napi/accesstoken/src/napi_atmanager.cpp
+++ b/frameworks/js/napi/accesstoken/src/napi_atmanager.cpp
@@ -1159,7 +1159,7 @@ bool NapiAtManager::FillPermStateChangeInfo(const napi_env env, const napi_value
     registerPermStateChangeInfo.subscriber->SetCallbackRef(callback);
     registerPermStateChangeInfo.threadId_ = std::this_thread::get_id();
     std::shared_ptr<RegisterPermStateChangeScopePtr> *subscriber =
-        new (std::nothrow) std::shared_ptr<RegisterPermStateChangeScopePtr>(
+        new (std::nothrow) std::make_shared<RegisterPermStateChangeScopePtr>(
             registerPermStateChangeInfo.subscriber);
     if (subscriber == nullptr) {
         ACCESSTOKEN_LOG_ERROR(LABEL, "Failed to create subscriber");
diff --git a/interfaces/innerkits/accesstoken/src/accesstoken_death_recipient.h b/interfaces/innerkits/accesstoken/src/accesstoken_death_recipient.h
index 2b9e8350c12e81049c55d37e90b7291fe83d79c8..bf566fdcfc19ef3005302f97bfceda4cce8ec935 100644
--- a/interfaces/innerkits/accesstoken/src/accesstoken_death_recipient.h
+++ b/interfaces/innerkits/accesstoken/src/accesstoken_death_recipient.h
@@ -25,7 +25,7 @@ namespace AccessToken {
 class AccessTokenDeathRecipient : public IRemoteObject::DeathRecipient {
 public:
     AccessTokenDeathRecipient() {}
-    virtual ~AccessTokenDeathRecipient() override = default;
+    ~AccessTokenDeathRecipient() override = default;
     void OnRemoteDied(const wptr<IRemoteObject>& object) override;
 };
 }  // namespace AccessToken
diff --git a/services/accesstokenmanager/main/cpp/src/permission/permission_manager.cpp b/services/accesstokenmanager/main/cpp/src/permission/permission_manager.cpp
index 6149d2920a7d884121cf70fb8de92ee27fa00da3..644f81c6b5fd6a31aceed1c20554c56197ef753b 100644
--- a/services/accesstokenmanager/main/cpp/src/permission/permission_manager.cpp
+++ b/services/accesstokenmanager/main/cpp/src/permission/permission_manager.cpp
@@ -574,8 +574,7 @@ void PermissionManager::ScopeToString(
     copy(tokenIDs.begin(), tokenIDs.end(), std::ostream_iterator<uint32_t>(str, ", "));
     std::string tokenidStr = str.str();
 
-    std::string permStr;
-    permStr = accumulate(permList.begin(), permList.end(), std::string(" "));
+    std::string permStr = accumulate(permList.begin(), permList.end(), std::string(" "));
 
     ACCESSTOKEN_LOG_INFO(LABEL, "TokenidStr = %{public}s permStr =%{public}s",
         tokenidStr.c_str(), permStr.c_str());
diff --git a/services/privacymanager/src/record/permission_record_manager.cpp b/services/privacymanager/src/record/permission_record_manager.cpp
index 0268ac750e7dbe462569b0320f0205da498b27c1..83e9faa34e394c3f3a37c3920b7ca1dfb5c60037 100644
--- a/services/privacymanager/src/record/permission_record_manager.cpp
+++ b/services/privacymanager/src/record/permission_record_manager.cpp
@@ -1211,8 +1211,7 @@ int32_t PermissionRecordManager::StopUsingPermission(
 
 void PermissionRecordManager::PermListToString(const std::vector<std::string>& permList)
 {
-    std::string permStr;
-    permStr = accumulate(permList.begin(), permList.end(), std::string(" "));
+    std::string permStr = accumulate(permList.begin(), permList.end(), std::string(" "));
 
     ACCESSTOKEN_LOG_INFO(LABEL, "PermStr =%{public}s.", permStr.c_str());
 }