Skip to content

Commit d830f10

Browse files
authored
Merge pull request #4496 from joostjager/fix-fuzz-missing-msgs
fuzz: handle missing SendTx* message events in chanmon_consistency
2 parents 1050ed9 + 279f2c1 commit d830f10

File tree

1 file changed

+32
-0
lines changed

1 file changed

+32
-0
lines changed

fuzz/src/chanmon_consistency.rs

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1513,6 +1513,14 @@ pub fn do_test<Out: Output + MaybeSend + MaybeSync>(
15131513
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
15141514
*node_id == a_id
15151515
},
1516+
MessageSendEvent::SendTxRemoveInput { ref node_id, .. } => {
1517+
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
1518+
*node_id == a_id
1519+
},
1520+
MessageSendEvent::SendTxRemoveOutput { ref node_id, .. } => {
1521+
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
1522+
*node_id == a_id
1523+
},
15161524
MessageSendEvent::SendTxComplete { ref node_id, .. } => {
15171525
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
15181526
*node_id == a_id
@@ -1521,6 +1529,14 @@ pub fn do_test<Out: Output + MaybeSend + MaybeSync>(
15211529
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
15221530
*node_id == a_id
15231531
},
1532+
MessageSendEvent::SendTxInitRbf { ref node_id, .. } => {
1533+
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
1534+
*node_id == a_id
1535+
},
1536+
MessageSendEvent::SendTxAckRbf { ref node_id, .. } => {
1537+
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
1538+
*node_id == a_id
1539+
},
15241540
MessageSendEvent::SendTxSignatures { ref node_id, .. } => {
15251541
if Some(*node_id) == expect_drop_id { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
15261542
*node_id == a_id
@@ -1713,6 +1729,22 @@ pub fn do_test<Out: Output + MaybeSend + MaybeSync>(
17131729
}
17141730
}
17151731
},
1732+
MessageSendEvent::SendTxInitRbf { ref node_id, ref msg } => {
1733+
for (idx, dest) in nodes.iter().enumerate() {
1734+
if dest.get_our_node_id() == *node_id {
1735+
out.locked_write(format!("Delivering tx_init_rbf from node {} to node {}.\n", $node, idx).as_bytes());
1736+
dest.handle_tx_init_rbf(nodes[$node].get_our_node_id(), msg);
1737+
}
1738+
}
1739+
},
1740+
MessageSendEvent::SendTxAckRbf { ref node_id, ref msg } => {
1741+
for (idx, dest) in nodes.iter().enumerate() {
1742+
if dest.get_our_node_id() == *node_id {
1743+
out.locked_write(format!("Delivering tx_ack_rbf from node {} to node {}.\n", $node, idx).as_bytes());
1744+
dest.handle_tx_ack_rbf(nodes[$node].get_our_node_id(), msg);
1745+
}
1746+
}
1747+
},
17161748
MessageSendEvent::SendTxSignatures { ref node_id, ref msg } => {
17171749
for (idx, dest) in nodes.iter().enumerate() {
17181750
if dest.get_our_node_id() == *node_id {

0 commit comments

Comments
 (0)