Merge pull request #493 from gfrey/fix_fieldpath_parser

Fix infinite loop in fieldPath parser for single-letter field names

Author: a-hilaly

Diff for: pkg/graph/fieldpath/parser.go

@@ -73,8 +73,7 @@ func (p *parser) parse() ([]Segment, error) {
 				return nil, err
 			}
 			segments = append(segments, Segment{Name: field, Index: -1})
-
-		} else if p.pos+1 < p.len && p.input[p.pos] != '[' {
+		} else if p.pos < p.len && p.input[p.pos] != '[' {
 			// Parse unquoted field until we hit a [ or .
 			field, err := p.parseUnquotedField()
 			if err != nil {

Diff for: pkg/graph/fieldpath/parser_test.go

@@ -25,6 +25,14 @@ func TestParse(t *testing.T) {
 		want    []Segment
 		wantErr bool
 	}{
+		{
+			name: "simple single letter path",
+			path: "data.A",
+			want: []Segment{
+				{Name: "data", Index: -1},
+				{Name: "A", Index: -1},
+			},
+		},
 		{
 			name: "simple path",
 			path: "spec.containers",