Skip to content
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.

Sign up for GitHub

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

Jump to bottom

tests: Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot). Increase fuzzing coverage. #18314

Merged
merged 12 commits into from
Mar 11, 2020
Merged

tests: Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot). Increase fuzzing coverage. #18314

merged 12 commits into from
Mar 11, 2020

Conversation

practicalswift
Copy link
Contributor

Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot).

Increase fuzzing coverage.

@practicalswift
tests: Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot…
7b169ca 
…), uint160 and uint256
@practicalswift
tests: Remove unit test from fuzzing harness
516cc6f
@practicalswift
tests: Re-arrange test cases in parse_univalue to increase coverage
46ef4cf
@practicalswift
tests: Fuzz currently uncovered code path in TxToUniv(...)
9f8d74a
@practicalswift
tests: Increase fuzzing coverage of DecompressScript(...)
c2c58f6
@practicalswift
tests: Fuzz operator!= of CService
81b58a3
@practicalswift
tests: Fuzz RecursiveDynamicUsage(const std::shared_ptr<X>& p)
117a706
@practicalswift
tests: Fuzz DecodeHexBlk(...)
e57e670
@practicalswift
tests: Simplify code by removing unwarranted use of unique_ptr:s
d3d4892
@practicalswift
tests: Fuzz DecodeBase64PSBT(...)
47a2631
@practicalswift
tests: Fuzz HasAllDesirableServiceFlags(...) and MayHaveUsefulAddress…
7a861a6 
…DB(...)
@practicalswift
tests: Add fuzzing of CSubNet, CNetAddr and CService related functions
08eab0f
@fanquake fanquake added the Tests label Mar 11, 2020
@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.

@DrahtBot DrahtBot mentioned this pull request Mar 11, 2020
Merged
@practicalswift
Copy link
Contributor Author

practicalswift commented Mar 11, 2020 •
edited
Loading

Travis failure seems unrelated :)

@maflcko
Copy link
Member

maflcko commented Mar 11, 2020

ACK 08eab0f 🗾

Show signature and timestamp

Signature:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

ACK 08eab0f599 🗾
-----BEGIN PGP SIGNATURE-----

iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYEACgkQzit1aX5p
pUiIfwv6AoWRE7OkwYdz5rXuDDWtKmW2Wqoci64n3h26fdV5qu++VbBsmmr3Lifu
laEQGrgAGfwmN5eAhiRteE1p3vdAJ8zFfBrn3Up0yn/1x7EL52ykzg9VWLoRaS4s
2cNiKtq4hsx1zxrILUVl4N5L1Fzz6fb1WxVzwcj8vME+ieE5nrdmjzUBHSfaKMoz
sLAcwGwWNhNAs3IZnaZhiI8Mim/55vu8qiEZQNaXX0sxh2NWWX0wmH0uB2MFb436
mx8+TNTcwRoItKK/orGAFiRlhq6uqUx/ekQDBVSe+iKkYwP+EtJwL/hOJr3YVCa2
UIecr1Fah/1/OTcplwnN3suVeKOkXprBhRFYGTXlUTNe6n81+gCOQhW8Mrllplf6
43KbN277GbFMySwm/UqM1yeSjfcPr2hSoeVI2+P+Wj/Z+N7ZciUrwy9o5AjNOAT3
Ihu21c2FAI8HuUW1FPFO6Pu1QmeTgDC/X4qXf35hEt2oO97Sdf3seJLWNP4Bl77v
SLjvlKpc
=lckd
-----END PGP SIGNATURE-----

Timestamp of file with hash 0cf36490d2b68e3f8cb34a7d24aeb3f7323132d809b20e7ed1744b6c48e5988f -

@maflcko maflcko merged commit 249114b into bitcoin:master Mar 11, 2020
@MarkLTZ MarkLTZ mentioned this pull request Apr 4, 2020
Closed
Fabcien pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Dec 9, 2020
@practicalswift @Fabcien
tests: Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot…
7ef9597 
…). Increase fuzzing coverage.

Summary:
```
Add deserialization fuzzing of SnapshotMetadata (utxo_snapshot).

Increase fuzzing coverage.
```

Backport of core [[bitcoin/bitcoin#18314 | PR18314]].

The commit 117a706faba586f2095f97cf630b709b3e29a947:
bitcoin/bitcoin@117a706
is not ported as this function has been removed from our codebase (was
dead code, see D5992.

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

Reviewers: #bitcoin_abc, deadalnix

Reviewed By: #bitcoin_abc, deadalnix

Differential Revision: https://reviews.bitcoinabc.org/D8636
@practicalswift practicalswift deleted the fuzzers-deserialize-more branch April 10, 2021 19:40
kwvg added a commit to kwvg/dash that referenced this pull request May 7, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
bc311e5 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request May 7, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
3d87918 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
08d8545 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
b5e69b9 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jun 14, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
581a85d 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jun 18, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
5ac75aa 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jun 18, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
278f0d2 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 4, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
bd3b8d3 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 4, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
3c32c01 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
9753c51 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
6ee33ba 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 6, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
75fa4ba 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 13, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
b2647b6 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 13, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
3d3f272 
…a (utxo_snapshot). Increase fuzzing coverage
kwvg added a commit to kwvg/dash that referenced this pull request Jul 15, 2022
@kwvg
partial bitcoin#18314: Add deserialization fuzzing of SnapshotMetadat…
a7109d8 
…a (utxo_snapshot). Increase fuzzing coverage
PastaPastaPasta added a commit to dashpay/dash that referenced this pull request Jul 17, 2022
@PastaPastaPasta
Merge pull request #4886 from kittywhiskers/fuzz3
30d6584 
backport: bitcoin#18867, bitcoin#19247, bitcoin#19222, bitcoin#18363, bitcoin#18190, bitcoin#18393, partial bitcoin#18047, bitcoin#18314, bitcoin#19143 (fuzzing harness backports: part 3)
@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.
Reviewers
No reviews
Assignees
No one assigned
Labels
Tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@practicalswift @DrahtBot @maflcko @fanquake