Skip to content

lightningd: fix reference to uninitialized channel_type during opening. - #7031

Merged
cdecker merged 3 commits into
ElementsProject:masterfrom
rustyrussell:fix-channel-type
Feb 2, 2024
Merged

lightningd: fix reference to uninitialized channel_type during opening.#7031
cdecker merged 3 commits into
ElementsProject:masterfrom
rustyrussell:fix-channel-type

Conversation

@rustyrussell

Copy link
Copy Markdown
Contributor

Not sure how this got through CI...

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell added Type::Bug An error, flaw, or fault that produces an incorrect or unexpected result crash labels Feb 2, 2024
@rustyrussell rustyrussell added this to the v24.02 milestone Feb 2, 2024
@cdecker

cdecker commented Feb 2, 2024

Copy link
Copy Markdown
Member

ACK c6c879a

It can change the node_announcement size, so don't check that.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
```
2024-02-02T10:30:38.6479143Z __________________ ERROR at teardown of test_multifunding_one __________________
...
2024-02-02T10:30:38.6491895Z >           raise ValueError(str(errors))
2024-02-02T10:30:38.6492208Z E           ValueError:
2024-02-02T10:30:38.6492465Z E           Node errors:
2024-02-02T10:30:38.6492833Z E            - lightningd-2: had bad gossip messages
2024-02-02T10:30:38.6493535Z E           Global errors:
...
2024-02-02T10:30:38.7458545Z lightningd-2 2024-02-02T10:25:48.410Z DEBUG   0266e4598d1d3c415f572a8488830b60f7e744ed9235eb0b1ba93283b315c03518-gossipd: Ignoring future channel_announcment for 103x2x1 (current block 102)
2024-02-02T10:30:38.7460140Z lightningd-2 2024-02-02T10:25:48.411Z UNUSUAL lightningd: Bad gossip order: could not find channel 103x2x1 for peer's channel update
```

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@cdecker
cdecker merged commit 3c9fd0c into ElementsProject:master Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crash Type::Bug An error, flaw, or fault that produces an incorrect or unexpected result

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants