diff --git a/services/diffpatch/bzip2/lz4_adapter.h b/services/diffpatch/bzip2/lz4_adapter.h
index 51c1cfac469c7ea23cabfd4aba63e3f9d7a7d98c..f6b287b6f7c4def7f5b69881e194ae3fe0b66df5 100644
--- a/services/diffpatch/bzip2/lz4_adapter.h
+++ b/services/diffpatch/bzip2/lz4_adapter.h
@@ -44,7 +44,7 @@ protected:
     int32_t compressionLevel_ { 0 };
     int32_t blockIndependence_ { 0 };
     int32_t contentChecksumFlag_ { 0 };
-    int32_t blockSizeID_ { 0 };
+    uint32_t blockSizeID_ { 0 };
     int32_t autoFlush_ {1};
 };
 
@@ -85,4 +85,4 @@ private:
     int32_t CompressData(const BlockBuffer &srcData) override;
 };
 } // namespace updatepatch
-#endif // LZ4_ADAPTER_H
\ No newline at end of file
+#endif // LZ4_ADAPTER_H
diff --git a/services/script/script_interpreter/script_expression.cpp b/services/script/script_interpreter/script_expression.cpp
index f9d31943f56058b5b14a73089ac3c74f3e7c6446..d3f6293b8575882990c543fb41f9d9808d5c10cf 100644
--- a/services/script/script_interpreter/script_expression.cpp
+++ b/services/script/script_interpreter/script_expression.cpp
@@ -128,7 +128,7 @@ UScriptValuePtr BinaryExpression::Execute(ScriptInterpreter &inter, UScriptConte
         left = left_->Execute(inter, local);
     }
 
-    if (action_ == OR_OPERATOR && left->IsTrue()) {
+    if (action_ == OR_OPERATOR && left_ != nullptr && left->IsTrue()) {
         INTERPRETER_LOGE(inter, local, "BinaryExpression::Execute left:%s %s",
             UScriptValue::ScriptToString(left).c_str(), opStr[action_].c_str());
         return std::make_shared<IntegerValue>(1);
diff --git a/services/ui/updater_ui.cpp b/services/ui/updater_ui.cpp
index 8118029cd9a3acbf25cb9571238f20969bd06d01..a69029374b812bff5cfa989dd0ac570539958c43 100644
--- a/services/ui/updater_ui.cpp
+++ b/services/ui/updater_ui.cpp
@@ -449,7 +449,7 @@ void UpdaterUiInit()
     HdfInit();
 }
 
-void DeleteView()
+static void DeleteOtherView()
 {
     if (g_updateInfoLabel != nullptr) {
         delete g_updateInfoLabel;
@@ -487,6 +487,11 @@ void DeleteView()
         delete g_logResultLabel;
         g_logResultLabel = nullptr;
     }
+}
+
+void DeleteView()
+{
+    DeleteOtherView();
     if (g_textLabel0 != nullptr) {
         delete g_textLabel0;
         g_textLabel0 = nullptr;
@@ -516,4 +521,4 @@ void DeleteView()
         g_sfDev = nullptr;
     }
 }
-} // namespace updater
+} // namespace updater
\ No newline at end of file
diff --git a/services/updater.cpp b/services/updater.cpp
index a1534384da92ace0b1e88f8176871c61b46491a8..43fb6b6fc99d53782bc9c787a6a63e71d049fa59 100644
--- a/services/updater.cpp
+++ b/services/updater.cpp
@@ -116,9 +116,9 @@ int UpdatePreProcess(PkgManager::PkgManagerPtr pkgManager, const std::string &pa
     return ret;
 }
 
-static UpdaterStatus IsSpaceCapacitySufficient(PkgManager::PkgManagerPtr pkgManager,
-    const std::string &packagePath)
+static UpdaterStatus IsSpaceCapacitySufficient(const std::string &packagePath)
 {
+    PkgManager::PkgManagerPtr pkgManager = hpackage::PkgManager::CreatePackageInstance();
     UPDATER_ERROR_CHECK(pkgManager != nullptr, "pkgManager is nullptr", return UPDATE_CORRUPT);
     std::vector<std::string> fileIds;
     int ret = pkgManager->LoadPackageWithoutUnPack(packagePath, fileIds);
@@ -184,7 +184,7 @@ UpdaterStatus DoInstallUpdaterPackage(PkgManager::PkgManagerPtr pkgManager, cons
     if (retryCount > 0) {
         LOG(INFO) << "Retry for " << retryCount << " time(s)";
     } else {
-        UpdaterStatus ret = IsSpaceCapacitySufficient(pkgManager, packagePath);
+        UpdaterStatus ret = IsSpaceCapacitySufficient(packagePath);
         // Only handle UPATE_ERROR and UPDATE_SUCCESS here.
         // If it returns UPDATE_CORRUPT, which means something wrong with package manager.
         // Let package verify handle this.