Merge pull request #3399 from TheBlueMatt/2024-11-fuzz-assert

TheBlueMatt · web-flow · commit 6bf218198fbb · 2024-11-07T21:43:12.000Z
Drop broken debug assertion on peer buffer lengths
diff --git a/lightning/src/ln/peer_handler.rs b/lightning/src/ln/peer_handler.rs
@@ -1365,7 +1365,6 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM
 	/// Append a message to a peer's pending outbound/write gossip broadcast buffer
 	fn enqueue_encoded_gossip_broadcast(&self, peer: &mut Peer, encoded_message: MessageBuf) {
 		peer.msgs_sent_since_pong += 1;
-		debug_assert!(peer.gossip_broadcast_buffer.len() <= OUTBOUND_BUFFER_LIMIT_DROP_GOSSIP);
 		peer.gossip_broadcast_buffer.push_back(encoded_message);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1365,7 +1365,6 @@ impl<Descriptor: SocketDescriptor, CM: Deref, RM: Deref, OM: Deref, L: Deref, CM`
`1365`	`1365`	`/// Append a message to a peer's pending outbound/write gossip broadcast buffer`
`1366`	`1366`	`fn enqueue_encoded_gossip_broadcast(&self, peer: &mut Peer, encoded_message: MessageBuf) {`
`1367`	`1367`	`peer.msgs_sent_since_pong += 1;`
`1368`		`- debug_assert!(peer.gossip_broadcast_buffer.len() <= OUTBOUND_BUFFER_LIMIT_DROP_GOSSIP);`
`1369`	`1368`	`peer.gossip_broadcast_buffer.push_back(encoded_message);`
`1370`	`1369`	`}`
`1371`	`1370`