Skip to content

tests: Add fuzzing harness for V1TransportDeserializer (P2P transport) #17771

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

Merged

Conversation

practicalswift
Copy link
Contributor

@practicalswift practicalswift commented Dec 18, 2019

Add fuzzing harness for V1TransportDeserializer (P2P transport).

Testing this PR

Run:

$ make distclean
$ ./autogen.sh
$ CC=clang CXX=clang++ ./configure --enable-fuzz \
      --with-sanitizers=address,fuzzer,undefined
$ make
$ src/test/fuzz/p2p_transport_deserializer
…

@fanquake fanquake added the Tests label Dec 18, 2019
@practicalswift practicalswift force-pushed the fuzzers-p2p_transport_deserializer branch 2 times, most recently from 567176a to 38c6fde Compare December 18, 2019 20:52
@DrahtBot
Copy link
Contributor

DrahtBot commented Dec 19, 2019

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

No conflicts as of last run.

@practicalswift practicalswift force-pushed the fuzzers-p2p_transport_deserializer branch from 38c6fde to 982322b Compare January 21, 2020 07:08
@practicalswift practicalswift force-pushed the fuzzers-p2p_transport_deserializer branch from 982322b to 200d4cf Compare January 21, 2020 11:56
@practicalswift practicalswift force-pushed the fuzzers-p2p_transport_deserializer branch from 200d4cf to 2f63ffd Compare January 22, 2020 13:08
@practicalswift
Copy link
Contributor Author

@MarcoFalke Would you mind reviewing? :)

@maflcko
Copy link
Member

maflcko commented Feb 27, 2020

ACK 2f63ffd

@maflcko maflcko merged commit 324a6df into bitcoin:master Feb 27, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 28, 2020
…alizer (P2P transport)

2f63ffd tests: Add fuzzing harness for V1TransportDeserializer (P2P transport) (practicalswift)

Pull request description:

  Add fuzzing harness for `V1TransportDeserializer` (P2P transport).

  **Testing this PR**

  Run:

  ```
  $ make distclean
  $ ./autogen.sh
  $ CC=clang CXX=clang++ ./configure --enable-fuzz \
        --with-sanitizers=address,fuzzer,undefined
  $ make
  $ src/test/fuzz/p2p_transport_deserializer
  …
  ```

ACKs for top commit:
  MarcoFalke:
    ACK 2f63ffd

Tree-SHA512: 8507d4a0414d16f1b8cc9649e3e638f74071dddc990d7e5d7e6faf77697f50bdaf133e49e2371edd29068a069a074469ef53148c6bfc9950510460b81d87646a
jasonbcox pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Nov 3, 2020
Summary:
Backport of core [[bitcoin/bitcoin#17771 | PR17771]].

The missing corpus change is not relevant for us.

Depends on D8228.

Test Plan:
  ninja bitcoin-fuzzers
  ./test/fuzz/test_runner.py <path_to_corpus>

Reviewers: #bitcoin_abc, PiRK

Reviewed By: PiRK

Differential Revision: https://reviews.bitcoinabc.org/D8229
sidhujag pushed a commit to syscoin-core/syscoin that referenced this pull request Nov 10, 2020
…alizer (P2P transport)

2f63ffd tests: Add fuzzing harness for V1TransportDeserializer (P2P transport) (practicalswift)

Pull request description:

  Add fuzzing harness for `V1TransportDeserializer` (P2P transport).

  **Testing this PR**

  Run:

  ```
  $ make distclean
  $ ./autogen.sh
  $ CC=clang CXX=clang++ ./configure --enable-fuzz \
        --with-sanitizers=address,fuzzer,undefined
  $ make
  $ src/test/fuzz/p2p_transport_deserializer
  …
  ```

ACKs for top commit:
  MarcoFalke:
    ACK 2f63ffd

Tree-SHA512: 8507d4a0414d16f1b8cc9649e3e638f74071dddc990d7e5d7e6faf77697f50bdaf133e49e2371edd29068a069a074469ef53148c6bfc9950510460b81d87646a
@practicalswift practicalswift deleted the fuzzers-p2p_transport_deserializer branch April 10, 2021 19:39
kwvg added a commit to kwvg/dash that referenced this pull request Feb 27, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Feb 27, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Feb 28, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Feb 28, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Feb 28, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Mar 13, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Mar 24, 2022
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

4 participants