Skip to content

Various fixes found fuzzing, add subsystem event#18

Merged
mkj merged 19 commits intomainfrom
matt/dev
Jun 11, 2025
Merged

Various fixes found fuzzing, add subsystem event#18
mkj merged 19 commits intomainfrom
matt/dev

Conversation

@mkj
Copy link
Copy Markdown
Owner

@mkj mkj commented Jun 11, 2025

I've been testing a server fuzzer target locally, will push that once tidied up. These are some fixes/improvements found while running it.

mkj added 19 commits June 11, 2025 23:36
To be used in cases of input packets where a failure shouldn't
be a (debug) panic.
Bad peer keys might not have a correct length.
Previously it would present multiple auth events
if the client sent subsequent (in-flight?) requests.
Send an error response, proper handling to be implemented later.
channel number is stored in events now. Rename from ch to num since "ch"
is already used for the channel itself in some places.
Zero length data inputs are now just ignored.
Previously this triggered a debug assertion.
Previously would hit a bug()
Previously the channel would be presented as ready
Once an app calls channel_done() it should not receive any more events
or channel data.
@mkj mkj merged commit cb6c720 into main Jun 11, 2025
1 check passed
@mkj mkj deleted the matt/dev branch June 11, 2025 16:04
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.

1 participant