KSP-CKAN · Feb 27, 2018 · Mar 4, 2018
Showing with 10 additions and 6 deletions.

+1 −0 CHANGELOG.md

+9 −6 Core/KSPPathUtils.cs
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ All notable changes to this project will be documented in this file.
 - [GUI] Fix GUI sort by size (#2311 by: HebaruSan; reviewed: politas)
 - [Core] Don't crash if download_hash isn't set (#2313 by: HebaruSan; reviewed: politas)
 - [GUI] Fix GUI instance name checking (#2316 by: HebaruSan; reviewed: politas)
+- [Core] Fix ArgumentOutOfRangeException when removing files from game root (#2332 by: HebaruSan; reviewed: politas)
 
 ## v1.24.0 (Bruce)
 

diff --git a/Core/KSPPathUtils.cs b/Core/KSPPathUtils.cs
@@ -144,7 +144,7 @@ public static string KSPSteamPath()
                     // Found Steam library
                     if (line.Contains("BaseInstallFolder"))
                     {
-                        
+
                         // This assumes config file is valid, we just skip it if it looks funny.
                         string[] split_line = line.Split('"');
 
@@ -189,7 +189,7 @@ public static string GetLastPathElement(string path)
 
         /// <summary>
         /// Gets the leading path elements. Ex: /a/b/c returns /a/b
-        /// 
+        ///
         /// Returns empty string if there is no leading path. (Eg: "Example.dll" -> "");
         /// </summary>
         /// <returns>The leading path elements.</returns>
@@ -242,10 +242,13 @@ public static string ToRelative(string path, string root)
                     )
                 );
             }
-        
+
             // The +1 here is because root will never have
-            // a trailing slash.
-            return path.Remove(0, root.Length + 1);
+            // a trailing slash. However, if the strings are
+            // the same, it causes an exception.
+            return path.Length > root.Length
+                ? path.Remove(0, root.Length + 1)
+                : "";
         }
 
         /// <summary>
@@ -285,4 +288,4 @@ public static string ToAbsolute(string path, string root)
             return NormalizePath(Path.Combine(root, path));
         }
     }
-}
+}