From 122a935306d33f1fc6031ec65e37b6cdb41cbd54 Mon Sep 17 00:00:00 2001 From: Colin LeMahieu Date: Wed, 6 Mar 2024 15:58:23 +0000 Subject: [PATCH] Change block::account to return an std::optional if the block actually contains the field. Fixes up call sites that checked against the old sentinel value 0. Convert many call sites to using ledger::account(block&) when possible. --- nano/core_test/active_transactions.cpp | 4 ++-- nano/core_test/backlog.cpp | 4 +--- nano/core_test/ledger.cpp | 2 +- nano/lib/blocks.cpp | 9 ++++----- nano/lib/blocks.hpp | 9 ++++++--- nano/node/active_transactions.cpp | 2 +- nano/node/blockprocessor.cpp | 2 +- nano/node/bootstrap/bootstrap_bulk_pull.cpp | 2 +- nano/node/bootstrap_ascending/service.cpp | 2 +- nano/node/cli.cpp | 2 +- nano/node/confirmation_height_bounded.cpp | 6 +----- nano/node/confirmation_height_unbounded.cpp | 6 +----- nano/node/ipc/flatbuffers_util.cpp | 4 ++-- nano/node/json_handler.cpp | 10 +++++----- nano/node/node.cpp | 6 +----- nano/node/nodeconfig.cpp | 4 ++-- nano/node/process_live_dispatcher.cpp | 3 +-- nano/node/request_aggregator.cpp | 4 ++-- nano/qt/qt.cpp | 2 +- nano/rpc_test/rpc.cpp | 4 ++-- nano/secure/common.cpp | 10 +++++----- nano/secure/ledger.cpp | 12 +++--------- nano/store/component.cpp | 8 ++++---- 23 files changed, 49 insertions(+), 68 deletions(-) diff --git a/nano/core_test/active_transactions.cpp b/nano/core_test/active_transactions.cpp index 2b1bc50d0..85fc7a1ef 100644 --- a/nano/core_test/active_transactions.cpp +++ b/nano/core_test/active_transactions.cpp @@ -1534,7 +1534,7 @@ TEST (active_transactions, allow_limited_overflow) // Insert the first part of the blocks into normal election scheduler for (auto const & block : blocks1) { - node.scheduler.priority.activate (block->account (), node.store.tx_begin_read ()); + node.scheduler.priority.activate (block->account ().value (), node.store.tx_begin_read ()); } // Ensure number of active elections reaches AEC limit and there is no overfill @@ -1596,7 +1596,7 @@ TEST (active_transactions, allow_limited_overflow_adapt) // Insert the first part of the blocks into normal election scheduler for (auto const & block : blocks1) { - node.scheduler.priority.activate (block->account (), node.store.tx_begin_read ()); + node.scheduler.priority.activate (block->account ().value (), node.store.tx_begin_read ()); } // Ensure number of active elections reaches AEC limit and there is no overfill diff --git a/nano/core_test/backlog.cpp b/nano/core_test/backlog.cpp index fa58f702c..70ccb6b28 100644 --- a/nano/core_test/backlog.cpp +++ b/nano/core_test/backlog.cpp @@ -33,9 +33,7 @@ TEST (backlog, population) auto all_activated = [&] () { nano::lock_guard lock{ mutex }; return std::all_of (blocks.begin (), blocks.end (), [&] (auto const & item) { - auto account = item->account (); - debug_assert (!account.is_zero ()); - return activated.count (account) != 0; + return activated.count (item->account ().value ()) != 0; }); }; ASSERT_TIMELY (5s, all_activated ()); diff --git a/nano/core_test/ledger.cpp b/nano/core_test/ledger.cpp index 9b776a165..2e6115132 100644 --- a/nano/core_test/ledger.cpp +++ b/nano/core_test/ledger.cpp @@ -4052,7 +4052,7 @@ TEST (ledger, epoch_open_pending) ASSERT_TIMELY_EQ (10s, 1, node1.unchecked.count ()); ASSERT_FALSE (node1.ledger.block_or_pruned_exists (epoch_open->hash ())); // Open block should be inserted into unchecked - auto blocks = node1.unchecked.get (nano::hash_or_account (epoch_open->account ()).hash); + auto blocks = node1.unchecked.get (nano::hash_or_account (epoch_open->account ().value ()).hash); ASSERT_EQ (blocks.size (), 1); ASSERT_EQ (blocks[0].block->full_hash (), epoch_open->full_hash ()); // New block to process epoch open diff --git a/nano/lib/blocks.cpp b/nano/lib/blocks.cpp index 688e614d1..253ecaa7e 100644 --- a/nano/lib/blocks.cpp +++ b/nano/lib/blocks.cpp @@ -185,10 +185,9 @@ nano::link const & nano::block::link () const return link; } -nano::account const & nano::block::account () const +std::optional nano::block::account () const { - static nano::account account{}; - return account; + return std::nullopt; } nano::qualified_root nano::block::qualified_root () const @@ -643,7 +642,7 @@ nano::block_hash const & nano::open_block::previous () const return result; } -nano::account const & nano::open_block::account () const +std::optional nano::open_block::account () const { return hashables.account; } @@ -1213,7 +1212,7 @@ nano::block_hash const & nano::state_block::previous () const return hashables.previous; } -nano::account const & nano::state_block::account () const +std::optional nano::state_block::account () const { return hashables.account; } diff --git a/nano/lib/blocks.hpp b/nano/lib/blocks.hpp index e89f2dddb..18514a907 100644 --- a/nano/lib/blocks.hpp +++ b/nano/lib/blocks.hpp @@ -33,7 +33,6 @@ public: virtual void hash (blake2b_state &) const = 0; virtual uint64_t block_work () const = 0; virtual void block_work_set (uint64_t) = 0; - virtual nano::account const & account () const; // Previous block in account's chain, zero for open block virtual nano::block_hash const & previous () const = 0; // Source block for open/receive blocks, zero otherwise. @@ -64,6 +63,10 @@ public: // If there are any changes to the hashables, call this to update the cached hash void refresh (); +public: // Direct access to the block fields or nullopt if the block type does not have the specified field + // Account field for open/state blocks + virtual std::optional account () const; + protected: mutable nano::block_hash cached_hash{ 0 }; /** @@ -210,7 +213,7 @@ public: uint64_t block_work () const override; void block_work_set (uint64_t) override; nano::block_hash const & previous () const override; - nano::account const & account () const override; + std::optional account () const override; nano::block_hash const & source () const override; nano::root const & root () const override; nano::account const & representative () const override; @@ -325,7 +328,7 @@ public: uint64_t block_work () const override; void block_work_set (uint64_t) override; nano::block_hash const & previous () const override; - nano::account const & account () const override; + std::optional account () const override; nano::root const & root () const override; nano::link const & link () const override; nano::account const & representative () const override; diff --git a/nano/node/active_transactions.cpp b/nano/node/active_transactions.cpp index 2545c5d41..b5c4a83df 100644 --- a/nano/node/active_transactions.cpp +++ b/nano/node/active_transactions.cpp @@ -186,7 +186,7 @@ void nano::active_transactions::notify_observers (nano::election_status const & void nano::active_transactions::handle_final_votes_confirmation (std::shared_ptr const & block, nano::store::read_transaction const & transaction, nano::election_status_type status) { - auto const & account = !block->account ().is_zero () ? block->account () : block->sideband ().account; + auto account = node.ledger.account (*block); bool is_canary_not_set = !node.ledger.cache.final_votes_confirmation_canary.load (); bool is_canary_account = account == node.network_params.ledger.final_votes_canary_account; diff --git a/nano/node/blockprocessor.cpp b/nano/node/blockprocessor.cpp index 731a000f7..54297c733 100644 --- a/nano/node/blockprocessor.cpp +++ b/nano/node/blockprocessor.cpp @@ -362,7 +362,7 @@ nano::block_status nano::block_processor::process_one (store::write_transaction } case nano::block_status::gap_epoch_open_pending: { - node.unchecked.put (block->account (), block); // Specific unchecked key starting with epoch open block account public key + node.unchecked.put (block->account ().value_or (0), block); // Specific unchecked key starting with epoch open block account public key node.stats.inc (nano::stat::type::ledger, nano::stat::detail::gap_source); break; } diff --git a/nano/node/bootstrap/bootstrap_bulk_pull.cpp b/nano/node/bootstrap/bootstrap_bulk_pull.cpp index cad42863b..00e9a8ece 100644 --- a/nano/node/bootstrap/bootstrap_bulk_pull.cpp +++ b/nano/node/bootstrap/bootstrap_bulk_pull.cpp @@ -188,7 +188,7 @@ void nano::bulk_pull_client::received_block (boost::system::error_code ec, std:: } if (pull_blocks == 0 && block_expected) { - known_account = block->account (); + known_account = block->account ().value_or (0); } if (connection->block_count++ == 0) { diff --git a/nano/node/bootstrap_ascending/service.cpp b/nano/node/bootstrap_ascending/service.cpp index 2db59429d..aad14eb00 100644 --- a/nano/node/bootstrap_ascending/service.cpp +++ b/nano/node/bootstrap_ascending/service.cpp @@ -167,7 +167,7 @@ void nano::bootstrap_ascending::service::inspect (store::transaction const & tx, break; case nano::block_status::gap_source: { - const auto account = block.previous ().is_zero () ? block.account () : ledger.account (tx, block.previous ()).value (); + const auto account = block.previous ().is_zero () ? block.account ().value () : ledger.account (tx, block.previous ()).value (); const auto source = block.source ().is_zero () ? block.link ().as_block_hash () : block.source (); // Mark account as blocked because it is missing the source block diff --git a/nano/node/cli.cpp b/nano/node/cli.cpp index 984f1656f..2f3372e40 100644 --- a/nano/node/cli.cpp +++ b/nano/node/cli.cpp @@ -1324,7 +1324,7 @@ void reset_confirmation_heights (nano::store::write_transaction const & transact store.confirmation_height.clear (transaction); // Then make sure the confirmation height of the genesis account open block is 1 - store.confirmation_height.put (transaction, constants.genesis->account (), { 1, constants.genesis->hash () }); + store.confirmation_height.put (transaction, constants.genesis->account ().value (), { 1, constants.genesis->hash () }); } bool is_using_rocksdb (std::filesystem::path const & data_path, boost::program_options::variables_map const & vm, std::error_code & ec) diff --git a/nano/node/confirmation_height_bounded.cpp b/nano/node/confirmation_height_bounded.cpp index 53c3ffbf3..4d221a3f5 100644 --- a/nano/node/confirmation_height_bounded.cpp +++ b/nano/node/confirmation_height_bounded.cpp @@ -104,11 +104,7 @@ void nano::confirmation_height_bounded::process (std::shared_ptr or release_assert (block); } } - nano::account account (block->account ()); - if (account.is_zero ()) - { - account = block->sideband ().account; - } + auto account = ledger.account (*block); // Checks if we have encountered this account before but not commited changes yet, if so then update the cached confirmation height nano::confirmation_height_info confirmation_height_info; diff --git a/nano/node/confirmation_height_unbounded.cpp b/nano/node/confirmation_height_unbounded.cpp index 03c4c42a2..cae1ec5cf 100644 --- a/nano/node/confirmation_height_unbounded.cpp +++ b/nano/node/confirmation_height_unbounded.cpp @@ -79,11 +79,7 @@ void nano::confirmation_height_unbounded::process (std::shared_ptr } release_assert (block); - nano::account account (block->account ()); - if (account.is_zero ()) - { - account = block->sideband ().account; - } + auto account = ledger.account (*block); auto block_height = block->sideband ().height; uint64_t confirmation_height = 0; diff --git a/nano/node/ipc/flatbuffers_util.cpp b/nano/node/ipc/flatbuffers_util.cpp index 8cf1fe5e4..eb90c4731 100644 --- a/nano/node/ipc/flatbuffers_util.cpp +++ b/nano/node/ipc/flatbuffers_util.cpp @@ -6,7 +6,7 @@ std::unique_ptr nano::ipc::flatbuffers_builder::from (nano::state_block const & block_a, nano::amount const & amount_a, bool is_state_send_a, bool is_state_epoch_a) { auto block (std::make_unique ()); - block->account = block_a.account ().to_account (); + block->account = block_a.account ().value ().to_account (); block->hash = block_a.hash ().to_string (); block->previous = block_a.previous ().to_string (); block->representative = block_a.representative ().to_account (); @@ -63,7 +63,7 @@ std::unique_ptr nano::ipc::flatbuffers_builder::from (nano: auto block (std::make_unique ()); block->hash = block_a.hash ().to_string (); block->source = block_a.source ().to_string (); - block->account = block_a.account ().to_account (); + block->account = block_a.account ().value ().to_account (); block->representative = block_a.representative ().to_account (); block_a.signature.encode_hex (block->signature); block->work = nano::to_string_hex (block_a.work); diff --git a/nano/node/json_handler.cpp b/nano/node/json_handler.cpp index eb2db379a..f6afe32f7 100644 --- a/nano/node/json_handler.cpp +++ b/nano/node/json_handler.cpp @@ -415,7 +415,7 @@ uint64_t nano::json_handler::difficulty_ledger (nano::block const & block_a) if (!link.is_zero () && !details.is_send) { auto block_link = node.ledger.block (transaction, link.as_block_hash ()); - if (block_link != nullptr && node.store.pending.exists (transaction, nano::pending_key (block_a.account (), link.as_block_hash ()))) + if (block_link != nullptr && node.store.pending.exists (transaction, nano::pending_key (block_a.account ().value (), link.as_block_hash ()))) { details.epoch = std::max (details.epoch, block_link->sideband ().details.epoch); details.is_receive = true; @@ -1131,7 +1131,7 @@ void nano::json_handler::active_difficulty () void nano::json_handler::available_supply () { - auto genesis_balance (node.balance (node.network_params.ledger.genesis->account ())); // Cold storage genesis + auto genesis_balance (node.balance (node.network_params.ledger.genesis->account ().value ())); // Cold storage genesis auto landing_balance (node.balance (nano::account ("059F68AAB29DE0D3A27443625C7EA9CDDB6517A8B76FE37727EF6A4D76832AD5"))); // Active unavailable account auto faucet_balance (node.balance (nano::account ("8E319CE6F3025E5B2DF66DA7AB1467FE48F1679C13DD43BFDB29FA2E9FC40D3B"))); // Faucet account auto burned_balance ((node.balance_pending (nano::account{}, false)).second); // Burning 0 account @@ -1149,7 +1149,7 @@ void nano::json_handler::block_info () auto block = node.ledger.block (transaction, hash); if (block != nullptr) { - nano::account account (block->account ().is_zero () ? block->sideband ().account : block->account ()); + auto account = node.ledger.account (*block); response_l.put ("block_account", account.to_account ()); auto amount = node.ledger.amount (transaction, hash); if (amount) @@ -1306,7 +1306,7 @@ void nano::json_handler::blocks_info () if (block != nullptr) { boost::property_tree::ptree entry; - nano::account account (block->account ().is_zero () ? block->sideband ().account : block->account ()); + auto account = node.ledger.account (*block); entry.put ("block_account", account.to_account ()); auto amount = node.ledger.amount (transaction, hash); if (amount) @@ -2454,7 +2454,7 @@ public: } else { - tree.put ("account", handler.node.ledger.constants.genesis->account ().to_account ()); + tree.put ("account", handler.node.ledger.constants.genesis->account ().value ().to_account ()); tree.put ("amount", nano::dev::constants.genesis_amount.convert_to ()); } } diff --git a/nano/node/node.cpp b/nano/node/node.cpp index 0af8e6da7..bcf845bf9 100644 --- a/nano/node/node.cpp +++ b/nano/node/node.cpp @@ -1226,11 +1226,7 @@ void nano::node::receive_confirmed (store::transaction const & block_transaction void nano::node::process_confirmed_data (store::transaction const & transaction_a, std::shared_ptr const & block_a, nano::block_hash const & hash_a, nano::account & account_a, nano::uint128_t & amount_a, bool & is_state_send_a, bool & is_state_epoch_a, nano::account & pending_account_a) { // Faster account calculation - account_a = block_a->account (); - if (account_a.is_zero ()) - { - account_a = block_a->sideband ().account; - } + account_a = ledger.account (*block_a); // Faster amount calculation auto previous (block_a->previous ()); auto previous_balance = ledger.balance (transaction_a, previous); diff --git a/nano/node/nodeconfig.cpp b/nano/node/nodeconfig.cpp index beb93cb4f..edd039d82 100644 --- a/nano/node/nodeconfig.cpp +++ b/nano/node/nodeconfig.cpp @@ -49,7 +49,7 @@ nano::node_config::node_config (const std::optional & peering_port_a, { case nano::networks::nano_dev_network: enable_voting = true; - preconfigured_representatives.push_back (network_params.ledger.genesis->account ()); + preconfigured_representatives.push_back (network_params.ledger.genesis->account ().value ()); break; case nano::networks::nano_beta_network: { @@ -72,7 +72,7 @@ nano::node_config::node_config (const std::optional & peering_port_a, break; case nano::networks::nano_test_network: preconfigured_peers.push_back (default_test_peer_network); - preconfigured_representatives.push_back (network_params.ledger.genesis->account ()); + preconfigured_representatives.push_back (network_params.ledger.genesis->account ().value ()); break; default: debug_assert (false); diff --git a/nano/node/process_live_dispatcher.cpp b/nano/node/process_live_dispatcher.cpp index 8411851d4..4fd77e3c3 100644 --- a/nano/node/process_live_dispatcher.cpp +++ b/nano/node/process_live_dispatcher.cpp @@ -45,8 +45,7 @@ void nano::process_live_dispatcher::process_live (nano::block const & block, sto // Start collecting quorum on block if (ledger.dependents_confirmed (transaction, block)) { - auto account = block.account ().is_zero () ? block.sideband ().account : block.account (); - scheduler.activate (account, transaction); + scheduler.activate (ledger.account (block), transaction); } if (websocket.server && websocket.server->any_subscriber (nano::websocket::topic::new_unconfirmed_block)) diff --git a/nano/node/request_aggregator.cpp b/nano/node/request_aggregator.cpp index f822ace91..43e417219 100644 --- a/nano/node/request_aggregator.cpp +++ b/nano/node/request_aggregator.cpp @@ -229,7 +229,7 @@ std::pair>, std::vectoraccount ().is_zero () ? block->sideband ().account : block->account (), confirmation_height_info); + ledger.store.confirmation_height.get (transaction, ledger.account (*block), confirmation_height_info); generate_final_vote = (confirmation_height_info.height >= block->sideband ().height); } } @@ -265,7 +265,7 @@ std::pair>, std::vectoraccount ().is_zero () ? block->sideband ().account : block->account (), confirmation_height_info); + ledger.store.confirmation_height.get (transaction, ledger.account (*block), confirmation_height_info); generate_final_vote = (confirmation_height_info.height >= block->sideband ().height); } } diff --git a/nano/qt/qt.cpp b/nano/qt/qt.cpp index 640080b55..4942294f3 100644 --- a/nano/qt/qt.cpp +++ b/nano/qt/qt.cpp @@ -566,7 +566,7 @@ public: } else { - account = ledger.constants.genesis->account (); + account = ledger.constants.genesis->account ().value (); amount = nano::dev::constants.genesis_amount; } } diff --git a/nano/rpc_test/rpc.cpp b/nano/rpc_test/rpc.cpp index 16fd73146..9045fc413 100644 --- a/nano/rpc_test/rpc.cpp +++ b/nano/rpc_test/rpc.cpp @@ -3314,7 +3314,7 @@ TEST (rpc, pending_exists) ASSERT_TRUE (pending_exists ("1")); ASSERT_TRUE (pending_exists ("1")); - reset_confirmation_height (node->store, block1->account ()); + reset_confirmation_height (node->store, block1->account ().value ()); ASSERT_TRUE (pending_exists ("0")); request.put ("include_only_confirmed", "false"); ASSERT_TRUE (pending_exists ("1")); @@ -3419,7 +3419,7 @@ TEST (rpc, wallet_receivable) ASSERT_EQ (sources[block1->hash ()], nano::dev::genesis_key.pub); ASSERT_TRUE (check_block_response_count (system, rpc_ctx, request, 1)); - reset_confirmation_height (system.nodes.front ()->store, block1->account ()); + reset_confirmation_height (system.nodes.front ()->store, block1->account ().value ()); ASSERT_TRUE (check_block_response_count (system, rpc_ctx, request, 0)); request.put ("include_only_confirmed", "false"); ASSERT_TRUE (check_block_response_count (system, rpc_ctx, request, 1)); diff --git a/nano/secure/common.cpp b/nano/secure/common.cpp index 091e45fcc..0812ee4ea 100644 --- a/nano/secure/common.cpp +++ b/nano/secure/common.cpp @@ -127,15 +127,15 @@ nano::ledger_constants::ledger_constants (nano::work_thresholds & work, nano::ne : network_a == nano::networks::nano_test_network ? nano_test_final_votes_canary_height : nano_live_final_votes_canary_height) { - nano_beta_genesis->sideband_set (nano::block_sideband (nano_beta_genesis->account (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); - nano_dev_genesis->sideband_set (nano::block_sideband (nano_dev_genesis->account (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); - nano_live_genesis->sideband_set (nano::block_sideband (nano_live_genesis->account (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); - nano_test_genesis->sideband_set (nano::block_sideband (nano_test_genesis->account (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); + nano_beta_genesis->sideband_set (nano::block_sideband (nano_beta_genesis->account ().value (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); + nano_dev_genesis->sideband_set (nano::block_sideband (nano_dev_genesis->account ().value (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); + nano_live_genesis->sideband_set (nano::block_sideband (nano_live_genesis->account ().value (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); + nano_test_genesis->sideband_set (nano::block_sideband (nano_test_genesis->account ().value (), 0, std::numeric_limits::max (), 1, nano::seconds_since_epoch (), nano::epoch::epoch_0, false, false, false, nano::epoch::epoch_0)); nano::link epoch_link_v1; char const * epoch_message_v1 ("epoch v1 block"); strncpy ((char *)epoch_link_v1.bytes.data (), epoch_message_v1, epoch_link_v1.bytes.size ()); - epochs.add (nano::epoch::epoch_1, genesis->account (), epoch_link_v1); + epochs.add (nano::epoch::epoch_1, genesis->account ().value (), epoch_link_v1); nano::link epoch_link_v2; nano::account nano_live_epoch_v2_signer; diff --git a/nano/secure/ledger.cpp b/nano/secure/ledger.cpp index 561a87c07..be4afa8d2 100644 --- a/nano/secure/ledger.cpp +++ b/nano/secure/ledger.cpp @@ -1132,13 +1132,7 @@ bool nano::ledger::rollback (store::write_transaction const & transaction_a, nan nano::account nano::ledger::account (nano::block const & block) { debug_assert (block.has_sideband ()); - nano::account result (block.account ()); - if (result.is_zero ()) - { - result = block.sideband ().account; - } - debug_assert (!result.is_zero ()); - return result; + return block.account ().value_or (block.sideband ().account); } std::optional nano::ledger::account (store::transaction const & transaction, nano::block_hash const & hash) const @@ -1431,7 +1425,7 @@ bool nano::ledger::block_confirmed (store::transaction const & transaction_a, na if (block_l) { nano::confirmation_height_info confirmation_height_info; - store.confirmation_height.get (transaction_a, block_l->account ().is_zero () ? block_l->sideband ().account : block_l->account (), confirmation_height_info); + store.confirmation_height.get (transaction_a, account (*block_l), confirmation_height_info); auto confirmed (confirmation_height_info.height >= block_l->sideband ().height); return confirmed; } @@ -1616,7 +1610,7 @@ bool nano::ledger::migrate_lmdb_to_rocksdb (std::filesystem::path const & data_p auto random_block (store.block.random (lmdb_transaction)); error |= rocksdb_store->block.get (rocksdb_transaction, random_block->hash ()) == nullptr; - auto account = random_block->account ().is_zero () ? random_block->sideband ().account : random_block->account (); + auto account = this->account (*random_block); nano::account_info account_info; error |= rocksdb_store->account.get (rocksdb_transaction, account, account_info); diff --git a/nano/store/component.cpp b/nano/store/component.cpp index 67826f634..c95c16a19 100644 --- a/nano/store/component.cpp +++ b/nano/store/component.cpp @@ -31,11 +31,11 @@ void nano::store::component::initialize (store::write_transaction const & transa auto hash_l (constants.genesis->hash ()); block.put (transaction_a, hash_l, *constants.genesis); ++ledger_cache_a.block_count; - confirmation_height.put (transaction_a, constants.genesis->account (), nano::confirmation_height_info{ 1, constants.genesis->hash () }); + confirmation_height.put (transaction_a, constants.genesis->account ().value (), nano::confirmation_height_info{ 1, constants.genesis->hash () }); ++ledger_cache_a.cemented_count; ledger_cache_a.final_votes_confirmation_canary = (constants.final_votes_canary_account == constants.genesis->account () && 1 >= constants.final_votes_canary_height); - account.put (transaction_a, constants.genesis->account (), { hash_l, constants.genesis->account (), constants.genesis->hash (), std::numeric_limits::max (), nano::seconds_since_epoch (), 1, nano::epoch::epoch_0 }); + account.put (transaction_a, constants.genesis->account ().value (), { hash_l, constants.genesis->account ().value (), constants.genesis->hash (), std::numeric_limits::max (), nano::seconds_since_epoch (), 1, nano::epoch::epoch_0 }); ++ledger_cache_a.account_count; - ledger_cache_a.rep_weights.representation_put (constants.genesis->account (), std::numeric_limits::max ()); - frontier.put (transaction_a, hash_l, constants.genesis->account ()); + ledger_cache_a.rep_weights.representation_put (constants.genesis->account ().value (), std::numeric_limits::max ()); + frontier.put (transaction_a, hash_l, constants.genesis->account ().value ()); }