GMP-809 nil document check on load

danielpcox · web-flow · commit 4112a878cdcc · 2023-05-24T11:44:16.000-04:00
diff --git a/redisearch/document.go b/redisearch/document.go
@@ -105,7 +105,10 @@ func loadDocument(arr []interface{}, idIdx, scoreIdx, payloadIdx, fieldsIdx int)
 	}
 
 	if fieldsIdx > 0 {
-		lst := arr[idIdx+fieldsIdx].([]interface{})
+		lst, ok := arr[idIdx+fieldsIdx].([]interface{})
+		if !ok && arr[idIdx+fieldsIdx] == nil {
+			return doc, fmt.Errorf("document was nil")
+		}
 		doc.loadFields(lst)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,10 @@ func loadDocument(arr []interface{}, idIdx, scoreIdx, payloadIdx, fieldsIdx int)`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`if fieldsIdx > 0 {`
`108`		`- lst := arr[idIdx+fieldsIdx].([]interface{})`
	`108`	`+ lst, ok := arr[idIdx+fieldsIdx].([]interface{})`
	`109`	`+ if !ok && arr[idIdx+fieldsIdx] == nil {`
	`110`	`+ return doc, fmt.Errorf("document was nil")`
	`111`	`+ }`
`109`	`112`	`doc.loadFields(lst)`
`110`	`113`	`}`
`111`	`114`