refactor: removed pre-MN_RR logic of validation of CL

The fork MN_RR is active awhile on testnet and mainnet and no more need legacy check

src/evo/cbtx.cpp

@@ -332,7 +332,7 @@ bool CalcCbTxMerkleRootQuorums(const CBlock& block, const CBlockIndex* pindexPre
 }
 
 bool CheckCbTxBestChainlock(const CBlock& block, const CBlockIndex* pindex,
-                            const llmq::CChainLocksHandler& chainlock_handler, BlockValidationState& state, const bool check_clhdiff)
+                            const llmq::CChainLocksHandler& chainlock_handler, BlockValidationState& state)
 {
     if (block.vtx[0]->nType != TRANSACTION_COINBASE) {
         return true;
@@ -353,18 +353,16 @@ bool CheckCbTxBestChainlock(const CBlock& block, const CBlockIndex* pindex,
         return true;
     }
 
-    if (check_clhdiff) {
-        auto prevBlockCoinbaseChainlock = GetNonNullCoinbaseChainlock(pindex->pprev);
-        // If std::optional prevBlockCoinbaseChainlock is empty, then up to the previous block, coinbase Chainlock is null.
-        if (prevBlockCoinbaseChainlock.has_value()) {
-            // Previous block Coinbase has a non-null Chainlock: current block's Chainlock must be non-null and at least as new as the previous one
-            if (!opt_cbTx->bestCLSignature.IsValid()) {
-                // IsNull() doesn't exist for CBLSSignature: we assume that a non valid BLS sig is null
-                return state.Invalid(BlockValidationResult::BLOCK_CONSENSUS, "bad-cbtx-null-clsig");
-            }
-            if (opt_cbTx->bestCLHeightDiff > prevBlockCoinbaseChainlock.value().second + 1) {
-                return state.Invalid(BlockValidationResult::BLOCK_CONSENSUS, "bad-cbtx-older-clsig");
-            }
+    const auto prevBlockCoinbaseChainlock = GetNonNullCoinbaseChainlock(pindex->pprev);
+    // If std::optional prevBlockCoinbaseChainlock is empty, then up to the previous block, coinbase Chainlock is null.
+    if (prevBlockCoinbaseChainlock.has_value()) {
+        // Previous block Coinbase has a non-null Chainlock: current block's Chainlock must be non-null and at least as new as the previous one
+        if (!opt_cbTx->bestCLSignature.IsValid()) {
+            // IsNull() doesn't exist for CBLSSignature: we assume that a non valid BLS sig is null
+            return state.Invalid(BlockValidationResult::BLOCK_CONSENSUS, "bad-cbtx-null-clsig");
+        }
+        if (opt_cbTx->bestCLHeightDiff > prevBlockCoinbaseChainlock.value().second + 1) {
+            return state.Invalid(BlockValidationResult::BLOCK_CONSENSUS, "bad-cbtx-older-clsig");
         }
     }
 

src/evo/cbtx.h

@@ -96,7 +96,7 @@ bool CalcCbTxMerkleRootQuorums(const CBlock& block, const CBlockIndex* pindexPre
                                BlockValidationState& state);
 
 bool CheckCbTxBestChainlock(const CBlock& block, const CBlockIndex* pindexPrev,
-                            const llmq::CChainLocksHandler& chainlock_handler, BlockValidationState& state, const bool check_clhdiff);
+                            const llmq::CChainLocksHandler& chainlock_handler, BlockValidationState& state);
 bool CalcCbTxBestChainlock(const llmq::CChainLocksHandler& chainlock_handler, const CBlockIndex* pindexPrev,
                            uint32_t& bestCLHeightDiff, CBLSSignature& bestCLSignature);
 

src/evo/specialtxman.cpp

@@ -188,7 +188,7 @@ bool CSpecialTxProcessor::ProcessSpecialTxsInBlock(const CBlock& block, const CB
         nTimeMerkle += nTime5 - nTime4;
         LogPrint(BCLog::BENCHMARK, "        - CheckCbTxMerkleRoots: %.2fms [%.2fs]\n", 0.001 * (nTime5 - nTime4), nTimeMerkle * 0.000001);
 
-        if (fCheckCbTxMerkleRoots && !CheckCbTxBestChainlock(block, pindex, m_clhandler, state, DeploymentActiveAt(*pindex, m_consensus_params, Consensus::DEPLOYMENT_MN_RR))) {
+        if (fCheckCbTxMerkleRoots && !CheckCbTxBestChainlock(block, pindex, m_clhandler, state)) {
             // pass the state returned by the function above
             return false;
         }

test/functional/feature_llmq_chainlocks.py

@@ -20,7 +20,7 @@
 
 class LLMQChainLocksTest(DashTestFramework):
     def set_test_params(self):
-        self.set_dash_test_params(5, 4, [["-testactivationheight=mn_rr@1100"]] * 5)
+        self.set_dash_test_params(5, 4)
 
     def run_test(self):
         # Connect all nodes to node1 so that we always have the whole network connected
@@ -243,14 +243,8 @@ def run_test(self):
             assert_equal(tip_1['cbTx']['bestCLSignature'], tip_0['cbTx']['bestCLSignature'])
             assert_equal(tip_1['cbTx']['bestCLHeightDiff'], tip_0['cbTx']['bestCLHeightDiff'] + 1)
 
-        self.log.info("Test that bestCLHeightDiff conditions are relaxed before mn_rr")
-        self.test_bestCLHeightDiff(False)
-
-        self.activate_mn_rr(expected_activation_height=1100)
-        self.log.info("Activated mn_rr at height:" + str(self.nodes[0].getblockcount()))
-
-        self.log.info("Test that bestCLHeightDiff conditions are stricter after mn_rr")
-        self.test_bestCLHeightDiff(True)
+        self.log.info("Test bestCLHeightDiff restrictions")
+        self.test_bestCLHeightDiff()
 
     def create_chained_txs(self, node, amount):
         txid = node.sendtoaddress(node.getnewaddress(), amount)
@@ -293,11 +287,10 @@ def test_coinbase_best_cl(self, node, expected_cl_in_cb=True, expected_null_cl=F
         else:
             assert "bestCLHeightDiff" not in cbtx and "bestCLSignature" not in cbtx
 
-    def test_bestCLHeightDiff(self, mn_rr_active):
+    def test_bestCLHeightDiff(self):
         # We need 2 blocks we can grab clsigs from
         for _ in range(2):
             self.wait_for_chainlocked_block_all_nodes(self.generate(self.nodes[0], 1, sync_fun=self.no_op)[0])
-        assert_equal(softfork_active(self.nodes[1], "mn_rr"), mn_rr_active)
         tip1_hash = self.nodes[1].getbestblockhash()
 
         self.isolate_node(1)
@@ -339,7 +332,7 @@ def test_bestCLHeightDiff(self, mn_rr_active):
         mal_block.hashMerkleRoot = mal_block.calc_merkle_root()
         mal_block.solve()
         result = self.nodes[1].submitblock(mal_block.serialize().hex())
-        assert_equal(result, "bad-cbtx-older-clsig" if mn_rr_active else "bad-cbtx-invalid-clsig")
+        assert_equal(result, "bad-cbtx-older-clsig")
         assert_equal(self.nodes[1].getbestblockhash(), tip1_hash)
 
         # Update the sig too and it should pass now when mn_rr is not active and fail otherwise
@@ -350,12 +343,8 @@ def test_bestCLHeightDiff(self, mn_rr_active):
         mal_block.hashMerkleRoot = mal_block.calc_merkle_root()
         mal_block.solve()
         result = self.nodes[1].submitblock(mal_block.serialize().hex())
-        if mn_rr_active:
-            assert_equal(result, "bad-cbtx-older-clsig")
-            assert_equal(self.nodes[1].getbestblockhash(), tip1_hash)
-        else:
-            assert_equal(result, None)
-            assert not self.nodes[1].getbestblockhash() == tip1_hash
+        assert_equal(result, "bad-cbtx-older-clsig")
+        assert_equal(self.nodes[1].getbestblockhash(), tip1_hash)
 
         self.reconnect_isolated_node(1, 0)
         self.sync_all()