From b2ebcabdd5c56fa734a6eaf8eacc8b8c5c3db9a6 Mon Sep 17 00:00:00 2001
From: Dean Herbert <pe@ppy.sh>
Date: Sat, 29 Jan 2022 13:27:19 +0900
Subject: [PATCH] Fix potential crash during stable install migration due to
 multiple configuration files

Apparently this can be a thing on windows.

Closes https://github.com/ppy/osu/discussions/16689.
---
 osu.Game/IO/StableStorage.cs | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/osu.Game/IO/StableStorage.cs b/osu.Game/IO/StableStorage.cs
index f5a8c4dc9e..84b7da91fc 100644
--- a/osu.Game/IO/StableStorage.cs
+++ b/osu.Game/IO/StableStorage.cs
@@ -34,11 +34,17 @@ namespace osu.Game.IO
 
         private string locateSongsDirectory()
         {
-            string configFile = GetFiles(".", $"osu!.{Environment.UserName}.cfg").SingleOrDefault();
+            var configurationFiles = GetFiles(".", $"osu!.{Environment.UserName}.cfg");
 
-            if (configFile != null)
+            // GetFiles returns case insensitive results, so multiple files could exist.
+            // Prefer a case-correct match, but fallback to any available.
+            string usableConfigFile =
+                configurationFiles.FirstOrDefault(f => f.Contains(Environment.UserName, StringComparison.Ordinal))
+                ?? configurationFiles.FirstOrDefault();
+
+            if (usableConfigFile != null)
             {
-                using (var stream = GetStream(configFile))
+                using (var stream = GetStream(usableConfigFile))
                 using (var textReader = new StreamReader(stream))
                 {
                     string line;