diff --git a/services/src/fileoper/external_storage_utils.cpp b/services/src/fileoper/external_storage_utils.cpp
index 20ca2545ee9afa5640993e8d9d6c30e2c91dbcea..884039858bcacbaf528fffcf14a871b1495de998 100644
--- a/services/src/fileoper/external_storage_utils.cpp
+++ b/services/src/fileoper/external_storage_utils.cpp
@@ -37,11 +37,11 @@ static bool GetPathFromUri(const string &uri, string &path)
     uint len = EXTERNAL_STORAGE_URI.size();
     string head = uri.substr(0, len);
     if (head != EXTERNAL_STORAGE_URI) {
-        ERR_LOG("invalid format uri %{public}s, head check fail", uri.c_str());
+        ERR_LOG("invalid format uri %{private}s, head check fail", uri.c_str());
         return false;
     }
     if (uri.size() == len) {
-        ERR_LOG("uri content is invalid %{public}s", uri.c_str());
+        ERR_LOG("uri content is invalid %{private}s", uri.c_str());
         return false;
     }
     path = uri.substr(len);
@@ -116,13 +116,13 @@ int ExternalStorageUtils::DoListFile(const std::string &type, const std::string
     DEBUG_LOG("limit %{public}lld, offset %{public}lld", count, offset);
     std::string path;
     if (!ConvertUriToAbsolutePath(uri, path)) {
-        ERR_LOG("invalid uri[%{public}s].", uri.c_str());
+        ERR_LOG("invalid uri[%{private}s].", uri.c_str());
         return E_NOEXIST;
     }
 
     DIR *dir = opendir(path.c_str());
     if (!dir) {
-        ERR_LOG("opendir path[%{public}s] fail.", path.c_str());
+        ERR_LOG("opendir path[%{private}s] fail.", path.c_str());
         return E_NOEXIST;
     }
     if (offset != 0) {
diff --git a/services/src/fileoper/media_file_utils.cpp b/services/src/fileoper/media_file_utils.cpp
index c60ec96053be48ce210bd8aeadd0ce97362c995b..36fa68bf3c66e818f1a3e1613b272c33650d1e8b 100644
--- a/services/src/fileoper/media_file_utils.cpp
+++ b/services/src/fileoper/media_file_utils.cpp
@@ -81,12 +81,12 @@ bool GetPathID(const string &uriPath, string &index)
 {
     string::size_type pos = uriPath.find_last_of('/');
     if (pos == string::npos) {
-        ERR_LOG("invalid uri %{public}s", uriPath.c_str());
+        ERR_LOG("invalid uri %{private}s", uriPath.c_str());
         return false;
     }
     index = uriPath.substr(pos + 1);
     if (!IsNumber(index)) {
-        ERR_LOG("invalid uri %{public}s invalid id %{public}s", uriPath.c_str(), index.c_str());
+        ERR_LOG("invalid uri %{private}s invalid id %{public}s", uriPath.c_str(), index.c_str());
         return false;
     }
     return true;
@@ -331,7 +331,7 @@ int MediaFileUtils::DoInsert(const string &name, const string &path, const strin
         Media::MEDIA_FILEOPRN_CREATEASSET);
     int index = abilityHelper->Insert(createAsset, values);
     if (index < 0) {
-        ERR_LOG("Fail to create fail file %{public}s uri %{public}s album %{public}s", name.c_str(),
+        ERR_LOG("Fail to create fail file %{public}s uri %{private}s album %{public}s", name.c_str(),
             path.c_str(), albumPath.c_str());
         return E_CREATE_FAIL;
     }
@@ -420,7 +420,7 @@ bool MediaFileUtils::InitHelper(sptr<IRemoteObject> obj)
     if (abilityHelper == nullptr) {
         abilityHelper =  AppExecFwk::DataAbilityHelper::Creator(obj, make_shared<Uri>(Media::MEDIALIBRARY_DATA_URI));
         if (abilityHelper == nullptr) {
-            DEBUG_LOG("get %{public}s helper fail", Media::MEDIALIBRARY_DATA_URI.c_str());
+            DEBUG_LOG("get %{private}s helper fail", Media::MEDIALIBRARY_DATA_URI.c_str());
             return false;
         }
     }