Fix slice bounds errors when editing notes on short entries

src/cli/edit.go

@@ -30,9 +30,7 @@ func EditEntry(targetLocation string, hidePassword bool, field int) {
 	unencryptedEntry := offline.DecryptGPG(targetLocation)
 
 	// ensure slice is long enough for field
-	for len(unencryptedEntry) <= field {
+	unencryptedEntry = offline.EnsureSliceLength(unencryptedEntry, field)
-		unencryptedEntry = append(unencryptedEntry, "")
-	}
 
 	// edit the field
 	switch field {
@@ -48,6 +46,7 @@ func EditEntry(targetLocation string, hidePassword bool, field int) {
 	writeEntryShortcut(targetLocation, unencryptedEntry, hidePassword)
 }
 
+// EditEntryNote edits the note of an entry at targetLocation (user input)
 func EditEntryNote(targetLocation string, hidePassword bool) {
 	// ensure targetLocation exists
 	offline.TargetIsFile(targetLocation, true, 2)
@@ -55,6 +54,10 @@ func EditEntryNote(targetLocation string, hidePassword bool) {
 	// read old entry data
 	unencryptedEntry := offline.DecryptGPG(targetLocation)
 
+	// ensure slice is long enough for note
+	// avoids errors when storing non-note data
+	unencryptedEntry = offline.EnsureSliceLength(unencryptedEntry, 2) // 2 is used because it is the index of URL, the last non-note field
+
 	// store non-note data separately
 	nonNoteData := unencryptedEntry[:3]
 

src/offline/edit.go

@@ -23,3 +23,11 @@ func Rename(oldLocation string, newLocation string) {
 	// TODO If in online mode, check if oldLocation is a directory and rename it on the server
 	os.Exit(0)
 }
+
+// EnsureSliceLength ensures slice is long enough to contain the specified index
+func EnsureSliceLength(slice []string, index int) []string {
+	for len(slice) <= index {
+		slice = append(slice, "")
+	}
+	return slice
+}