Merge pull request #2 from chaos95/master

Handle 'index out of range' errors more gracefully

README.md

@@ -3,6 +3,12 @@ yaml is a lightweight and flexible command-line YAML processor
 
 The aim of the project is to be the [jq](https://github.com/stedolan/jq) or sed of yaml files.
 
+## Install
+[Download latest binary](https://github.com/mikefarah/yaml/releases/latest) or alternatively:
+```
+go get github.com/mikefarah/yaml
+```
+
 ## Features
 - Written in portable go, so you can download a lovely dependency free binary
 - Deep read a yaml file with a given path
@@ -11,11 +17,6 @@ The aim of the project is to be the [jq](https://github.com/stedolan/jq) or sed
 - Convert from json to yaml
 - Convert from yaml to json
 
-[Download latest binary](https://github.com/mikefarah/yaml/releases/latest) or alternatively:
-```
-go get github.com/mikefarah/yaml
-```
-
 ## Read examples
 ```
 yaml r <yaml file> <path>

data_navigator.go

@@ -58,7 +58,7 @@ func recurse(value interface{}, head string, tail []string) interface{} {
 }
 
 func readArray(array []interface{}, head int64, tail []string) interface{} {
-	if head > int64(len(array)) {
+	if head >= int64(len(array)) {
 		return nil
 	}
 

data_navigator_test.go

@@ -83,6 +83,17 @@ b:
 	assertResult(t, nil, readMap(data, "b", []string{"d", "3"}))
 }
 
+func TestReadMap_with_array_out_of_bounds_by_1(t *testing.T) {
+	var data = parseData(`
+---
+b:
+  d:
+    - 3
+    - 4
+`)
+	assertResult(t, nil, readMap(data, "b", []string{"d", "2"}))
+}
+
 func TestReadMap_with_array_splat(t *testing.T) {
 	var data = parseData(`
 e: