diff --git a/services/samgr/native/source/collect/common_event_collect.cpp b/services/samgr/native/source/collect/common_event_collect.cpp
index 568a63e33e08f3c84fcdab64a0a64e699f27c21d..47d895579f9b3542fa20e73bb90deffac570a0f1 100644
--- a/services/samgr/native/source/collect/common_event_collect.cpp
+++ b/services/samgr/native/source/collect/common_event_collect.cpp
@@ -156,7 +156,7 @@ void CommonEventCollect::AddSkillsEvent(EventFwk::MatchingSkills& skill)
 {
     std::lock_guard<std::mutex> autoLock(commomEventLock_);
     for (auto& commonEventName : commonEventNames_) {
-        HILOGD("CommonEventCollect add event: %{puhlic}s", commonEventName.c_str());
+        HILOGD("CommonEventCollect add event: %{public}s", commonEventName.c_str());
         skill.AddEvent(commonEventName);
     }
 }
diff --git a/services/samgr/native/source/collect/device_param_collect.cpp b/services/samgr/native/source/collect/device_param_collect.cpp
index 1a89f822e65fb2ef860f27394a7c68852951d51c..8a4f7dfc74c5a7f6b1ac48c2b32c7f31c792fbdf 100644
--- a/services/samgr/native/source/collect/device_param_collect.cpp
+++ b/services/samgr/native/source/collect/device_param_collect.cpp
@@ -87,7 +87,7 @@ void DeviceParamCollect::WatchParameters()
 {
     std::lock_guard<std::mutex> autoLock(paramLock_);
     for (auto param : pendingParams_) {
-        HILOGD("DeviceParamCollect watch param: %{puhlic}s", param.c_str());
+        HILOGD("DeviceParamCollect watch param: %{public}s", param.c_str());
         int32_t result = WatchParameter(param.c_str(), DeviceParamCallback, this);
         if (result != ERR_OK) {
             HILOGE("DeviceParamCollect watch events: %{public}s failed", param.c_str());