diff --git a/interfaces/kits/js/src/file_manager_napi.cpp b/interfaces/kits/js/src/file_manager_napi.cpp
index 8d9d0add4876fdfe5333461e5d8a62a786066602..bf6732d660c9f85215dc30f6a8fe976280bdff6e 100644
--- a/interfaces/kits/js/src/file_manager_napi.cpp
+++ b/interfaces/kits/js/src/file_manager_napi.cpp
@@ -31,7 +31,7 @@ using namespace std;
 using namespace DistributedFS;
 struct AsyncFileInfoArg {
     NRef ref_;
-    vector<unique_ptr<FileInfo>> fileRes_;
+    vector<shared_ptr<FileInfo>> fileRes_;
     explicit AsyncFileInfoArg(NVal ref) : ref_(ref), fileRes_() {};
     ~AsyncFileInfoArg() = default;
 };
@@ -181,7 +181,7 @@ void CreateFileArray(napi_env env, shared_ptr<AsyncFileInfoArg> arg)
 {
     for (unsigned int i = 0; i < arg->fileRes_.size(); i++) {
         NVal obj = NVal::CreateObject(env);
-        unique_ptr<FileInfo> &res = arg->fileRes_[i];
+        shared_ptr<FileInfo> &res = arg->fileRes_[i];
         obj.AddProp("name", NVal::CreateUTF8String(env, res->GetName()).val_);
         obj.AddProp("path", NVal::CreateUTF8String(env, res->GetPath()).val_);
         obj.AddProp("type", NVal::CreateUTF8String(env, res->GetType()).val_);
diff --git a/services/src/client/file_manager_proxy.cpp b/services/src/client/file_manager_proxy.cpp
index b3a120ba2ed6315afd6c1358f14f0d842f9e57d8..629e95a8d0c0397e43c75b177e832fa3c366cb35 100644
--- a/services/src/client/file_manager_proxy.cpp
+++ b/services/src/client/file_manager_proxy.cpp
@@ -14,6 +14,7 @@
  */
 #include "file_manager_proxy.h"
 
+#include "cmd_response.h"
 #include "file_info.h"
 #include "file_manager_service_def.h"
 #include "file_manager_service_errno.h"
@@ -25,9 +26,19 @@ using namespace std;
 
 namespace OHOS {
 namespace FileManagerService {
+static int GetCmdResponse(MessageParcel &reply, sptr<CmdResponse> &cmdResponse)
+{
+    cmdResponse = reply.ReadParcelable<CmdResponse>();
+    if (cmdResponse == nullptr) {
+        ERR_LOG("Unmarshalling cmdResponse fail");
+        return FAIL;
+    }
+    return cmdResponse->GetErr();
+}
+
 FileManagerProxy::FileManagerProxy(const sptr<IRemoteObject> &impl)
     : IRemoteProxy<IFileManagerService>(impl) {}
-int FileManagerProxy::GetRoot(const CmdOptions &option, vector<unique_ptr<FileInfo>> &fileRes)
+int FileManagerProxy::GetRoot(const CmdOptions &option, vector<shared_ptr<FileInfo>> &fileRes)
 {
     CmdOptions op(option);
     if (op.GetDevInfo().GetName() == "external_storage") {
@@ -41,15 +52,12 @@ int FileManagerProxy::GetRoot(const CmdOptions &option, vector<unique_ptr<FileIn
             ERR_LOG("GetRoot inner error send request fail %{public}d", err);
             return FAIL;
         }
-        int count = 0;
-        reply.ReadInt32(count);
-        for (int i = 0; i < count; i++) {
-            string rootPath;
-            reply.ReadString(rootPath);
-            unique_ptr<FileInfo> file = make_unique<FileInfo>(FILE_ROOT_NAME, rootPath, ALBUM_TYPE);
-            fileRes.emplace_back(move(file));
+        sptr<CmdResponse> cmdResponse;
+        err = GetCmdResponse(reply, cmdResponse);
+        if (err != ERR_NONE) {
+            return err;
         }
-        reply.ReadInt32(err);
+        fileRes = cmdResponse->GetFileInfoList();
         return err;
     } else {
         unique_ptr<FileInfo> image = make_unique<FileInfo>(IMAGE_ROOT_NAME, FISRT_LEVEL_ALBUM, ALBUM_TYPE);
@@ -82,8 +90,12 @@ int FileManagerProxy::CreateFile(const std::string &path, const std::string &fil
         ERR_LOG("inner error send request fail %{public}d", err);
         return FAIL;
     }
-    reply.ReadString(uri);
-    reply.ReadInt32(err);
+    sptr<CmdResponse> cmdResponse;
+    err = GetCmdResponse(reply, cmdResponse);
+    if (err != ERR_NONE) {
+        return err;
+    }
+    uri = cmdResponse->GetUri();
     return err;
 }
 
@@ -104,7 +116,7 @@ IFmsClient *IFmsClient::GetFmsInstance()
 }
 
 int FileManagerProxy::ListFile(const std::string &type, const std::string &path, const CmdOptions &option,
-    std::vector<unique_ptr<FileInfo>> &fileRes)
+    std::vector<std::shared_ptr<FileInfo>> &fileRes)
 {
     MessageParcel data;
     CmdOptions op(option);
@@ -130,14 +142,12 @@ int FileManagerProxy::ListFile(const std::string &type, const std::string &path,
         ERR_LOG("inner error send request fail %{public}d", err);
         return FAIL;
     }
-    int fileInfoNum = 0;
-    reply.ReadInt32(fileInfoNum);
-    while (fileInfoNum) {
-        unique_ptr<FileInfo> file(reply.ReadParcelable<FileInfo>());
-        fileRes.emplace_back(move(file));
-        fileInfoNum--;
+    sptr<CmdResponse> cmdResponse;
+    err = GetCmdResponse(reply, cmdResponse);
+    if (err != ERR_NONE) {
+        return err;
     }
-    reply.ReadInt32(err);
+    fileRes = cmdResponse->GetFileInfoList();
     return err;
 }
 
diff --git a/services/src/client/file_manager_proxy.h b/services/src/client/file_manager_proxy.h
index 085d32361989e70bc933156aab91ba787f78c6ec..494648ed26703f008083ec301e84e68a30329f74 100644
--- a/services/src/client/file_manager_proxy.h
+++ b/services/src/client/file_manager_proxy.h
@@ -29,10 +29,10 @@ public:
     virtual ~FileManagerProxy() = default;
     int Mkdir(const std::string &name, const std::string &path) override;
     int ListFile(const std::string &type, const std::string &path, const CmdOptions &option,
-        std::vector<std::unique_ptr<FileInfo>> &fileRes) override;
+        std::vector<std::shared_ptr<FileInfo>> &fileRes) override;
     int CreateFile(const std::string &path, const std::string &fileName,
         const CmdOptions &option, std::string &uri) override;
-    int GetRoot(const CmdOptions &option, std::vector<std::unique_ptr<FileInfo>> &fileRes) override;
+    int GetRoot(const CmdOptions &option, std::vector<std::shared_ptr<FileInfo>> &fileRes) override;
 private:
     static inline BrokerDelegator<FileManagerProxy> delegator_;
 };
diff --git a/services/src/client/ifms_client.h b/services/src/client/ifms_client.h
index 386c4febe5c168cdf0a62da52b9acb289fcb65b5..ba0ce14abfe063493042b82988963f0a168dee63 100644
--- a/services/src/client/ifms_client.h
+++ b/services/src/client/ifms_client.h
@@ -24,8 +24,8 @@ public:
     static IFmsClient *GetFmsInstance();
     virtual int Mkdir(const std::string &name, const std::string &path) = 0;
     virtual int ListFile(const std::string &type, const std::string &path, const CmdOptions &option,
-        std::vector<std::unique_ptr<FileInfo>> &fileRes) = 0;
-    virtual int GetRoot(const CmdOptions &option, std::vector<std::unique_ptr<FileInfo>> &fileRes) = 0;
+        std::vector<std::shared_ptr<FileInfo>> &fileRes) = 0;
+    virtual int GetRoot(const CmdOptions &option, std::vector<std::shared_ptr<FileInfo>> &fileRes) = 0;
     virtual int CreateFile(const std::string &path, const std::string &fileName,
         const CmdOptions &option, std::string &uri) = 0;
 };
diff --git a/services/src/fileoper/cmd_response.h b/services/src/fileoper/cmd_response.h
new file mode 100644
index 0000000000000000000000000000000000000000..eb65bac408d0c51add40144f857d02b37098586d
--- /dev/null
+++ b/services/src/fileoper/cmd_response.h
@@ -0,0 +1,98 @@
+/*
+ * Copyright (C) 2021 Huawei Device Co., Ltd.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+#ifndef STORAGE_SERVICES_CMD_RESPONSE_H
+#define STORAGE_SERVICES_CMD_RESPONSE_H
+
+#include <memory>
+#include <string>
+#include <vector>
+
+#include "file_info.h"
+#include "parcel.h"
+namespace OHOS {
+namespace FileManagerService {
+class CmdResponse : public Parcelable {
+public:
+    CmdResponse() = default;
+    CmdResponse(int &err, std::string &uri, std::vector<std::shared_ptr<FileInfo>> &fileInfoList)
+        : err_(err), uri_(uri), vecFileInfo_(fileInfoList)
+    {}
+    ~CmdResponse() = default;
+
+    void SetErr(const int err)
+    {
+        err_ = err;
+    }
+
+    int GetErr() const
+    {
+        return err_;
+    }
+
+    void SetUri(const std::string &uri)
+    {
+        uri_ = uri;
+    }
+
+    std::string GetUri() const
+    {
+        return uri_;
+    }
+
+    void SetFileInfoList(std::vector<std::shared_ptr<FileInfo>> &fileInfoList)
+    {
+        vecFileInfo_ = fileInfoList;
+    }
+
+    std::vector<std::shared_ptr<FileInfo>> GetFileInfoList()
+    {
+        return vecFileInfo_;
+    }
+
+    virtual bool Marshalling(Parcel &parcel) const override
+    {
+        parcel.WriteInt32(err_);
+        parcel.WriteString(uri_);
+        int fileCount = vecFileInfo_.size();
+        parcel.WriteInt32(fileCount);
+        for (int i = 0; i < fileCount; i++) {
+            parcel.WriteParcelable(vecFileInfo_[i].get());
+        }
+        return true;
+    }
+
+    static CmdResponse* Unmarshalling(Parcel &parcel)
+    {
+        auto *obj = new (std::nothrow) CmdResponse();
+        if (obj == nullptr) {
+            return nullptr;
+        }
+        obj->err_ = parcel.ReadInt32();
+        obj->uri_ = parcel.ReadString();
+        int fileCount = parcel.ReadInt32();
+        for (int i = 0; i < fileCount; i++) {
+            std::shared_ptr<FileInfo> file(parcel.ReadParcelable<FileInfo>());
+            obj->vecFileInfo_.emplace_back(move(file));
+        }
+        return obj;
+    }
+private:
+    int err_;
+    std::string uri_;
+    std::vector<std::shared_ptr<FileInfo>> vecFileInfo_;
+};
+} // FileManagerService
+} // namespace OHOS
+#endif // STORAGE_SERVICES_CMD_RESPONSE_H
\ No newline at end of file
diff --git a/services/src/fileoper/external_storage_oper.cpp b/services/src/fileoper/external_storage_oper.cpp
index a23e5b92ad1d1ba7d64335f06eb919302539c2db..af53f499e37747e3b31933855b60fe81bbd03ab9 100644
--- a/services/src/fileoper/external_storage_oper.cpp
+++ b/services/src/fileoper/external_storage_oper.cpp
@@ -17,6 +17,7 @@
 
 #include <vector>
 
+#include "cmd_response.h"
 #include "external_storage_utils.h"
 #include "file_info.h"
 #include "file_manager_service_def.h"
@@ -66,18 +67,36 @@ int ExternalStorageOper::OperProcess(uint32_t code, MessageParcel &data, Message
 
 int ExternalStorageOper::GetRoot(const std::string &name, const std::string &path, MessageParcel &reply) const
 {
-    return ExternalStorageUtils::DoGetRoot(name, path, reply);
+    std::vector<std::shared_ptr<FileInfo>> fileList;
+    int ret = ExternalStorageUtils::DoGetRoot(name, path, fileList);
+    CmdResponse cmdResponse;
+    cmdResponse.SetErr(ret);
+    cmdResponse.SetFileInfoList(fileList);
+    reply.WriteParcelable(&cmdResponse);
+    return ret;
 }
 
 int ExternalStorageOper::CreateFile(const std::string &uri, const std::string &name, MessageParcel &reply) const
 {
-    return ExternalStorageUtils::DoCreateFile(uri, name, reply);
+    std::string resultUir;
+    int ret = ExternalStorageUtils::DoCreateFile(uri, name, resultUir);
+    CmdResponse cmdResponse;
+    cmdResponse.SetErr(ret);
+    cmdResponse.SetUri(resultUir);
+    reply.WriteParcelable(&cmdResponse);
+    return ret;
 }
 
 int ExternalStorageOper::ListFile(const std::string &type, const std::string &uri, const CmdOptions &option,
     MessageParcel &reply) const
 {
-    return ExternalStorageUtils::DoListFile(type, uri, reply);
+    std::vector<std::shared_ptr<FileInfo>> fileList;
+    int ret = ExternalStorageUtils::DoListFile(type, uri, fileList);
+    CmdResponse cmdResponse;
+    cmdResponse.SetErr(ret);
+    cmdResponse.SetFileInfoList(fileList);
+    reply.WriteParcelable(&cmdResponse);
+    return ret;
 }
 } // namespace FileManagerService
 } // namespace OHOS
\ No newline at end of file
diff --git a/services/src/fileoper/external_storage_utils.cpp b/services/src/fileoper/external_storage_utils.cpp
index 27db06ace8acbf3fdc609d3975c26b76da5f2f2e..fba7dd0cce470a263918695b91d89eb58f84f39f 100644
--- a/services/src/fileoper/external_storage_utils.cpp
+++ b/services/src/fileoper/external_storage_utils.cpp
@@ -59,7 +59,7 @@ static bool GetRealPath(string &path)
     return true;
 }
 
-static bool GetFileInfo(const std::string &path, const std::string &name, unique_ptr<FileInfo> &fileInfo)
+static bool GetFileInfo(const std::string &path, const std::string &name, shared_ptr<FileInfo> &fileInfo)
 {
     std::string fullPath(path);
     size_t len = fullPath.size();
@@ -104,46 +104,35 @@ static bool ConvertUriToAbsolutePath(const std::string &uri, std::string &path)
     return true;
 }
 
-int ExternalStorageUtils::DoListFile(const std::string &type, const std::string &uri, MessageParcel &reply)
+int ExternalStorageUtils::DoListFile(const std::string &type, const std::string &uri,
+    std::vector<shared_ptr<FileInfo>> &fileList)
 {
     std::string path;
-    int fileCount = 0;
     if (!ConvertUriToAbsolutePath(uri, path)) {
         ERR_LOG("invalid uri[%{public}s].", uri.c_str());
-        reply.WriteInt32(fileCount);
         return E_NOEXIST;
     }
 
     DIR *dir = opendir(path.c_str());
     if (!dir) {
         ERR_LOG("opendir path[%{public}s] fail.", path.c_str());
-        reply.WriteInt32(fileCount);
         return E_NOEXIST;
     }
-    std::vector<unique_ptr<FileInfo>> fileList;
     for (struct dirent *ent = readdir(dir); ent != nullptr; ent = readdir(dir)) {
         if (strcmp(ent->d_name, ".") == 0 || strcmp(ent->d_name, "..") == 0) {
             continue;
         }
-        unique_ptr<FileInfo> fileInfo = make_unique<FileInfo>();
+        shared_ptr<FileInfo> fileInfo = make_shared<FileInfo>();
         if (!GetFileInfo(path, ent->d_name, fileInfo)) {
             continue;
         }
         fileList.push_back(move(fileInfo));
     }
     closedir(dir);
-    fileCount = static_cast<int32_t>(fileList.size());
-    reply.WriteInt32(fileCount);
-    if (fileCount == 0) {
-        return E_EMPTYFOLDER;
-    }
-    for (int i = 0; i < fileCount; i++) {
-        reply.WriteParcelable(fileList[i].get());
-    }
     return SUCCESS;
 }
 
-int ExternalStorageUtils::DoCreateFile(const std::string &uri, const std::string &name, MessageParcel &reply)
+int ExternalStorageUtils::DoCreateFile(const std::string &uri, const std::string &name, std::string &resultUri)
 {
     std::string path;
     if (!ConvertUriToAbsolutePath(uri, path)) {
@@ -167,48 +156,23 @@ int ExternalStorageUtils::DoCreateFile(const std::string &uri, const std::string
         return E_CREATE_FAIL;
     }
     close(fd);
-
-    std::string fullUri(EXTERNAL_STORAGE_URI);
-    fullUri.append(path);
-    reply.WriteString(fullUri);
+    resultUri = EXTERNAL_STORAGE_URI + path;
     return SUCCESS;
 }
 
-int ExternalStorageUtils::DoGetRoot(const std::string &name, const std::string &path, MessageParcel &reply)
+int ExternalStorageUtils::DoGetRoot(const std::string &name, const std::string &path,
+    std::vector<shared_ptr<FileInfo>> &fileList)
 {
     vector<string> vecRootPath;
     if (!StorageManagerInf::GetMountedVolumes(vecRootPath)) {
         ERR_LOG("there is valid extorage storage");
-        reply.WriteInt32(0);
         return FAIL;
     }
-    reply.WriteInt32(vecRootPath.size());
     for (auto rootPath : vecRootPath) {
-        reply.WriteString(rootPath);
+        shared_ptr<FileInfo> fileInfo = make_shared<FileInfo>(FILE_ROOT_NAME, rootPath, ALBUM_TYPE);
+        fileList.push_back(move(fileInfo));
     }
     return SUCCESS;
 }
-
-bool ExternalStorageUtils::PopFileInfo(FileInfo &fileInfo, MessageParcel &reply)
-{
-    std::string path;
-    std::string type;
-    std::string name;
-    int64_t size = 0;
-    int64_t at = 0;
-    int64_t mt = 0;
-
-    reply.ReadString(path);
-    reply.ReadString(type);
-    reply.ReadString(name);
-    reply.ReadInt64(size);
-    reply.ReadInt64(at);
-    reply.ReadInt64(mt);
-    fileInfo = FileInfo(name, path, type);
-    fileInfo.SetSize(size);
-    fileInfo.SetAddedTime(at);
-    fileInfo.SetModifiedTime(mt);
-    return true;
-}
 } // namespace FileManagerService
 } // namespace OHOS
\ No newline at end of file
diff --git a/services/src/fileoper/external_storage_utils.h b/services/src/fileoper/external_storage_utils.h
index 61a139da2f78a11cac8d74788324ef114380334a..52687984e8190f24b109f682d46c523ca1413776 100644
--- a/services/src/fileoper/external_storage_utils.h
+++ b/services/src/fileoper/external_storage_utils.h
@@ -15,6 +15,7 @@
 #ifndef STORAGE_SERIVCES_EXTERNAL_STORAGE_UTILS_H
 #define STORAGE_SERIVCES_EXTERNAL_STORAGE_UTILS_H
 
+#include <memory>
 #include <string>
 #include <vector>
 
@@ -28,10 +29,11 @@ class ExternalStorageUtils {
 public:
     ExternalStorageUtils();
     ~ExternalStorageUtils();
-    static int DoListFile(const std::string &type, const std::string &uri, MessageParcel &reply);
-    static int DoCreateFile(const std::string &uri, const std::string &name, MessageParcel &reply);
-    static bool PopFileInfo(FileInfo &fileInfo, MessageParcel &reply);
-    static int DoGetRoot(const std::string &name, const std::string &path, MessageParcel &reply);
+    static int DoListFile(const std::string &type, const std::string &uri,
+        std::vector<std::shared_ptr<FileInfo>> &fileList);
+    static int DoCreateFile(const std::string &uri, const std::string &name, std::string &resultUri);
+    static int DoGetRoot(const std::string &name, const std::string &path,
+        std::vector<std::shared_ptr<FileInfo>> &fileList);
 };
 } // namespace FileManagerService
 } // namespace OHOS
diff --git a/services/src/fileoper/file_info.cpp b/services/src/fileoper/file_info.cpp
index 46168b766553ee2e69abf24383153a876ac23528..3eaf90067c1e03dc9f90954ed76c30fed9b15017 100644
--- a/services/src/fileoper/file_info.cpp
+++ b/services/src/fileoper/file_info.cpp
@@ -28,11 +28,13 @@ bool FileInfo::Marshalling(Parcel &parcel) const
     parcel.WriteInt64(modifiedTime_);
     return true;
 }
+
 FileInfo* FileInfo::Unmarshalling(Parcel &parcel)
 {
     auto *obj = new (std::nothrow) FileInfo();
     if (obj == nullptr) {
         ERR_LOG("Unmarshalling fail");
+        return nullptr;
     }
     obj->path_ = parcel.ReadString();
     obj->name_ = parcel.ReadString();
diff --git a/services/src/fileoper/media_file_oper.cpp b/services/src/fileoper/media_file_oper.cpp
index dbc2520c03dab003f8814dd7a4b0fd6cb9589edb..3e9fd3fe2c0d3af3a313a8e2ccc0b9a8afa554f8 100644
--- a/services/src/fileoper/media_file_oper.cpp
+++ b/services/src/fileoper/media_file_oper.cpp
@@ -18,6 +18,7 @@
 #include <vector>
 
 #include "cmd_options.h"
+#include "cmd_response.h"
 #include "file_info.h"
 #include "file_manager_service_def.h"
 #include "file_manager_service_errno.h"
@@ -58,9 +59,7 @@ int MediaFileOper::OperProcess(uint32_t code, MessageParcel &data, MessageParcel
         case Operation::CREATE_FILE: {
             string name = data.ReadString();
             string path = data.ReadString();
-            string uri;
-            errCode = CreateFile(name, path, uri);
-            reply.WriteString(uri);
+            errCode = CreateFile(name, path, reply);
             break;
         }
         default: {
@@ -71,17 +70,33 @@ int MediaFileOper::OperProcess(uint32_t code, MessageParcel &data, MessageParcel
     return errCode;
 }
 
-int MediaFileOper::CreateFile(const std::string &name, const std::string &path, std::string &uri) const
+int MediaFileOper::CreateFile(const std::string &name, const std::string &path, MessageParcel &reply) const
 {
     string type = "file";
-    return MediaFileUtils::DoInsert(name, path, type, uri);
+    std::string uri;
+    int ret = MediaFileUtils::DoInsert(name, path, type, uri);
+    CmdResponse cmdResponse;
+    cmdResponse.SetErr(ret);
+    cmdResponse.SetUri(uri);
+    reply.WriteParcelable(&cmdResponse);
+    return ret;
 }
 
 int MediaFileOper::ListFile(const string &type, const string &path, int offset, int count, MessageParcel &reply) const
 {
     shared_ptr<NativeRdb::AbsSharedResultSet> result;
     int res = MediaFileUtils::DoListFile(type, path, offset, count, result);
-    return  MediaFileUtils::GetFileInfoFromResult(result, reply, res);
+    if (res != SUCCESS) {
+        return res;
+    }
+
+    std::vector<std::shared_ptr<FileInfo>> fileList;
+    res = MediaFileUtils::GetFileInfoFromResult(result, fileList);
+    CmdResponse cmdResponse;
+    cmdResponse.SetErr(res);
+    cmdResponse.SetFileInfoList(fileList);
+    reply.WriteParcelable(&cmdResponse);
+    return res;
 }
 
 int MediaFileOper::Mkdir(const string &name, const string &path) const
diff --git a/services/src/fileoper/media_file_oper.h b/services/src/fileoper/media_file_oper.h
index 8220a0ca5841b05d082625dbe1b27c8d6ee77fce..0652065da9e711487ba1fcc0a243dbfbbc217227 100644
--- a/services/src/fileoper/media_file_oper.h
+++ b/services/src/fileoper/media_file_oper.h
@@ -25,7 +25,7 @@ public:
     virtual ~MediaFileOper() = default;
     int OperProcess(uint32_t code, MessageParcel &data, MessageParcel &reply) const override;
 private:
-    int CreateFile(const std::string &name, const std::string &path, std::string &uri) const;
+    int CreateFile(const std::string &name, const std::string &path, MessageParcel &reply) const;
     int ListFile(const std::string &type, const std::string &path, int offset, int count, MessageParcel &data) const;
     int Mkdir(const std::string &name, const std::string &path) const;
 };
diff --git a/services/src/fileoper/media_file_utils.cpp b/services/src/fileoper/media_file_utils.cpp
index 5fbcba7406d3b3bdc156099484f2f9e0349f7671..da9768130389e1f619dc91a315e174c0ba4b2fe6 100644
--- a/services/src/fileoper/media_file_utils.cpp
+++ b/services/src/fileoper/media_file_utils.cpp
@@ -357,7 +357,8 @@ bool MediaFileUtils::InitMediaTableColIndexMap(shared_ptr<NativeRdb::AbsSharedRe
     return true;
 }
 
-bool MediaFileUtils::PushFileInfo(shared_ptr<NativeRdb::AbsSharedResultSet> result, MessageParcel &reply)
+bool MediaFileUtils::GetFileInfo(shared_ptr<NativeRdb::AbsSharedResultSet> result,
+    shared_ptr<FileInfo> &fileInfo)
 {
     if (!InitMediaTableColIndexMap(result)) {
         ERR_LOG("InitMediaTableColIndexMap returns fail");
@@ -368,45 +369,39 @@ bool MediaFileUtils::PushFileInfo(shared_ptr<NativeRdb::AbsSharedResultSet> resu
     result->GetString(mediaTableMap[index++].first, id);
     string uri;
     result->GetString(mediaTableMap[index++].first, uri);
-    unique_ptr<FileInfo> file = make_unique<FileInfo>();
+
     string path = uri + "/" + id;
-    file->SetPath(path);
+    fileInfo->SetPath(path);
     string type;
     result->GetString(mediaTableMap[index++].first, type);
-    file->SetType(type);
+    fileInfo->SetType(type);
     string name;
     result->GetString(mediaTableMap[index++].first, name);
-    file->SetName(name);
+    fileInfo->SetName(name);
     int64_t value;
     result->GetLong(mediaTableMap[index++].first, value);
-    file->SetSize(value);
+    fileInfo->SetSize(value);
     result->GetLong(mediaTableMap[index++].first, value);
-    file->SetAddedTime(value);
+    fileInfo->SetAddedTime(value);
     result->GetLong(mediaTableMap[index++].first, value);
-    file->SetModifiedTime(value);
-    reply.WriteParcelable(file.get());
+    fileInfo->SetModifiedTime(value);
     return true;
 }
 
 int MediaFileUtils::GetFileInfoFromResult(shared_ptr<NativeRdb::AbsSharedResultSet> result,
-    MessageParcel &reply, int res)
+    std::vector<shared_ptr<FileInfo>> &fileList)
 {
     int count = 0;
-    if (res) {
-        // deal with status isn't succ;
-        ERR_LOG("AbsSharedResultSet status isn't succ");
-        reply.WriteInt32(count);
-        return res;
-    }
     result->GetRowCount(count);
-    reply.WriteInt32(count);
     if (count == 0) {
         ERR_LOG("AbsSharedResultSet null");
         return E_EMPTYFOLDER;
     }
     result->GoToFirstRow();
     for (int i = 0; i < count; i++) {
-        PushFileInfo(result, reply);
+        shared_ptr<FileInfo> fileInfo = make_shared<FileInfo>();
+        GetFileInfo(result, fileInfo);
+        fileList.push_back(move(fileInfo));
         result->GoToNextRow();
     }
     return SUCCESS;
diff --git a/services/src/fileoper/media_file_utils.h b/services/src/fileoper/media_file_utils.h
index 56ceff40d38cf582596d676d3ed9f247332f49d1..2bdfdfa167f9adf33479203c87e5cc68bed63740 100644
--- a/services/src/fileoper/media_file_utils.h
+++ b/services/src/fileoper/media_file_utils.h
@@ -40,10 +40,9 @@ public:
     static std::shared_ptr<NativeRdb::AbsSharedResultSet> DoQuery(const std::string &selection,
         const std::vector<std::string> &selectionArgs);
     static int DoInsert(const std::string &name, const std::string &path, const std::string &type, std::string &uri);
-    static bool PushFileInfo(std::shared_ptr<NativeRdb::AbsSharedResultSet> result, MessageParcel &reply);
-    static bool PopFileInfo(FileInfo &file, MessageParcel &reply);
+    static bool GetFileInfo(std::shared_ptr<NativeRdb::AbsSharedResultSet> result, std::shared_ptr<FileInfo> &fileInfo);
     static int GetFileInfoFromResult(std::shared_ptr<NativeRdb::AbsSharedResultSet> result,
-        MessageParcel &reply, int res);
+        std::vector<std::shared_ptr<FileInfo>> &fileList);
     static bool InitMediaTableColIndexMap(std::shared_ptr<NativeRdb::AbsSharedResultSet> result);
     static bool InitHelper(sptr<IRemoteObject> obj);
 private: