Merge pull request #1536 from felix91gr/patch-2

millenomi · web-flow · commit 2733f62c81b7 · 2018-04-25T16:30:41.000-07:00
Fix bug of "/" crashing `NSPathUtilities.resolvingSymlinksInPath`
diff --git a/Foundation/NSPathUtilities.swift b/Foundation/NSPathUtilities.swift
@@ -317,7 +317,7 @@ public extension NSString {
         }
         
         // TODO: pathComponents keeps final path separator if any. Check that logic.
-        if components.last == "/" {
+        if components.last == "/" && components.count > 1 {
             components.removeLast()
         }
         

Original file line number	Diff line number	Diff line change
`@@ -317,7 +317,7 @@ public extension NSString {`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`// TODO: pathComponents keeps final path separator if any. Check that logic.`
`320`		`- if components.last == "/" {`
	`320`	`+ if components.last == "/" && components.count > 1 {`
`321`	`321`	`components.removeLast()`
`322`	`322`	`}`
`323`	`323`