Skip to content

Conversation

practicalswift
Copy link
Contributor

@practicalswift practicalswift commented Feb 21, 2020

Add fuzzing harness for Golomb-Rice coding (GolombRiceEncode/GolombRiceDecode).

Test this PR using:

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

@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 21, 2020

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.

@practicalswift practicalswift force-pushed the fuzzers-golomb-rice branch 5 times, most recently from 93a2edd to 84cdf2a Compare March 6, 2020 17:01
@practicalswift
Copy link
Contributor Author

Rebased :)

@practicalswift
Copy link
Contributor Author

Rebased again :)

@practicalswift
Copy link
Contributor Author

Rebased again :)

@practicalswift
Copy link
Contributor Author

@MarcoFalke Would you mind reviewing? :)

@practicalswift
Copy link
Contributor Author

@MarcoFalke Feedback addressed! Please re-review :)

@maflcko
Copy link
Member

maflcko commented Apr 20, 2020

Can't merge with red travis

@practicalswift
Copy link
Contributor Author

@MarcoFalke Oh, thanks! Header guard now updated after util/ move :)

@maflcko maflcko merged commit c4c3f11 into bitcoin:master Apr 20, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 20, 2020
…g (GolombRiceEncode/GolombRiceDecode)

69749fb tests: Add fuzzing harness for Golomb-Rice coding (GolombRiceEncode/GolombRiceDecode) (practicalswift)

Pull request description:

  Add fuzzing harness for Golomb-Rice coding (`GolombRiceEncode`/`GolombRiceDecode`).

  Test this PR using:

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

Top commit has no ACKs.

Tree-SHA512: 1b26512301b8c22ab3b804d9b9e4baf933f26f8c05e462d583863badcec7e694548a34849a0d7c4ff7d58b19f6338b51819976ecf642bc4659b04ef71182d748
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Jan 21, 2021
…olombRiceDecode)

Summary: Backport of core [[bitcoin/bitcoin#18190 | PR18190]].

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

Reviewers: #bitcoin_abc, majcosta

Reviewed By: #bitcoin_abc, majcosta

Differential Revision: https://reviews.bitcoinabc.org/D9005
@practicalswift practicalswift deleted the fuzzers-golomb-rice branch April 10, 2021 19:40
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants