Limit reply size on TMGetObjectByHash queries

src/test/rpc/Subscribe_test.cpp

@@ -872,7 +872,7 @@ class Subscribe_test : public beast::unit_test::suite
             for (int i = 0; i < numReplies; ++i)
             {
                 std::uint32_t idx{0};
-                auto reply = wsc.getMsg(100ms);
+                auto reply = wsc.getMsg(1s);
                 if (reply)
                 {
                     auto r = *reply;

src/xrpld/overlay/detail/PeerImp.cpp

@@ -1351,8 +1351,8 @@
     {
         // If we've never been in synch, there's nothing we can do
         // with a transaction
-        JLOG(p_journal_.debug()) << "Ignoring incoming transaction: "
-                                 << "Need network ledger";
+        JLOG(p_journal_.debug())
+            << "Ignoring incoming transaction: Need network ledger";
         return;
     }
 
@@ -2618,6 +2618,13 @@
                         newObj.set_ledgerseq(obj.ledgerseq());
 
                     // VFALCO NOTE "seq" in the message is obsolete
+
+                    // Check if by adding this object, reply has reached its
+                    // limit
+                    if (reply.objects_size() >= Tuning::hardMaxReplyNodes)
+                    {
+                        break;
+                    }
                 }
             }
         }