AI-generated analysisPublished automatically and not human-verified. Validated context appears in community notes below.
← Watch feed
Informational 18 Bitcoin

lightningd: save short_channel_id in inflight struct as soon as it gets mined.

Public commit record

What the developer wrote

Authored by Rusty Russell

73/100 · Adequate
lightningd: save short_channel_id in inflight struct as soon as it gets mined.

This also removed the last call to wallet_transaction_locate, so remove that too.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
The short version

What changed, and why it matters

This is a small internal cleanup change in Core Lightning. It stores a channel identifier (short_channel_id) in memory as soon as a splicing transaction is mined, instead of looking it up from the database every time it is needed. It also removes an unused database helper function. There is no obvious security vulnerability here; it is primarily a code-quality and efficiency improvement.

Recommended action

No security action required. Treat as normal code review/merge for maintainability. If auditing, verify that `inflight->scid` is always set before use in `splice_depth_cb()` and that the reorg/free path cannot lead to a use-after-free or stale pointer.

Security signals we found

01

No direct security signal: change is a refactoring/caching improvement.

02

Removal of a database lookup path reduces attack surface slightly, but no vulnerability is identified in the removed code.

03

No input validation changes, no memory safety defects visible in diff, no privilege changes.

Risk score

Why this scored 18/100

Our methodology →
Potential impact 2/30
Exploitability 1/25
Stealth signal 1/15
Affected reach 2/15
Confidence 8/10
Evidence quality 4/5
Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.