Fix race condition when deleting VM bundle

Array should not change between check and use.
yep · Nov 29, 2024 · 59c7a4d · 59c7a4d
1 parent d534f0b
commit 59c7a4d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/virtualOS.xcodeproj/project.pbxproj b/virtualOS.xcodeproj/project.pbxproj
@@ -268,7 +268,7 @@
 				"CODE_SIGN_IDENTITY[sdk=macosx*]" = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
 				COMBINE_HIDPI_IMAGES = YES;
-				CURRENT_PROJECT_VERSION = 17;
+				CURRENT_PROJECT_VERSION = 18;
 				DEAD_CODE_STRIPPING = YES;
 				DEVELOPMENT_TEAM = 2AD47BTDQ6;
 				ENABLE_HARDENED_RUNTIME = YES;
@@ -301,7 +301,7 @@
 				"CODE_SIGN_IDENTITY[sdk=macosx*]" = "Apple Development";
 				CODE_SIGN_STYLE = Automatic;
 				COMBINE_HIDPI_IMAGES = YES;
-				CURRENT_PROJECT_VERSION = 17;
+				CURRENT_PROJECT_VERSION = 18;
 				DEAD_CODE_STRIPPING = YES;
 				DEVELOPMENT_TEAM = 2AD47BTDQ6;
 				ENABLE_HARDENED_RUNTIME = YES;

diff --git a/virtualOS/Model/FileModel.swift b/virtualOS/Model/FileModel.swift
@@ -30,7 +30,6 @@ struct FileModel {
 
     func getRestoreImages() -> [String] {
         var result: [String] = []
-
         if let urls = try? FileManager.default.contentsOfDirectory(at: URL.baseURL, includingPropertiesForKeys: nil, options: []) {
             for url in urls {
                 if url.lastPathComponent.hasSuffix("ipsw") {

diff --git a/virtualOS/Model/TableViewDataSource.swift b/virtualOS/Model/TableViewDataSource.swift
@@ -23,8 +23,9 @@ final class TableViewDataSource: NSObject, NSTableViewDataSource {
     }
 
     func vmBundle(forRow row: Int) -> VMBundle? {
-        if row < fileModel.getVMBundles().count {
-            return fileModel.getVMBundles()[row]
+        let bundles = fileModel.getVMBundles()
+        if row < bundles.count {
+            return bundles[row]
         } else {
             return nil
         }