Add a check to throw an error is a nil pointer is passed to unmarshal…

…OrCopy (#5334)
hypermodeinc · Apr 30, 2020 · 63a5e3a · 63a5e3a
1 parent 56846ae
commit 63a5e3a
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/posting/mvcc.go b/posting/mvcc.go
@@ -229,6 +229,11 @@ func (txn *Txn) CommitToDisk(writer *TxnWriter, commitTs uint64) error {
 }
 
 func unmarshalOrCopy(plist *pb.PostingList, item *badger.Item) error {
+	if plist == nil {
+		return errors.Errorf("cannot unmarshal value to a nil posting list of key %s",
+			hex.Dump(item.Key()))
+	}
+
 	return item.Value(func(val []byte) error {
 		if len(val) == 0 {
 			// empty pl