no need to hash it again (#13261)

prysmaticlabs · Dec 5, 2023 · 705e98e · 705e98e
1 parent ce23443
commit 705e98e
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/beacon-chain/operations/attestations/kv/unaggregated.go b/beacon-chain/operations/attestations/kv/unaggregated.go
@@ -121,15 +121,11 @@ func (c *AttCaches) DeleteSeenUnaggregatedAttestations() (int, error) {
 	defer c.unAggregateAttLock.Unlock()
 
 	count := 0
-	for _, att := range c.unAggregatedAtt {
+	for r, att := range c.unAggregatedAtt {
 		if att == nil || helpers.IsAggregated(att) {
 			continue
 		}
 		if seen, err := c.hasSeenBit(att); err == nil && seen {
-			r, err := hashFn(att)
-			if err != nil {
-				return count, errors.Wrap(err, "could not tree hash attestation")
-			}
 			delete(c.unAggregatedAtt, r)
 			count++
 		}