Skip to content

Commit 1fa2da3

Browse files
committed
[fuzz] Allow SendAnnouncementSigs events in chanmon_consistency
Because of the merge between peer reconnection and channel monitor updating channel restoration code, we now sometimes generate (somewhat spurious) announcement signatures when restoring channel monitor updating. This should not result in a fuzzing failure.
1 parent d90aac8 commit 1fa2da3

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

fuzz/src/chanmon_consistency.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -603,6 +603,9 @@ pub fn do_test<Out: test_logger::Output>(data: &[u8], out: Out) {
603603
events::MessageSendEvent::SendFundingLocked { .. } => {
604604
// Can be generated as a reestablish response
605605
},
606+
events::MessageSendEvent::SendAnnouncementSignatures { .. } => {
607+
// Can be generated as a reestablish response
608+
},
606609
events::MessageSendEvent::PaymentFailureNetworkUpdate { .. } => {
607610
// Can be generated due to a payment forward being rejected due to a
608611
// channel having previously failed a monitor update
@@ -623,6 +626,7 @@ pub fn do_test<Out: test_logger::Output>(data: &[u8], out: Out) {
623626
events::MessageSendEvent::SendRevokeAndACK { .. } => {},
624627
events::MessageSendEvent::SendChannelReestablish { .. } => {},
625628
events::MessageSendEvent::SendFundingLocked { .. } => {},
629+
events::MessageSendEvent::SendAnnouncementSignatures { .. } => {},
626630
events::MessageSendEvent::PaymentFailureNetworkUpdate { .. } => {},
627631
_ => panic!("Unhandled message event"),
628632
}
@@ -635,6 +639,7 @@ pub fn do_test<Out: test_logger::Output>(data: &[u8], out: Out) {
635639
events::MessageSendEvent::SendRevokeAndACK { .. } => {},
636640
events::MessageSendEvent::SendChannelReestablish { .. } => {},
637641
events::MessageSendEvent::SendFundingLocked { .. } => {},
642+
events::MessageSendEvent::SendAnnouncementSignatures { .. } => {},
638643
events::MessageSendEvent::PaymentFailureNetworkUpdate { .. } => {},
639644
_ => panic!("Unhandled message event"),
640645
}
@@ -656,6 +661,7 @@ pub fn do_test<Out: test_logger::Output>(data: &[u8], out: Out) {
656661
if *node_id != drop_node_id { true } else { panic!("peer_disconnected should drop msgs bound for the disconnected peer"); }
657662
},
658663
events::MessageSendEvent::SendFundingLocked { .. } => false,
664+
events::MessageSendEvent::SendAnnouncementSignatures { .. } => false,
659665
events::MessageSendEvent::PaymentFailureNetworkUpdate { .. } => false,
660666
_ => panic!("Unhandled message event"),
661667
};

0 commit comments

Comments
 (0)