What the developer wrote
Authored by ziggie
chanstate: add previous state lookup
Add FindPreviousState to the chanstate commitment store subinterface now that
RevocationLog is a chanstate value type.
This extends the store contract without changing runtime behavior. The
existing ChannelStateDB method already satisfies the new method.
What changed, and why it matters
This commit only adds a new method declaration to an internal Go interface in LND's channel state package. It does not change any running code, data handling, or security behavior. The commit message explicitly says it extends the store contract without changing runtime behavior. There is no security issue visible in this change.
No action needed. This is a non-functional interface change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds FindPreviousState to the OpenChannelCommitmentStore interface in chanstate/interface.go. The method signature takes a channel, an update number, and returns a RevocationLog, ChannelCommitment, and error. No implementation is added or modified; the commit message states the existing ChannelStateDB method already satisfies this new interface method. This is a pure interface contract extension with no functional or security impact.
Changed components
chanstate/interface.goInspect captured patch +5 / −0
diff --git a/chanstate/interface.go b/chanstate/interface.go
index 8730184..c0a3b67 100644
--- a/chanstate/interface.go
+++ b/chanstate/interface.go
@@ -277,6 +277,11 @@ type OpenChannelCommitmentStore[Channel any] interface {
// RemoteRevocationStore returns the most up to date commitment version
// of the revocation storage tree for the remote party.
RemoteRevocationStore(channel Channel) (shachain.Store, error)
+
+ // FindPreviousState scans through the append-only log in an attempt to
+ // recover the previous channel state indicated by the update number.
+ FindPreviousState(channel Channel, updateNum uint64) (
+ *RevocationLog, *ChannelCommitment, error)
}
// OpenChannelFwdPkgStore owns forwarding packages tied to open channel records.
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.