Skip to content

askrene: exclude stubchannels from routing - #9422

Open
niftynei wants to merge 1 commit into
ElementsProject:masterfrom
niftynei:nifty/askrene-stub-crash
Open

askrene: exclude stubchannels from routing#9422
niftynei wants to merge 1 commit into
ElementsProject:masterfrom
niftynei:nifty/askrene-stub-crash

Conversation

@niftynei

Copy link
Copy Markdown
Collaborator

My node has some recovered channels, which were causing a crash in askrene, due to the fact that they have duplicate scids.

Easy fix is simply to skip recovered stubs when planning routes.

Aug 15 16:34:14 citrine lightningd[1976574]: cln-askrene: ./plugins/askrene/child/additional_costs.h:26: additional_cost_htable_add: Assertion `!additional_c ost_htable_getmatch_(ht, k, h, v, &i)' failed.

@niftynei
niftynei requested a review from Lagrang3 as a code owner August 15, 2026 22:18
My node has some recovered channels, which were causing a crash
in askrene, due to the fact that they have duplicate scids.

Easy fix is simply to skip recovered stubs when planning
routes.

Aug 15 16:34:14 citrine lightningd[1976574]: cln-askrene: ./plugins/askrene/child/additional_costs.h:26: additional_cost_htable_add: Assertion `!additional_c    ost_htable_getmatch_(ht, k, h, v, &i)' failed.

Changelog-Fixed: Crash when trying to pay and you have recovered channel stubs in your peers list
@niftynei
niftynei force-pushed the nifty/askrene-stub-crash branch from c1ed0d3 to 7fac551 Compare August 15, 2026 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants