[1.0.4 -> main] Fix fetch block with no block log

libraries/chain/controller.cpp

@@ -1199,14 +1199,14 @@ struct controller_impl {
 
    signed_block_ptr fork_db_fetch_block_by_id( const block_id_type& id ) const {
       return fork_db_.apply<signed_block_ptr>([&](const auto& fork_db) {
-         auto bsp = fork_db.get_block(id);
+         auto bsp = fork_db.get_block(id, include_root_t::yes);
          return bsp ? bsp->block : signed_block_ptr{};
       });
    }
 
    signed_block_ptr fork_db_fetch_block_on_best_branch_by_num(uint32_t block_num) const {
       return fork_db_.apply<signed_block_ptr>([&](const auto& fork_db) {
-         auto bsp = fork_db.search_on_head_branch(block_num);
+         auto bsp = fork_db.search_on_head_branch(block_num, include_root_t::yes);
          if (bsp) return bsp->block;
          return signed_block_ptr{};
       });

libraries/chain/fork_database.cpp

@@ -459,8 +459,8 @@ namespace eosio::chain {
    BSP fork_database_impl<BSP>::search_on_branch_impl( const block_id_type& h, uint32_t block_num, include_root_t include_root ) const {
       if (!root)
          return {};
-      if( include_root == include_root_t::yes && root->id() == h && root->block_num() == block_num ) {
-         return root;
+      if( include_root == include_root_t::yes && root->block_num() == block_num ) {
+         return root; // root is root of every branch, no need to check h
       }
       if (block_num <= root->block_num())
          return {};
@@ -593,7 +593,7 @@ namespace eosio::chain {
    template<class BSP>
    BSP fork_database_impl<BSP>::get_block_impl(const block_id_type& id,
                                                include_root_t include_root /* = include_root_t::no */) const {
-      if( include_root == include_root_t::yes && root->id() == id ) {
+      if( include_root == include_root_t::yes && root && root->id() == id ) {
          return root;
       }
       auto itr = index.find( id );

plugins/producer_plugin/producer_plugin.cpp

@@ -1609,14 +1609,14 @@ void producer_plugin_impl::plugin_startup() {
       const auto fork_db_root     = chain.fetch_block_by_number(fork_db_root_num);
       if (fork_db_root) {
          on_irreversible_block(fork_db_root);
+
+         if (!_is_savanna_active && irreversible_mode() && chain_plug->accept_transactions()) {
+            wlog("Legacy consensus active. Accepting speculative transaction execution not recommended in read-mode=irreversible");
+         }
       } else {
          _irreversible_block_time = fc::time_point::maximum();
       }
 
-      if (!_is_savanna_active && irreversible_mode() && chain_plug->accept_transactions()) {
-         wlog("Legacy consensus active. Accepting speculative transaction execution not recommended in read-mode=irreversible");
-      }
-
       if (is_configured_producer()) {
          ilog("Launching block production for ${n} producers at ${time}.", ("n", _producers.size())("time", fc::time_point::now()));
 

unittests/database_tests.cpp

@@ -84,6 +84,57 @@ BOOST_AUTO_TEST_SUITE(database_tests)
          // Check the latest head block match
          BOOST_TEST(test.fetch_block_by_number(test.head().block_num())->calculate_id() ==
                     test.head().id());
+
+         // Verify LIB can be found
+         const auto lib_num = test.last_irreversible_block_num();
+         auto lib           = test.fetch_block_by_number(lib_num);
+         BOOST_REQUIRE(lib);
+         auto lib_id = lib->calculate_id();
+         BOOST_TEST(lib_num == lib->block_num());
+         lib = test.fetch_block_by_id(lib_id);
+         BOOST_REQUIRE(lib);
+         BOOST_TEST(lib->calculate_id() == lib_id);
+
+      } FC_LOG_AND_RETHROW()
+   }
+
+   // Test the block fetching methods on database, fetch_bock_by_id, and fetch_block_by_number
+   BOOST_AUTO_TEST_CASE_TEMPLATE( get_blocks_no_block_log, T, validating_testers ) {
+      try {
+         fc::temp_directory tempdir;
+
+         constexpr bool use_genesis = true;
+         T test(
+            tempdir,
+            [&](controller::config& cfg) {
+               cfg.blog = eosio::chain::empty_blocklog_config{};
+            },
+            use_genesis
+         );
+
+         // Ensure that future block doesn't exist
+         const auto nonexisting_future_block_num = test.head().block_num() + 1;
+         BOOST_TEST(test.fetch_block_by_number(nonexisting_future_block_num) == nullptr);
+         BOOST_TEST(test.fetch_block_by_id(sha256::hash("xx")) == nullptr);
+
+         test.produce_block();
+
+         // Previous nonexisting future block should exist now
+         BOOST_CHECK_NO_THROW(test.fetch_block_by_number(nonexisting_future_block_num));
+         // Check the latest head block match
+         BOOST_TEST(test.fetch_block_by_number(test.head().block_num())->calculate_id() == test.head().id());
+         BOOST_TEST(test.fetch_block_by_id(test.head().id())->calculate_id() == test.head().id());
+
+         // Verify LIB can be found
+         const auto lib_num = test.last_irreversible_block_num();
+         auto lib           = test.fetch_block_by_number(lib_num);
+         BOOST_REQUIRE(lib);
+         auto lib_id = lib->calculate_id();
+         BOOST_TEST(lib_num == lib->block_num());
+         lib = test.fetch_block_by_id(lib_id);
+         BOOST_REQUIRE(lib);
+         BOOST_TEST(lib->calculate_id() == lib_id);
+
       } FC_LOG_AND_RETHROW()
    }