Fix static analysis issues

[platform/core/appfw/app-installers.git] / src / common / step / filesystem / step_recover_files.cc

diff --git a/src/common/step/filesystem/step_recover_files.cc b/src/common/step/filesystem/step_recover_files.cc
index 6eb28cb..a4ef3e8 100644 (file)
--- a/src/common/step/filesystem/step_recover_files.cc
+++ b/src/common/step/filesystem/step_recover_files.cc
@@ -102,7 +102,9 @@ Step::Status StepRecoverFiles::RecoveryMountUpdate() {
   bf::path backup_zip_location = GetBackupPathForZipFile(zip_location);
   if (bf::exists(backup_zip_location)) {
     Remove(zip_location);
-    MoveFile(backup_zip_location, zip_location);
+    if (!MoveFile(backup_zip_location, zip_location))
+      LOG(ERROR) << "Failed to move " << backup_zip_location << " to "
+          << zip_location;
   }
 
   // During mount update some files are still backed up separately from zip
@@ -164,4 +166,3 @@ bool StepRecoverFiles::ClearPath(const bf::path& path) {
 
 }  // namespace filesystem
 }  // namespace common_installer
-