Avoid hashing headers twice in verify_justification (paritytech#973)

serban300 · Apr 8, 2024 · ec33400 · ec33400
1 parent 32d9fca
commit ec33400
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/bridges/primitives/header-chain/src/justification.rs b/bridges/primitives/header-chain/src/justification.rs
@@ -109,12 +109,8 @@ where
 		}
 	}
 
-	let ancestry_hashes = justification
-		.votes_ancestries
-		.iter()
-		.map(|h: &Header| h.hash())
-		.collect();
-	if visited_hashes != ancestry_hashes {
+	// both iterators are `BTree*` iterators, so have the same order => safe to compare
+	if !visited_hashes.iter().eq(ancestry_chain.ancestry.keys()) {
 		return Err(Error::InvalidPrecommitAncestries);
 	}