diff --git a/src/core/net/srp_client.cpp b/src/core/net/srp_client.cpp
index ef2087b5e..e916d0ba1 100644
--- a/src/core/net/srp_client.cpp
+++ b/src/core/net/srp_client.cpp
@@ -83,7 +83,7 @@ void Client::HostInfo::EnableAutoAddress(void)
     mNumAddresses = 0;
     mAutoAddress  = true;
 
-    LogInfo("HostInfo enabled auto address", GetNumAddresses());
+    LogInfo("HostInfo enabled auto address");
 }
 
 void Client::HostInfo::SetAddresses(const Ip6::Address *aAddresses, uint8_t aNumAddresses)
diff --git a/src/core/thread/dua_manager.cpp b/src/core/thread/dua_manager.cpp
index a10b8e374..fd35ee0cf 100644
--- a/src/core/thread/dua_manager.cpp
+++ b/src/core/thread/dua_manager.cpp
@@ -595,7 +595,7 @@ void DuaManager::HandleTmf<kUriDuaRegistrationNotify>(Coap::Message &aMessage, c
 
 exit:
     OT_UNUSED_VARIABLE(error);
-    LogInfo("Received DUA.ntf: %d", ErrorToString(error));
+    LogInfo("Received DUA.ntf: %s", ErrorToString(error));
 }
 
 Error DuaManager::ProcessDuaResponse(Coap::Message &aMessage)
diff --git a/src/core/thread/mesh_forwarder_ftd.cpp b/src/core/thread/mesh_forwarder_ftd.cpp
index 14e1a8d6b..1c86ada73 100644
--- a/src/core/thread/mesh_forwarder_ftd.cpp
+++ b/src/core/thread/mesh_forwarder_ftd.cpp
@@ -959,7 +959,7 @@ void MeshForwarder::GetForwardFramePriority(const FrameData &     aFrameData,
 exit:
     if (error != kErrorNone)
     {
-        LogNote("Failed to get forwarded frame priority, error:%s, len:%d, src:%d, dst:%s", ErrorToString(error),
+        LogNote("Failed to get forwarded frame priority, error:%s, len:%d, src:%s, dst:%s", ErrorToString(error),
                 frameData.GetLength(), aMeshAddrs.mSource.ToString().AsCString(),
                 aMeshAddrs.mDestination.ToString().AsCString());
     }