Fix cltv_expiry_delta comment
What the developer wrote
Authored by elnosh
Fix cltv_expiry_delta comment
Fix incorrect comment about cltv_expiry_delta.
The cltv_expiry on the outgoing HTLC must be less than the
one in the incoming HTLC.
What changed, and why it matters
This commit only fixes a comment describing a Lightning Network routing parameter. It changes no actual code logic, so it cannot affect security or behavior.
No action required; this is a non-functional documentation fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch updates the doc comment for cltv_expiry_delta in lightning/src/ln/msgs.rs. It corrects the wording from ‘minimum cltv_expiry value’ to ‘maximum cltv_expiry value’ and adjusts the example arithmetic accordingly. No executable Rust code is modified.
Changed components
lightning/src/ln/msgs.rs documentation comment onlyInspect captured patch +2 / −2
diff --git a/lightning/src/ln/msgs.rs b/lightning/src/ln/msgs.rs
index d49c573..6210d26 100644
--- a/lightning/src/ln/msgs.rs
+++ b/lightning/src/ln/msgs.rs
@@ -1491,9 +1491,9 @@ pub struct UnsignedChannelUpdate {
/// The number of blocks such that if:
/// `incoming_htlc.cltv_expiry < outgoing_htlc.cltv_expiry + cltv_expiry_delta`
/// then we need to fail the HTLC backwards. When forwarding an HTLC, `cltv_expiry_delta` determines
- /// the outgoing HTLC's minimum `cltv_expiry` value -- so, if an incoming HTLC comes in with a
+ /// the outgoing HTLC's maximum `cltv_expiry` value -- so, if an incoming HTLC comes in with a
/// `cltv_expiry` of 100000, and the node we're forwarding to has a `cltv_expiry_delta` value of 10,
- /// then we'll check that the outgoing HTLC's `cltv_expiry` value is at least 100010 before
+ /// then we'll check that the outgoing HTLC's `cltv_expiry` value is at most 99990 before
/// forwarding. Note that the HTLC sender is the one who originally sets this value when
/// constructing the route.
pub cltv_expiry_delta: u16,
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.