diff --git a/services/core/ability/alarm/src/timer_manager.cpp b/services/core/ability/alarm/src/timer_manager.cpp
index 243f3fba177fac044bf9637106ff083e171fdd28..2e99cf456bb2fac049c4030ddb1fae64f7547e84 100644
--- a/services/core/ability/alarm/src/timer_manager.cpp
+++ b/services/core/ability/alarm/src/timer_manager.cpp
@@ -43,7 +43,7 @@ TimerManager::~TimerManager()
 bool TimerManager::RegisterLooperEvent(EventType eventType, int64_t looperInterval,
     const OHOS::Utils::Timer::TimerCallback &callback)
 {
-    ENGINE_LOGI("TimerManager registerLooperEvent EventType %{puiblic}d", CAST_INT(eventType));
+    ENGINE_LOGI("TimerManager registerLooperEvent EventType %{public}d", CAST_INT(eventType));
     UnregisterLooperEvent(eventType);
     auto weakThis = weak_from_this();
     if (timer_ == nullptr) {
@@ -56,7 +56,7 @@ bool TimerManager::RegisterLooperEvent(EventType eventType, int64_t looperInterv
                 ENGINE_LOGE("TimerManager is destroyed");
                 return;
             }
-            ENGINE_LOGD("Looper EventType %{publoc}d triggered", CAST_INT(eventType));
+            ENGINE_LOGD("Looper EventType %{public}d triggered", CAST_INT(eventType));
             callback();
         },
         looperInterval * ONE_SECOND_MILLISECONDS, false);
@@ -68,7 +68,7 @@ bool TimerManager::RegisterLooperEvent(EventType eventType, int64_t looperInterv
 
 void TimerManager::UnregisterLooperEvent(EventType eventType)
 {
-    ENGINE_LOGI("TimerManager UnregisterLooperEvent EventType %{puiblic}d", CAST_INT(eventType));
+    ENGINE_LOGI("TimerManager UnregisterLooperEvent EventType %{public}d", CAST_INT(eventType));
     std::lock_guard<std::mutex> lockGuard(mutex_);
     auto result = registeredTimerIdMap_.find(eventType);
     if (result == registeredTimerIdMap_.end()) {
diff --git a/services/core/ability/utils/src/sha256_utils.cpp b/services/core/ability/utils/src/sha256_utils.cpp
index 7b69d2a5ac565cc7ad1acb803474021c2996dd52..a0e1d7beb7f7c329cbe47f9c15c77462a35ff269 100644
--- a/services/core/ability/utils/src/sha256_utils.cpp
+++ b/services/core/ability/utils/src/sha256_utils.cpp
@@ -44,7 +44,7 @@ std::string Sha256Utils::CalculateHashCode(std::string inputStr)
 bool Sha256Utils::CheckFileSha256String(const std::string &fileName, const std::string &sha256String)
 {
     if (!FileUtils::IsFileExist(fileName)) {
-        ENGINE_LOGE("check file sha256 failed, fileName = %{pubilc}s is not exist", fileName.c_str());
+        ENGINE_LOGE("check file sha256 failed, fileName = %{public}s is not exist", fileName.c_str());
         return false;
     }
     char sha256Result[SHA256_STRING_LEN] = {0}; // sha256Result len is 65
diff --git a/services/firmware/upgrade/status/src/firmware_result_process.cpp b/services/firmware/upgrade/status/src/firmware_result_process.cpp
index c4f12669722bb1b0356fad8ae7f2b276f1c648c7..193e6be178c457802e1d8b5416cb057b6c38c7be 100644
--- a/services/firmware/upgrade/status/src/firmware_result_process.cpp
+++ b/services/firmware/upgrade/status/src/firmware_result_process.cpp
@@ -79,7 +79,7 @@ UpdateResult FirmwareResultProcess::CompareVersion(const FirmwareComponent &comp
 {
     bool isResultSuccess = false;
     isResultSuccess = component.versionNumber == FirmwareUpdateAdapter::GetDisplayVersion();
-    FIRMWARE_LOGI("component.versionNumber=%{pubilc}s, GetDisplayVersion=%{pubilc}s",
+    FIRMWARE_LOGI("component.versionNumber=%{public}s, GetDisplayVersion=%{public}s",
         component.versionNumber.c_str(), FirmwareUpdateAdapter::GetDisplayVersion().c_str());
     UpdateResult updateResult;
     updateResult.spath = component.spath;