Copilot review feedback

src/libxrpl/shamap/SHAMapSync.cpp

@@ -486,6 +486,13 @@ SHAMap::addRootNode(
     intr_ptr::SharedPtr<SHAMapTreeNode> rootNode,
     SHAMapSyncFilter const* filter)
 {
+    XRPL_ASSERT(rootNode, "xrpl::SHAMap::addKnownNode : non-null root node");
+    if (!rootNode)
+    {
+        JLOG(journal_.error()) << "Null node received";
+        return SHAMapAddNode::invalid();
+    }
+
     // we already have a root_ node
     if (root_->getHash().isNonZero())
     {
@@ -527,6 +534,12 @@ SHAMap::addKnownNode(
     SHAMapSyncFilter const* filter)
 {
     XRPL_ASSERT(!nodeID.isRoot(), "xrpl::SHAMap::addKnownNode : valid node input");
+    XRPL_ASSERT(treeNode, "xrpl::SHAMap::addKnownNode : non-null tree node");
+    if (!treeNode)
+    {
+        JLOG(journal_.error()) << "Null node received";
+        return SHAMapAddNode::invalid();
+    }
 
     if (!isSynching())
     {

src/test/shamap/SHAMapSync_test.cpp

@@ -110,6 +110,8 @@ public:
             unexpected(a.size() < 1, "NodeSize");
 
             auto node = SHAMapTreeNode::makeFromWire(makeSlice(std::get<1>(a[0])));
+            if (!node)
+                fail("", __FILE__, __LINE__);
             BEAST_EXPECT(
                 destination.addRootNode(source.getHash(), std::move(node), nullptr).isGood());
         }
@@ -148,6 +150,8 @@ public:
                 // non-deterministic number of times and the number of tests run
                 // should be deterministic
                 auto node = SHAMapTreeNode::makeFromWire(makeSlice(std::get<1>(b[i])));
+                if (!node)
+                    fail("", __FILE__, __LINE__);
                 if (!destination.addKnownNode(std::get<0>(b[i]), std::move(node), nullptr)
                          .isUseful())
                     fail("", __FILE__, __LINE__);

src/xrpld/app/ledger/detail/LedgerNodeHelpers.h

@@ -26,7 +26,7 @@ namespace xrpl {
  * @param ledger_node The ledger node to validate.
  * @return true if the ledger node has the expected fields, false otherwise.
  */
-bool
+[[nodiscard]] bool
 validateLedgerNode(protocol::TMLedgerNode const& ledger_node);
 
 /**