diff --git a/services/accesstokenmanager/main/cpp/src/permission/short_grant_manager.cpp b/services/accesstokenmanager/main/cpp/src/permission/short_grant_manager.cpp
index a3b23b5efc2729a509b275da261f148d7a460e5b..5317211e40498119e12fb5b501b78a67a7f7f72c 100644
--- a/services/accesstokenmanager/main/cpp/src/permission/short_grant_manager.cpp
+++ b/services/accesstokenmanager/main/cpp/src/permission/short_grant_manager.cpp
@@ -142,7 +142,6 @@ int ShortGrantManager::RefreshPermission(AccessTokenID tokenID, const std::strin
         shortGrantData_.begin(), shortGrantData_.end(), [tokenID, permission](const PermTimerData& data) {
         return data.tokenID == tokenID && data.permissionName == permission;
     });
-
     if (iter == shortGrantData_.end()) {
         auto iterator = std::find(g_shortGrantPermission.begin(), g_shortGrantPermission.end(), permission);
         if (iterator == g_shortGrantPermission.end()) {
@@ -251,7 +250,7 @@ void ShortGrantManager::ScheduleRevokeTask(AccessTokenID tokenID, const std::str
         LOGI(ATM_DOMAIN, ATM_TAG,
             "Token: %{public}d, permission: %{public}s, delay revoke permission end.", tokenID, permission.c_str());
     });
-    LOGI(ATM_DOMAIN, ATM_TAG, "cancelTimes %{public}d", cancelTimes);
+    LOGI(ATM_DOMAIN, ATM_TAG, "CancelTimes %{public}d.", cancelTimes);
     eventHandler->ProxyPostTask(delayed, taskName, cancelTimes * 1000); // 1000 means to ms
     return;
 #else
diff --git a/services/accesstokenmanager/main/cpp/src/token/token_modify_notifier.cpp b/services/accesstokenmanager/main/cpp/src/token/token_modify_notifier.cpp
index e4b74f2e79fb8267c7af4bbd891990b0d205a0b8..ac7206df273047dc46665ce909304aba2031ebda 100644
--- a/services/accesstokenmanager/main/cpp/src/token/token_modify_notifier.cpp
+++ b/services/accesstokenmanager/main/cpp/src/token/token_modify_notifier.cpp
@@ -175,11 +175,8 @@ int32_t TokenModifyNotifier::GetRemoteHapTokenInfo(const std::string& deviceID,
 int32_t TokenModifyNotifier::RegisterTokenSyncCallback(const sptr<IRemoteObject>& callback)
 {
     Utils::UniqueWriteGuard<Utils::RWLock> infoGuard(this->notifyLock_);
-    tokenSyncCallbackObject_ = new (std::nothrow) TokenSyncCallbackProxy(callback);
+    tokenSyncCallbackObject_ = new TokenSyncCallbackProxy(callback);
     tokenSyncCallbackDeathRecipient_ = sptr<TokenSyncCallbackDeathRecipient>::MakeSptr();
-    if (tokenSyncCallbackDeathRecipient_ == nullptr) {
-        return RET_FAILED;
-    }
     callback->AddDeathRecipient(tokenSyncCallbackDeathRecipient_);
     LOGI(ATM_DOMAIN, ATM_TAG, "Register token sync callback successful.");
     return ERR_OK;
diff --git a/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp b/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp
index ea42a01544c91ce3f390be114b2a34180410719c..9838e84848d8941ba4dfaacfc04d3980683d8fcf 100644
--- a/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp
+++ b/services/accesstokenmanager/test/database_dlopen/database_dlopen_coverage_test.cpp
@@ -28,7 +28,6 @@ namespace Security {
 namespace AccessToken {
 namespace {
 constexpr const int32_t WAIT_EVENTHANDLE_TIME = 2;
-constexpr const int32_t WAIT_DELAY_DLCLOSE_TIME = 7;
 }
 class AccessTokenDatabaseDlopenTest : public testing::Test {
 public: