diff --git a/services/package/pkg_manager/pkg_managerImpl.cpp b/services/package/pkg_manager/pkg_managerImpl.cpp
index 04f197a6366fc27a7726435d13dd4de8ecf241b2..f9834dbe898a5a6bf6df756cf32a52ca17904805 100644
--- a/services/package/pkg_manager/pkg_managerImpl.cpp
+++ b/services/package/pkg_manager/pkg_managerImpl.cpp
@@ -306,11 +306,11 @@ int32_t PkgManagerImpl::LoadPackage(const std::string &packagePath, const std::s
     PkgFile::PkgType pkgType = GetPkgTypeByName(packagePath);
     unzipToFile_ = ((pkgType == PkgFile::PKG_TYPE_GZIP) ? true : unzipToFile_);
     if (pkgType == PkgFile::PKG_TYPE_UPGRADE) {
-        if (LoadPackage(packagePath, fileIds, pkgType) != PKG_SUCCESS) {
+        if (int32_t ret = LoadPackage(packagePath, fileIds, pkgType); ret != PKG_SUCCESS) {
             ClearPkgFile();
-            UPDATER_LAST_WORD("LoadPackage failed", packagePath);
+            UPDATER_LAST_WORD("LoadPackage failed", packagePath, ret);
             PKG_LOGE("Parse %s fail ", packagePath.c_str());
-            return PKG_INVALID_FILE;
+            return ret;
         }
     } else if (pkgType != PkgFile::PKG_TYPE_NONE) {
         std::vector<std::string> innerFileNames;
@@ -368,7 +368,7 @@ int32_t PkgManagerImpl::ExtraAndLoadPackage(const std::string &path, const std::
         (void)mkdir(tempPath.c_str(), 0775); // 0775 : rwxrwxr-x
 #endif
     }
-    
+
     // Extract package to file or memory
     if (unzipToFile_ || type == PkgFile::PKG_TYPE_UPGRADE) {
         ret = CreatePkgStream(stream, tempPath + name + ".tmp", info->unpackedSize, PkgStream::PkgStreamType_Write);
diff --git a/services/package/pkg_verify/openssl_util.cpp b/services/package/pkg_verify/openssl_util.cpp
index 2554cc7d6f060ba82294cb05edb2f06814310ab1..ac4b2fa1267788cb44e80a5cb4f01586003043e3 100644
--- a/services/package/pkg_verify/openssl_util.cpp
+++ b/services/package/pkg_verify/openssl_util.cpp
@@ -231,7 +231,6 @@ int32_t CalcSha256Digest(const PkgStreamPtr srcData, const size_t dataLen, std::
         }
         SHA256_Update(&ctx, buffer.buffer, readLen);
         offset += readLen;
-        PKG_LOGI("use new remainLen");
         remainLen -= readLen;
     }
 
diff --git a/services/package/pkg_verify/pkg_verify_util.cpp b/services/package/pkg_verify/pkg_verify_util.cpp
index 004f8b63451b94d92ca56e3ef495788fef6bc85f..b128901b40373516fe746d1e54ce48c29b36bf98 100644
--- a/services/package/pkg_verify/pkg_verify_util.cpp
+++ b/services/package/pkg_verify/pkg_verify_util.cpp
@@ -210,7 +210,7 @@ std::string PkgVerifyUtil::GetPkgTime(const std::string &pkgPath) const
     return fileInfo;
 }
 
-void PkgVerifyUtil::WriteHash(std::vector<uint8_t> &hash, const std::string &pkgPath) const
+void PkgVerifyUtil::RecordHash(std::vector<uint8_t> &hash, const std::string &pkgPath) const
 {
     Updater::UPDATER_INIT_RECORD;
     std::string path = "/data/updater/hash_file";
@@ -220,7 +220,7 @@ void PkgVerifyUtil::WriteHash(std::vector<uint8_t> &hash, const std::string &pkg
             PKG_LOGE("open file failed");
             return;
         }
-        file.write(reinterpret_cast<char *>(hash.data()), sizeof(hash));
+        file.write(reinterpret_cast<char *>(hash.data()), hash.size());
         return;
     }
     std::ifstream file(path, std::ios::in);
@@ -232,8 +232,7 @@ void PkgVerifyUtil::WriteHash(std::vector<uint8_t> &hash, const std::string &pkg
     if (getline(file, lastHash)) {
         std::vector<unsigned char> lastHashVector {};
         lastHashVector.assign(lastHash.begin(), lastHash.end());
-        UPDATER_LAST_WORD(ConvertShaHex(static_cast<std::vector<uint8_t>>(lastHashVector)),
-                          ConvertShaHex(hash), GetPkgTime(pkgPath));
+        UPDATER_LAST_WORD(ConvertShaHex(lastHashVector), ConvertShaHex(hash), GetPkgTime(pkgPath));
     }
 }
 
@@ -266,7 +265,7 @@ int32_t PkgVerifyUtil::HashCheck(const PkgStreamPtr srcData, const size_t dataLe
         UPDATER_LAST_WORD(PKG_INVALID_DIGEST,
                           ConvertShaHex(hash).substr(0, INTERCEPT_HASH_LENGTH),
                           ConvertShaHex(sourceDigest).substr(0, INTERCEPT_HASH_LENGTH), fileInfo);
-        WriteHash(sourceDigest, path);
+        RecordHash(sourceDigest, path);
         return PKG_INVALID_DIGEST;
     }
 
diff --git a/services/package/pkg_verify/pkg_verify_util.h b/services/package/pkg_verify/pkg_verify_util.h
index e4da7ed612841d311390841920c1c23592f6ba35..a13deca82c141eb5cb47b28e30bd13dee6403583 100644
--- a/services/package/pkg_verify/pkg_verify_util.h
+++ b/services/package/pkg_verify/pkg_verify_util.h
@@ -51,7 +51,7 @@ public:
 
     std::string GetPkgTime(const std::string &pkgPath) const;
 
-    void WriteHash(std::vector<uint8_t> &hash, const std::string &pkgPath) const;
+    void RecordHash(std::vector<uint8_t> &hash, const std::string &pkgPath) const;
 private:
     bool isOldSigSupport_ {true};
 };