From 9249e6089ec4e2eb63f0896961f04d9dbe14651a Mon Sep 17 00:00:00 2001 From: marcofleon Date: Wed, 4 Mar 2026 19:15:34 +0000 Subject: [PATCH] validation: remove LoadChainTip call from ActivateSnapshot This call is a no-op. PopulateAndValidateSnapshot already sets both the chain tip and the coins cache best block to the snapshot block, so LoadChainTip always hits the early return when it finds that the two match (tip->GetBlockHash() == coins_cache.GetBestBlock()). --- src/validation.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/validation.cpp b/src/validation.cpp index 1285edc261b..3ed46c1ac3e 100644 --- a/src/validation.cpp +++ b/src/validation.cpp @@ -5741,8 +5741,6 @@ util::Result ChainstateManager::ActivateSnapshot( } Chainstate& chainstate{AddChainstate(std::move(snapshot_chainstate))}; - const bool chaintip_loaded{chainstate.LoadChainTip()}; - assert(chaintip_loaded); m_blockman.m_snapshot_height = Assert(chainstate.SnapshotBase())->nHeight; LogInfo("[snapshot] successfully activated snapshot %s", base_blockhash.ToString());