diff --git a/src/xrpld/overlay/detail/PeerImp.cpp b/src/xrpld/overlay/detail/PeerImp.cpp index fb0ced1a34..688d0ac314 100644 --- a/src/xrpld/overlay/detail/PeerImp.cpp +++ b/src/xrpld/overlay/detail/PeerImp.cpp @@ -1741,7 +1741,9 @@ PeerImp::onMessage(std::shared_ptr const& m) { auto* ledgerNode = m->mutable_nodes(i); - if (ledgerNode->has_nodedata().empty()) + // All nodes should have non-empty data. The field is required so we don't need to + // check for presence first. + if (ledgerNode->nodedata().empty()) { badData( "Received node with empty data while relaying ledger data for " +