From c84988a2ecef08d1c5aa309a28d7f2f2757372f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= Date: Tue, 10 Jun 2025 12:32:03 +0200 Subject: [PATCH] Do not attempt to add displayed score to list of sorted scores more than once This is a very dodgy fix, but it fixes an edge case that has so far - to my knowledge - not been reported by users in the wild, only by me trying to break things, so my hope is that we can do this and move on for now. --- osu.Game/Screens/Ranking/SoloResultsScreen.cs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/osu.Game/Screens/Ranking/SoloResultsScreen.cs b/osu.Game/Screens/Ranking/SoloResultsScreen.cs index 7d57bc80aa..1d09654063 100644 --- a/osu.Game/Screens/Ranking/SoloResultsScreen.cs +++ b/osu.Game/Screens/Ranking/SoloResultsScreen.cs @@ -74,6 +74,12 @@ namespace osu.Game.Screens.Ranking // this simplifies handling later. if (clonedScore.Equals(Score) || clonedScore.MatchesOnlineID(Score)) { + // this is a precautionary guard that prevents `Score` from appearing multiple times in the list. + // that can occur in rare cases wherein two local scores have the same online ID but different replay contents + // (this is possible e.g. in cases of client-side vs server-side recorded replays, see https://github.com/ppy/osu-server-spectator/issues/193) + if (sortedScores.Contains(Score)) + continue; + Score.Position = clonedScore.Position; sortedScores.Add(Score); }