Use HolderCommitmentPoint::current_transaction_number

jkczyz · Anyitechs · commit 48c6e90a2b06 · 2025-09-09T01:54:24.000+01:00
diff --git a/lightning/src/ln/channel.rs b/lightning/src/ln/channel.rs
@@ -9166,7 +9166,7 @@ where
 			return Err(ChannelError::close("Peer sent an invalid channel_reestablish to force close in a non-standard way".to_owned()));
 		}
 
-		let our_commitment_transaction = INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.next_transaction_number() - 1;
+		let our_commitment_transaction = INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.current_transaction_number();
 		if msg.next_remote_commitment_number > 0 {
 			let expected_point = self.context.holder_signer.as_ref()
 				.get_per_commitment_point(INITIAL_COMMITMENT_NUMBER - msg.next_remote_commitment_number + 1, &self.context.secp_ctx)

Original file line number	Diff line number	Diff line change
`@@ -9166,7 +9166,7 @@ where`
`9166`	`9166`	`return Err(ChannelError::close("Peer sent an invalid channel_reestablish to force close in a non-standard way".to_owned()));`
`9167`	`9167`	`}`
`9168`	`9168`
`9169`		`- let our_commitment_transaction = INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.next_transaction_number() - 1;`
	`9169`	`+ let our_commitment_transaction = INITIAL_COMMITMENT_NUMBER - self.holder_commitment_point.current_transaction_number();`
`9170`	`9170`	`if msg.next_remote_commitment_number > 0 {`
`9171`	`9171`	`let expected_point = self.context.holder_signer.as_ref()`
`9172`	`9172`	`.get_per_commitment_point(INITIAL_COMMITMENT_NUMBER - msg.next_remote_commitment_number + 1, &self.context.secp_ctx)`