From c30a9b47e73d0339d5f0092483fa360403786c12 Mon Sep 17 00:00:00 2001 From: Pierre Krieger Date: Tue, 22 Mar 2022 19:40:12 +0100 Subject: [PATCH 1/2] Don't remove source knows block after verification --- src/sync/all_forks.rs | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/sync/all_forks.rs b/src/sync/all_forks.rs index 1bc97cd4cb..ef66fa24ec 100644 --- a/src/sync/all_forks.rs +++ b/src/sync/all_forks.rs @@ -1584,10 +1584,6 @@ impl HeaderVerify { // Block is valid! // Remove the block from `pending_blocks`. - self.parent.inner.blocks.remove_sources_known_block( - self.block_to_verify.block_number, - &self.block_to_verify.block_hash, - ); let pending_block = self.parent.inner.blocks.remove_unverified_block( self.block_to_verify.block_number, &self.block_to_verify.block_hash, From 1e0ea3c36043abcbeb09c3a775b68fbed3f195ee Mon Sep 17 00:00:00 2001 From: Pierre Krieger Date: Tue, 22 Mar 2022 19:44:36 +0100 Subject: [PATCH 2/2] Changelog --- bin/wasm-node/CHANGELOG.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/bin/wasm-node/CHANGELOG.md b/bin/wasm-node/CHANGELOG.md index 6204c6b4f4..71df57f090 100644 --- a/bin/wasm-node/CHANGELOG.md +++ b/bin/wasm-node/CHANGELOG.md @@ -2,6 +2,10 @@ ## Unreleased +### Fixed + +- Fix regression introduced in version 0.6.5 where we erroneously removed entries in the mapping of which peer knows which blocks, leading to failures to request data. ([#2168](https://github.com/paritytech/smoldot/pull/2168)) + ## 0.6.7 - 2022-03-22 ### Changed