Check if lookupDir exists before iterating

std::filesystem::recursive_directory_iterator throws if the directory doesn't exist.
Framstag · Nov 15, 2023 · 1c560ae · 1c560ae
1 parent e749cff
commit 1c560ae
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/libosmscout-client/src/osmscoutclient/MapManager.cpp b/libosmscout-client/src/osmscoutclient/MapManager.cpp
@@ -43,6 +43,11 @@ CancelableFuture<bool> MapManager::LookupDatabases()
     std::vector<std::filesystem::path> databaseFsDirectories;
 
     for (const auto &lookupDir:databaseLookupDirs){
+      if (!std::filesystem::exists(lookupDir) || !std::filesystem::is_directory(lookupDir)) {
+        osmscout::log.Warn() << "Lookup dir" << lookupDir << "doesn't exist";
+        continue;
+      }
+
       for (const auto & fInfo : std::filesystem::recursive_directory_iterator(lookupDir)) {
         auto entryPath = fInfo.path();
         if (fInfo.is_regular_file() && entryPath.has_filename() && entryPath.has_parent_path() && entryPath.filename() == TypeConfig::FILE_TYPES_DAT){