Adapt to chain and database block population changes.

Author: evoskuil

include/bitcoin/node/chasers/chaser_block.hpp

@@ -68,8 +68,7 @@ class BCN_API chaser_block
 
 private:
     void set_prevout(const system::chain::input& input) const NOEXCEPT;
-    bool populate(const system::chain::block& block,
-        const system::chain::context& ctx) const NOEXCEPT;
+    bool populate(const system::chain::block& block) const NOEXCEPT;
 };
 
 } // namespace node

src/chasers/chaser_block.cpp

@@ -117,12 +117,11 @@ code chaser_block::validate(const block& block,
             return ec;
     }
 
-    // Populate prevouts from self/tree (metadata not required).
-    if (!populate(block, ctx))
-        return system::error::relative_time_locked;
+    // Populate prevouts from self/tree (no metadata for accept/connect).
+    populate(block);
 
-    // Populate prevouts from store (metadata not required).
-    if (!archive().populate(block))
+    // Populate prevouts from store (no metadata for accept/connect).
+    if (!archive().populate_without_metadata(block))
         return network::error::protocol_violation;
 
     if ((ec = block.accept(ctx,
@@ -156,6 +155,8 @@ void chaser_block::update_milestone(const header&, size_t, size_t) NOEXCEPT
 void chaser_block::set_prevout(const input& input) const NOEXCEPT
 {
     const auto& point = input.point();
+    if (input.prevout || point.is_null())
+        return;
 
     // Scan all tree blocks for matching tx (linear :/ but legacy scenario)
     std::for_each(tree().begin(), tree().end(), [&](const auto& item) NOEXCEPT
@@ -173,25 +174,22 @@ void chaser_block::set_prevout(const input& input) const NOEXCEPT
             const outputs_cptr outs{ tx->outputs_ptr() };
             if (point.index() < outs->size())
             {
+                // prevout is mutable so can be set on a const object.
                 input.prevout = outs->at(point.index());
                 return;
             }
         }
     });
 }
 
-// metadata is mutable so can be set on a const object.
-bool chaser_block::populate(const block& block,
-    const system::chain::context& ctx) const NOEXCEPT
+bool chaser_block::populate(const block& block) const NOEXCEPT
 {
-    if (!block.populate(ctx))
-        return false;
+    block.populate();
 
-    const inputs_cptr ins{ block.inputs_ptr() };
-    std::for_each(ins->begin(), ins->end(), [&](const auto& in) NOEXCEPT
+    const auto& ins = *block.inputs_ptr();
+    std::for_each(ins.begin(), ins.end(), [&](const auto& in) NOEXCEPT
     {
-        if (!in->prevout && !in->point().is_null())
-            set_prevout(*in);
+        set_prevout(*in);
     });
 
     return true;

src/chasers/chaser_validate.cpp

@@ -238,18 +238,21 @@ code chaser_validate::populate(bool bypass, const chain::block& block,
 {
     const auto& query = archive();
 
-    // Relative locktime check is unnecessary under bypass, but cheap.
-    if (!block.populate(ctx))
-        return system::error::relative_time_locked;
-
     if (bypass)
     {
+        block.populate();
         if (!query.populate_without_metadata(block))
             return system::error::missing_previous_output;
     }
     else
     {
-        if (!query.populate(block))
+        // Internal maturity and time locks are verified here because they are
+        // the only necessary confirmation checks for internal spends.
+        if (const auto ec = block.populate_with_metadata(ctx))
+            return ec;
+
+        // Metadata identifies internal spends alowing confirmation bypass.
+        if (!query.populate_with_metadata(block))
             return system::error::missing_previous_output;
     }