Merge pull request #1201 from wakatime/bugfix/windows-split-drive

Fix panic for splitDrive

Author: alanhamlett

pkg/windows/windows.go

@@ -259,7 +259,7 @@ func parseNetUseColumns(line string) (netUseColumns, error) {
 
 // splitDrive splits a filepath into the drive letter and the path.
 func splitDrive(fp string) (string, string) {
-	if fp[1:2] != ":" || !unicode.IsLetter(rune(fp[0])) {
+	if len(fp) < 2 || fp[1:2] != ":" || !unicode.IsLetter(rune(fp[0])) {
 		return "", fp
 	}
 

pkg/windows/windows_internal_test.go

@@ -182,6 +182,11 @@ func TestSplitDrive(t *testing.T) {
 			ExpectedDriveLetter: ``,
 			ExpectedPath:        `_:\\remotepc\share`,
 		},
+		"one character drive": {
+			Filepath:            `A`,
+			ExpectedDriveLetter: "",
+			ExpectedPath:        `A`,
+		},
 	}
 
 	for name, test := range tests {