Fix Issue #81: Check return value of recursive call

Author: jensenak

parser.go

@@ -122,7 +122,6 @@ func searchKeys(data []byte, keys ...string) int {
 	}
 
 	var stackbuf [unescapeStackBufSize]byte // stack-allocated array for allocation-free unescaping of small strings
-
 	for i < ln {
 		switch data[i] {
 		case '"':
@@ -195,7 +194,11 @@ func searchKeys(data []byte, keys ...string) int {
 				if valueFound == nil {
 					return -1
 				} else {
-					return i + valueOffset + searchKeys(valueFound, keys[level+1:]...)
+					subIndex := searchKeys(valueFound, keys[level+1:]...)
+					if subIndex < 0 {
+						return -1
+					}
+					return i + valueOffset + subIndex
 				}
 			} else {
 				// Do not search for keys inside arrays

parser_test.go

@@ -357,7 +357,19 @@ var getTests = []GetTest{
 		path:    []string{"b"},
 		isFound: false,
 	},
-
+	{ // Issue #81
+		desc:	`missing key in object in array`,
+		json: 	`{"p":{"a":[{"u":"abc","t":"th"}]}}`,
+		path: 	[]string{"p", "a", "[0]", "x"},
+		isFound: false,
+	},
+	{ // Issue #81 counter test
+		desc:	`existing key in object in array`,
+		json: 	`{"p":{"a":[{"u":"abc","t":"th"}]}}`,
+		path: 	[]string{"p", "a", "[0]", "u"},
+		isFound: true,
+		data:	"abc",
+	},
 	{ // This test returns not found instead of a parse error, as checking for the malformed JSON would reduce performance
 		desc:    "malformed key (followed by comma followed by colon)",
 		json:    `{"a",:1}`,
@@ -620,7 +632,6 @@ func runGetTests(t *testing.T, testKind string, tests []GetTest, runner func(Get
 		if activeTest != "" && test.desc != activeTest {
 			continue
 		}
-
 		fmt.Println("Running:", test.desc)
 
 		value, dataType, err := runner(test)