Skip to content

Commit 07b4cb6

Browse files
Onion message fuzzer: update held_htlc_available handling
Fuzzers should always do more, not less. Post-merge feedback on e8f154d.
1 parent 88e1b56 commit 07b4cb6

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

fuzz/src/onion_message.rs

+8-3
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ use lightning::ln::script::ShutdownScript;
1313
use lightning::offers::invoice::UnsignedBolt12Invoice;
1414
use lightning::offers::invoice_request::UnsignedInvoiceRequest;
1515
use lightning::onion_message::async_payments::{
16-
AsyncPaymentsMessage, AsyncPaymentsMessageHandler, HeldHtlcAvailable, ReleaseHeldHtlc,
16+
AsyncPaymentsMessageHandler, HeldHtlcAvailable, ReleaseHeldHtlc,
1717
};
1818
use lightning::onion_message::messenger::{
1919
CustomOnionMessageHandler, Destination, MessageRouter, OnionMessagePath, OnionMessenger,
@@ -114,9 +114,14 @@ struct TestAsyncPaymentsMessageHandler {}
114114

115115
impl AsyncPaymentsMessageHandler for TestAsyncPaymentsMessageHandler {
116116
fn held_htlc_available(
117-
&self, _message: HeldHtlcAvailable, _responder: Option<Responder>,
117+
&self, message: HeldHtlcAvailable, responder: Option<Responder>,
118118
) -> ResponseInstruction<ReleaseHeldHtlc> {
119-
ResponseInstruction::NoResponse
119+
let responder = match responder {
120+
Some(resp) => resp,
121+
None => return ResponseInstruction::NoResponse,
122+
};
123+
responder
124+
.respond(ReleaseHeldHtlc { payment_release_secret: message.payment_release_secret })
120125
}
121126
fn release_held_htlc(&self, _message: ReleaseHeldHtlc) {}
122127
}

0 commit comments

Comments
 (0)