Merge pull request #190 from kcalvinalvin/2024-06-24-dont-include-0s

indexers: only include positions that were found
utreexo · Jun 27, 2024 · e166f10 · e166f10 · HalFinneyIsMyHomeBoy · Jul 1, 2024
2 parents 496c40a + e3fa68a
commit e166f10
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/blockchain/indexers/flatutreexoproofindex.go b/blockchain/indexers/flatutreexoproofindex.go
@@ -872,8 +872,10 @@ func (idx *FlatUtreexoProofIndex) GenerateUDataPartial(dels []wire.LeafData, pos
 
 	targets := make([]uint64, len(delHashes))
 	for i, delHash := range delHashes {
-		pos, _ := idx.utreexoState.state.GetLeafPosition(delHash)
-		targets[i] = pos
+		pos, found := idx.utreexoState.state.GetLeafPosition(delHash)
+		if found {
+			targets[i] = pos
+		}
 	}
 
 	ud.AccProof = utreexo.Proof{

diff --git a/blockchain/indexers/utreexoproofindex.go b/blockchain/indexers/utreexoproofindex.go
@@ -446,8 +446,10 @@ func (idx *UtreexoProofIndex) GenerateUDataPartial(dels []wire.LeafData, positio
 
 	targets := make([]uint64, len(delHashes))
 	for i, delHash := range delHashes {
-		pos, _ := idx.utreexoState.state.GetLeafPosition(delHash)
-		targets[i] = pos
+		pos, found := idx.utreexoState.state.GetLeafPosition(delHash)
+		if found {
+			targets[i] = pos
+		}
 	}
 	ud.AccProof = utreexo.Proof{
 		Targets: targets,