rpcdaemon: fix account state in debug_accountAt (#1893)

erigontech · Mar 11, 2024 · 65b0504 · 65b0504
1 parent 13d0b98
commit 65b0504
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/rpc-integration-tests.yml b/.github/workflows/rpc-integration-tests.yml
@@ -26,7 +26,7 @@ jobs:
       - name: Checkout RPC Tests Repository & Install Requirements
         run: |
           rm -rf ${{runner.workspace}}/rpc-tests
-          git -c advice.detachedHead=false clone --depth 1 --branch v0.4.0 https://github.com/erigontech/rpc-tests ${{runner.workspace}}/rpc-tests
+          git -c advice.detachedHead=false clone --depth 1 --branch v0.5.0 https://github.com/erigontech/rpc-tests ${{runner.workspace}}/rpc-tests
           cd ${{runner.workspace}}/rpc-tests
           pip3 install -r requirements.txt
 

diff --git a/silkworm/rpc/commands/debug_api.cpp b/silkworm/rpc/commands/debug_api.cpp
@@ -306,7 +306,7 @@ Task<void> DebugRpcApi::handle_debug_account_at(const nlohmann::json& request, n
         }
 
         const auto& block = block_with_hash->block;
-        auto block_number = block.header.number - 1;
+        auto block_number = block.header.number;
         const auto& transactions = block.transactions;
 
         SILK_DEBUG << "Block number: " << block_number << " #tnx: " << transactions.size();
@@ -324,7 +324,7 @@ Task<void> DebugRpcApi::handle_debug_account_at(const nlohmann::json& request, n
 
                     EVMExecutor executor{*chain_config_ptr, workers_, state};
 
-                    uint64_t index = std::min(static_cast<uint64_t>(transactions.size()), tx_index + 1);
+                    uint64_t index = std::min(static_cast<uint64_t>(transactions.size()), tx_index);
                     for (uint64_t idx{0}; idx < index; idx++) {
                         rpc::Transaction txn{transactions[idx]};
                         executor.call(block, txn);

diff --git a/silkworm/rpc/core/evm_executor.cpp b/silkworm/rpc/core/evm_executor.cpp
@@ -168,11 +168,12 @@ uint64_t EVMExecutor::refund_gas(const EVM& evm, const silkworm::Transaction& tx
     const uint64_t max_refund_quotient{rev >= EVMC_LONDON ? protocol::kMaxRefundQuotientLondon
                                                           : protocol::kMaxRefundQuotientFrontier};
     const uint64_t max_refund{(txn.gas_limit - gas_left) / max_refund_quotient};
-    const uint64_t refund = gas_refund < max_refund ? gas_refund : max_refund;  // min
+    const uint64_t refund = std::min(gas_refund, max_refund);
     gas_left += refund;
 
     const intx::uint256 base_fee_per_gas{evm.block().header.base_fee_per_gas.value_or(0)};
     SILK_DEBUG << "EVMExecutor::refund_gas txn.max_fee_per_gas: " << txn.max_fee_per_gas << " base_fee_per_gas: " << base_fee_per_gas;
+
     const intx::uint256 effective_gas_price{txn.max_fee_per_gas >= base_fee_per_gas ? txn.effective_gas_price(base_fee_per_gas)
                                                                                     : txn.max_priority_fee_per_gas};
     SILK_DEBUG << "EVMExecutor::refund_gas effective_gas_price: " << effective_gas_price;
@@ -218,8 +219,8 @@ ExecutionResult EVMExecutor::call(
     const Tracers& tracers,
     bool refund,
     bool gas_bailout) {
-    SILK_DEBUG << "EVMExecutor::call: " << block.header.number << " gasLimit: " << txn.gas_limit << " refund: " << refund << " gasBailout: " << gas_bailout;
-    SILK_DEBUG << "EVMExecutor::call: transaction: " << &txn;
+    SILK_DEBUG << "EVMExecutor::call: blockNumber: " << block.header.number << " gasLimit: " << txn.gas_limit << " refund: " << refund << " gasBailout: " << gas_bailout;
+    SILK_DEBUG << "EVMExecutor::call: transaction: " << rpc::Transaction{txn};
 
     auto& svc = use_service<AnalysisCacheService>(workers_);
     EVM evm{block, ibs_state_, config_};