Skip to content

Conversation

@Chand-ra
Copy link

Changelog-None: handle_onion_message() in connectd/onion_message.c is responsible for handling incoming onion messages from a peer. Since it deals with external input, add a test for it.

Checklist

Before submitting the PR, ensure the following tasks are completed. If an item is not applicable to your PR, please mark it as checked:

  • The changelog has been updated in the relevant commit(s) according to the guidelines.
  • Tests have been added or modified to reflect the changes.
  • Documentation has been reviewed and updated as needed.
  • Related issues have been listed and linked, including any that this PR closes.

@Chand-ra Chand-ra force-pushed the handle_onion branch 2 times, most recently from 5812ca9 to b0ea75e Compare October 1, 2025 14:56
@Chand-ra
Copy link
Author

Chand-ra commented Oct 1, 2025

The branch should now merge cleanly.

@rustyrussell rustyrussell added this to the v25.12 milestone Nov 7, 2025
Chandra Pratap added 2 commits November 10, 2025 15:13
Changelog-None: `handle_onion_message()` in `connectd/onion_message.c`
is responsible for handling incoming onion messages from a peer.
Since it deals with external input, add a test for it.
Add a minimal input set as a seed corpus for the newly introduced
test. This leads to discovery of interesting code paths faster.
@rustyrussell
Copy link
Contributor

Rebased with trivial fix to make it work as a unit test.

@rustyrussell rustyrussell merged commit df3043c into ElementsProject:master Nov 10, 2025
1 of 3 checks passed
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