Skip to content

Conversation

practicalswift
Copy link
Contributor

Add fuzzing harness for {Read,Write}{LE,BE}{16,32,64} (crypto/common.h).

See doc/fuzzing.md for information on how to fuzz Bitcoin Core. Don't forget to contribute any coverage increasing inputs you find to the Bitcoin Core fuzzing corpus repo.

Happy fuzzing :)

@maflcko
Copy link
Member

maflcko commented Jun 11, 2020

Any thoughts on clarifying that ConsumeFixedLengthByteVector returns a zero-filled byte vector when there remaining bytes are insufficient?

@practicalswift practicalswift force-pushed the fuzzers-crypto_common branch from 90ea479 to b429e0f Compare June 11, 2020 13:47
@practicalswift
Copy link
Contributor Author

@MarcoFalke Good idea. Added!

@practicalswift practicalswift force-pushed the fuzzers-crypto_common branch from b429e0f to cf5b8f6 Compare June 11, 2020 14:06
@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@maflcko
Copy link
Member

maflcko commented Jun 11, 2020

ACK cf5b8f6

@maflcko maflcko merged commit 8682414 into bitcoin:master Jun 11, 2020
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Feb 1, 2021
…common.h)

Summary: Backport of core [[bitcoin/bitcoin#19247 | PR19247]].

Test Plan:
  ninja bitcoin-fuzzers
  ./src/test/fuzz/crypto_common <path_to_corpus>

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

Differential Revision: https://reviews.bitcoinabc.org/D9117
@practicalswift practicalswift deleted the fuzzers-crypto_common branch April 10, 2021 19:41
kwvg added a commit to kwvg/dash that referenced this pull request May 7, 2022
kwvg added a commit to kwvg/dash that referenced this pull request May 7, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jun 18, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jun 18, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 4, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 4, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 13, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 13, 2022
kwvg added a commit to kwvg/dash that referenced this pull request Jul 15, 2022
PastaPastaPasta added a commit to dashpay/dash that referenced this pull request Jul 17, 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
None yet
Development

Successfully merging this pull request may close these issues.

4 participants