From c1284940e1da334498ca471952b2f8a4c8663033 Mon Sep 17 00:00:00 2001
From: Salman Ahmed <frenzibyte@gmail.com>
Date: Tue, 22 Jun 2021 10:49:37 +0300
Subject: [PATCH] Fix potentially providing the same skin instance twice in
 `AllSources`

---
 osu.Game/Skinning/SkinManager.cs | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/osu.Game/Skinning/SkinManager.cs b/osu.Game/Skinning/SkinManager.cs
index e220cad42a..3234cca0ac 100644
--- a/osu.Game/Skinning/SkinManager.cs
+++ b/osu.Game/Skinning/SkinManager.cs
@@ -251,10 +251,11 @@ namespace osu.Game.Skinning
             {
                 yield return CurrentSkin.Value;
 
-                if (CurrentSkin.Value is LegacySkin)
+                if (CurrentSkin.Value is LegacySkin && CurrentSkin.Value != DefaultLegacySkin)
                     yield return DefaultLegacySkin;
 
-                yield return DefaultSkin;
+                if (CurrentSkin.Value != DefaultSkin)
+                    yield return DefaultSkin;
             }
         }