Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a proper fuzzing test for the code in behaviour.rs #544

Open
tomaka opened this issue Dec 7, 2020 · 2 comments · May be fixed by paritytech/substrate#8781
Open

Add a proper fuzzing test for the code in behaviour.rs #544

tomaka opened this issue Dec 7, 2020 · 2 comments · May be fixed by paritytech/substrate#8781
Assignees
@kpp
Labels
D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase.

Comments

@tomaka
Copy link
Contributor

tomaka commented Dec 7, 2020

This 2000 lines of code file is quite complicated and very vulnerable to bugs. We should add some fuzzing code that opens/closes connections and substreams to check whether everything works properly.

cc #552
@tomaka tomaka mentioned this issue May 3, 2021
Merged
@kpp kpp self-assigned this May 11, 2021
@kpp kpp linked a pull request May 11, 2021 that will close this issue
Draft
@stale
Copy link

stale bot commented Jul 7, 2021

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the A5-stale label Jul 7, 2021
@tomaka
Copy link
Contributor Author

tomaka commented Jul 8, 2021

Issue still relevant and important.

@stale stale bot removed the A5-stale label Jul 8, 2021
@altonen altonen transferred this issue from paritytech/substrate Aug 24, 2023
@the-right-joyce the-right-joyce added D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase. and removed U2-some_time_soon labels Aug 25, 2023
@kpp kpp mentioned this issue Aug 30, 2023
Closed
claravanstaden pushed a commit to Snowfork/polkadot-sdk that referenced this issue Dec 8, 2023
quick grammar fix (paritytech#544)
7d7af1d
helin6 pushed a commit to boolnetwork/polkadot-sdk that referenced this issue Feb 5, 2024
@tgmichel
Logs sorted by block number ASC instead DESC (paritytech#544)
8c04e98
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kpp kpp

Labels
D1-medium Can be fixed by a coder with good Rust knowledge but little knowledge of the codebase.
Projects
Networking
Status: Backlog 🗒
Milestone
No milestone
4 participants
@kpp @tomaka @altonen @the-right-joyce