Merge pull request #114 from Corona-Studio/hotfix

Bugfix: Fix NRE in DefaultLauncherAccountParser.cs
Corona-Studio · Jan 26, 2024 · 4a1422d · 4a1422d
2 parents 16d6c5b + 821f6ce
commit 4a1422d
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/ProjBobcat/ProjBobcat/DefaultComponent/Launch/DefaultLauncherAccountParser.cs b/ProjBobcat/ProjBobcat/DefaultComponent/Launch/DefaultLauncherAccountParser.cs
@@ -143,10 +143,13 @@ public bool AddNewAccount(string uuid, AccountModel account, out Guid? id)
  public bool RemoveAccount(Guid id)
  {
  var result = Find(id);
+
  if (!result.HasValue) return false;
 
  var (key, _) = result.Value;
 
+ if (string.IsNullOrEmpty(key)) return false;
+
  LauncherAccount?.Accounts?.Remove(key);
  Save();