Assign ioError one time only.

Infomaniak · May 2, 2024 · 1408856 · 1408856
1 parent 728ee7b
commit 1408856
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/libcommonserver/utility/utility.cpp b/src/libcommonserver/utility/utility.cpp
@@ -780,14 +780,14 @@ bool Utility::checkIfDirEntryIsManaged(std::filesystem::recursive_directory_iter
     ioError = IoErrorSuccess;
 
     ItemType itemType;
-    if (!IoHelper::getItemType(dirIt->path(), itemType)) {
-        ioError = itemType.ioError;
+    bool result = IoHelper::getItemType(dirIt->path(), itemType);
+    ioError = itemType.ioError;
+    if (!result) {
         LOGW_WARN(logger(), L"Error in IoHelper::getItemType: " << Utility::formatIoError(dirIt->path(), ioError).c_str());
         return false;
     }
 
     if (itemType.ioError == IoErrorNoSuchFileOrDirectory || itemType.ioError == IoErrorAccessDenied) {
-        ioError = itemType.ioError;
         LOGW_DEBUG(logger(), L"Error in IoHelper::getItemType: " << formatIoError(dirIt->path(), ioError).c_str());
         return true;
     }