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.

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

Add cargo-fuzz fuzzer #29

Closed
wants to merge 11 commits into from
Closed

Add cargo-fuzz fuzzer #29

wants to merge 11 commits into from

Conversation

5225225
Copy link
Contributor

@5225225 5225225 commented Aug 21, 2022

This adds a https://github.com/rust-fuzz/cargo-fuzz fuzzer which I used to find #28

madadam and others added 11 commits October 25, 2021 08:01
By explicitly implementing `deserialize_str` which calls `visit_str` instead of `visit_bytes`.
By explicitly implementing `deserialize_identifier` to delegate to `deserialize_str`.
Fix deserialization of nested tuple
Minor change to support rust version >= 1.40.0
Add test for ser/de of struct with vec of tuples (issue toby#17)
josecelano added a commit to torrust/torrust-serde-bencode-archive that referenced this pull request Sep 25, 2023
a1597cc Add cargo-fuzz fuzzer (5225225)

Pull request description:

  From: toby#29
  By: [5225225](https://github.com/5225225)

  `This adds a https://github.com/rust-fuzz/cargo-fuzz fuzzer which I used to find toby#28`

ACKs for top commit:
  josecelano:
    ACK a1597cc

Tree-SHA512: 45c604038a0e48b3057612ff4a28ffce1f4985f94c8fc3807d08daff8e94f7f40173bb690296619f928075101d2712abd984ec27849d12621a72606113baa501
@josecelano
Copy link
Collaborator

Hi @5225225, this was also added to https://github.com/torrust/torrust-serde-bencode. But to be honest I do not know what this is for yet. My plan is:

  • Fork the repo.
  • Fix bugs.
  • Review and fix tests.
  • Update dependencies.
  • Publish a new upgraded/fixed version.

@josecelano
Copy link
Collaborator

Hi @5225225, It's finally merged into this repo.

@josecelano josecelano closed this Oct 9, 2023
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.

3 participants