fix(memlimit): fix mountinfo validation logic for multiple optional fields

Author: KimMachineGun

memlimit/cgroups.go

@@ -269,11 +269,9 @@ func parseMountInfoLine(line string) (mountInfo, error) {
 		return mountInfo{}, fmt.Errorf("invalid separator")
 	}
 
-	fields1 := strings.Split(fieldss[0], " ")
+	fields1 := strings.SplitN(fieldss[0], " ", 7)
 	if len(fields1) < 6 {
 		return mountInfo{}, fmt.Errorf("not enough fields before separator: %v", fields1)
-	} else if len(fields1) > 7 {
-		return mountInfo{}, fmt.Errorf("too many fields before separator: %v", fields1)
 	} else if len(fields1) == 6 {
 		fields1 = append(fields1, "")
 	}

memlimit/cgroups_test.go

@@ -57,11 +57,6 @@ func TestParseMountInfoLine(t *testing.T) {
 			input:   "36 35 98:0 /mnt1 /mnt2 rw,noatime master:1 - ext3",
 			wantErr: `not enough fields after separator: [ext3]`,
 		},
-		{
-			name:    "too many fields on left side",
-			input:   "36 35 98:0 /mnt1 /mnt2 rw,noatime extra master:1 - ext3 /dev/root rw,errors=continue",
-			wantErr: `too many fields before separator: [36 35 98:0 /mnt1 /mnt2 rw,noatime extra master:1]`,
-		},
 		{
 			name:    "too many fields on right side",
 			input:   "36 35 98:0 /mnt1 /mnt2 rw,noatime master:1 - ext3 /dev/root rw extra",
@@ -82,6 +77,16 @@ func TestParseMountInfoLine(t *testing.T) {
 				SuperOptions:   "rw,relatime",
 			},
 		},
+		{
+			name:  "multiple optional fields on left side (issue #26)",
+			input: "465 34 253:0 / / rw,relatime shared:409 master:1 - xfs /dev/mapper/fedora-root rw,seclabel,attr2,inode64,logbufs=8,logbsize=32k,noquota",
+			want: mountInfo{
+				Root:           "/",
+				MountPoint:     "/",
+				FilesystemType: "xfs",
+				SuperOptions:   "rw,seclabel,attr2,inode64,logbufs=8,logbsize=32k,noquota",
+			},
+		},
 	}
 
 	for _, tt := range tests {