tests: Add fuzzing harness for CScript and CScriptNum operations #18176
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1996425 to
0730ac9
Compare
|
ACK 0730ac9 |
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsNo conflicts as of last run. |
e2fb330 to
ab67715
Compare
|
@MarcoFalke Added |
ab67715 to
1be90b4
Compare
This was referenced Feb 21, 2020
1be90b4 to
f137f64
Compare
f137f64 to
7d27b9f
Compare
|
@MarcoFalke Thanks for reviewing. All feedback addressed. Please re-review :) |
7d27b9f to
ad040fc
Compare
|
Rebased again :) |
maflcko
approved these changes
Mar 9, 2020
There was a problem hiding this comment.
ACK ad040fc 👦
Show signature and timestamp
Signature:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512
ACK ad040fc2a503a0a5c7097dfe8aa6d341901436d9 👦
-----BEGIN PGP SIGNATURE-----
iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUgKHwv+OE49ZDbtGMchMWXzOuFyxStj7biNTlNtYQH2AIBvgxPl2zZbczejRMMA
xbUrKDh3SiE+mzPNyzOZ69XejxNGCTX1Kfel22F/XIwbKDw2kFv0v1F39pYyp4ZV
j7/CLVpqZaWdgqcdFvE5tYCBcflm/6bU95TabNkv3yjsZ0XEy7WGaGJTRsgeMoQd
wHIOc6kZerms8G04U/84PtHJBO4x0pU4zk2PRiEv9IcYYkAUFh1NDgIoJ+8MH9bt
uuSjVmOzKmbuc5mrq8VFbvgeHJlIBWtjNP7Ux2OkvUCxGeUDoMOluO4MlME/tZVQ
VJAQ6ZdOHSp032e5aOs9A9rwU+S6SlGxuoBlrjvK5vZm/nTwM8QXTU+hgza8hcdj
7IWPhHIFiEExdYRd31q/aZ7kU4fpp9GaD4jXq+5hiAFso2jkWTxAIH3sdIn0ilKp
+XxdLNNDSevd2KlCZK/DiwtTvU/Hp6WDPG9JW1YG0DS6jV1VeDmoacoN5M1hMYQz
8kqbBKQP
=dRzS
-----END PGP SIGNATURE-----
Timestamp of file with hash 009db7b3b5d4284568143d3ed9fcbb2849ef578b5853b7f040cd0dd8c4f64f5f -
ad040fc to
e37f536
Compare
|
ACK e37f536 🦂 Show signature and timestampSignature: Timestamp of file with hash |
sidhujag
pushed a commit
to syscoin/syscoin
that referenced
this pull request
Mar 11, 2020
…ptNum operations e37f536 Make lifetime correctness easier to see (avoid reference lifetime extension) (practicalswift) e7ddbd9 tests: Add fuzzing harness for CScriptNum operations (practicalswift) 65a52a0 tests: Add fuzzing harness for CScript operations (practicalswift) eb7c50c tests: Add common Consume* fuzzing functions (practicalswift) Pull request description: Add fuzzing harness for `CScript` and `CScriptNum` operations. Test this PR using: ``` $ make distclean $ ./autogen.sh $ CC=clang CXX=clang++ ./configure --enable-fuzz \ --with-sanitizers=address,fuzzer,undefined $ make $ src/test/fuzz/script_ops … $ src/test/fuzz/scriptnum_ops … ``` ACKs for top commit: MarcoFalke: ACK e37f536 🦂 Tree-SHA512: 5165d918ffe3f1e3e85ab0e61d8b05934f682d324cf63ce188da5890899df2b5727aba9ed10c0437260ecff8055250e60c79d81d764bc740a7652d543a7c5fa3
sidhujag
pushed a commit
to syscoin-core/syscoin
that referenced
this pull request
Nov 10, 2020
…ptNum operations e37f536 Make lifetime correctness easier to see (avoid reference lifetime extension) (practicalswift) e7ddbd9 tests: Add fuzzing harness for CScriptNum operations (practicalswift) 65a52a0 tests: Add fuzzing harness for CScript operations (practicalswift) eb7c50c tests: Add common Consume* fuzzing functions (practicalswift) Pull request description: Add fuzzing harness for `CScript` and `CScriptNum` operations. Test this PR using: ``` $ make distclean $ ./autogen.sh $ CC=clang CXX=clang++ ./configure --enable-fuzz \ --with-sanitizers=address,fuzzer,undefined $ make $ src/test/fuzz/script_ops … $ src/test/fuzz/scriptnum_ops … ``` ACKs for top commit: MarcoFalke: ACK e37f536 🦂 Tree-SHA512: 5165d918ffe3f1e3e85ab0e61d8b05934f682d324cf63ce188da5890899df2b5727aba9ed10c0437260ecff8055250e60c79d81d764bc740a7652d543a7c5fa3
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Jan 19, 2021
Summary: ``` Add fuzzing harness for CScript and CScriptNum operations. ``` Backport of core [[bitcoin/bitcoin#18176 | PR18176]]. Test Plan: ninja bitcoin-fuzzers ./test/fuzz/test_runner.py <path_to_corpus> Reviewers: #bitcoin_abc, PiRK Reviewed By: #bitcoin_abc, PiRK Differential Revision: https://reviews.bitcoinabc.org/D8959
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 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
PastaPastaPasta
added a commit
to dashpay/dash
that referenced
this pull request
Jul 12, 2022
backport: bitcoin#18417, bitcoin#18521, bitcoin#18529, bitcoin#18176, bitcoin#18423, bitcoin#17926, bitcoin#18353, bitcoin#18407, bitcoin#18455, bitcoin#18565, bitcoin#18867 (fuzzing harness backports: part 2)
knst
pushed a commit
to knst/dash
that referenced
this pull request
Jul 21, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add fuzzing harness for
CScriptandCScriptNumoperations.Test this PR using: