Skip to content

Conversation

practicalswift
Copy link
Contributor

@practicalswift practicalswift commented Feb 15, 2020

Add harness which fuzzes EvalScript and VerifyScript using a fuzzed signature checker.

Test this PR using:

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

Closes #17986.

@DrahtBot
Copy link
Contributor

DrahtBot commented Feb 18, 2020

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-signature_checker branch from 7c38db4 to 1cc9997 Compare March 9, 2020 15:15
@practicalswift
Copy link
Contributor Author

Rebased! :)

@practicalswift
Copy link
Contributor Author

Rebased :)

@practicalswift
Copy link
Contributor Author

Rebased! :)

@practicalswift practicalswift force-pushed the fuzzers-signature_checker branch from aacca1a to 5e47b19 Compare March 17, 2020 19:11
@practicalswift
Copy link
Contributor Author

Rebased :)

@maflcko maflcko merged commit e83a1de into bitcoin:master Mar 18, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 18, 2020
…erifyScript using a fuzzed signature checker

5e47b19 tests: Add harness which fuzzes EvalScript and VerifyScript using a fuzzed signature checker (practicalswift)

Pull request description:

  Add harness which fuzzes `EvalScript` and `VerifyScript` using a fuzzed signature checker.

  Test this PR using:

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

  Closes bitcoin#17986.

Top commit has no ACKs.

Tree-SHA512: a9988f8fa7919fe470756ca3e4e75764a589f590769aab452c8f4c254cf41667793e52131d470a12629ec3681fa7fc20091f371b8f3e3eec105674c2769e7d7e
sidhujag pushed a commit to syscoin-core/syscoin that referenced this pull request Nov 10, 2020
…erifyScript using a fuzzed signature checker

5e47b19 tests: Add harness which fuzzes EvalScript and VerifyScript using a fuzzed signature checker (practicalswift)

Pull request description:

  Add harness which fuzzes `EvalScript` and `VerifyScript` using a fuzzed signature checker.

  Test this PR using:

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

  Closes bitcoin#17986.

Top commit has no ACKs.

Tree-SHA512: a9988f8fa7919fe470756ca3e4e75764a589f590769aab452c8f4c254cf41667793e52131d470a12629ec3681fa7fc20091f371b8f3e3eec105674c2769e7d7e
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Jan 20, 2021
…uzzed signature checker

Summary:
```
Add harness which fuzzes EvalScript and VerifyScript using a fuzzed
signature checker.
```

Backport of core [[bitcoin/bitcoin#18155 | PR18155]].

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

Reviewers: #bitcoin_abc, PiRK

Reviewed By: #bitcoin_abc, PiRK

Differential Revision: https://reviews.bitcoinabc.org/D8968
@practicalswift practicalswift deleted the fuzzers-signature_checker branch April 10, 2021 19:40
vijaydasmp pushed a commit to vijaydasmp/dash that referenced this pull request Aug 14, 2022
…erifyScript using a fuzzed signature checker

5e47b19 tests: Add harness which fuzzes EvalScript and VerifyScript using a fuzzed signature checker (practicalswift)

Pull request description:

  Add harness which fuzzes `EvalScript` and `VerifyScript` using a fuzzed signature checker.

  Test this PR using:

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

  Closes bitcoin#17986.

Top commit has no ACKs.

Tree-SHA512: a9988f8fa7919fe470756ca3e4e75764a589f590769aab452c8f4c254cf41667793e52131d470a12629ec3681fa7fc20091f371b8f3e3eec105674c2769e7d7e
vijaydasmp pushed a commit to vijaydasmp/dash that referenced this pull request Aug 14, 2022
…erifyScript using a fuzzed signature checker

5e47b19 tests: Add harness which fuzzes EvalScript and VerifyScript using a fuzzed signature checker (practicalswift)

Pull request description:

  Add harness which fuzzes `EvalScript` and `VerifyScript` using a fuzzed signature checker.

  Test this PR using:

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

  Closes bitcoin#17986.

Top commit has no ACKs.

Tree-SHA512: a9988f8fa7919fe470756ca3e4e75764a589f590769aab452c8f4c254cf41667793e52131d470a12629ec3681fa7fc20091f371b8f3e3eec105674c2769e7d7e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 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.

fuzz script execution with dummy signature checker
4 participants