rust-bitcoin / rust-bitcoin / 13678650844

83%

push

github

Merge rust-bitcoin/rust-bitcoin#4185: fuzz: cover more stuff for Script

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/08e0d4f0e53d97edd655ec1103d60a2b4715fb91">08e0d4f0e<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16860fefd261ab3ed6e7be7cc25b5cdc98d24fba"> fuzz: cover minimal_non_dust_custom  for Script (Bruno Garcia)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/66fee1ef8722e4ada86c9f5493d0ec5aed1a816b">66fee1ef8</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16860fefd261ab3ed6e7be7cc25b5cdc98d24fba"> fuzz: add consume_u64 (Bruno Garcia)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/35e7027a083ddfe7674567ab60f9b7d6cf3d0bef">35e7027a0</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16860fefd261ab3ed6e7be7cc25b5cdc98d24fba"> fuzz: move consume_random_bytes to a util file (Bruno Garcia)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/eb8ecd5e3cf42bb01dbf6e16422da7b4ddc78b29">eb8ecd5e3</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/16860fefd261ab3ed6e7be7cc25b5cdc98d24fba"> fuzz: cover minimal_non_dust for Script (Bruno Garcia)
cab8a6134 fuzz: cover count_sigops{_legacy} for Script (Bruno Garcia)

Pull request description:

  This PR adds fuzz coverage for `count_sigops`, `count_sigops_legacy`, `minimal_non_dust` and `minimal_non_dust_custom`. In order to not duplicate `consume_random_bytes`, it moves it to a util file and adds a `consume_u64` function to be used in the `deserialize_script` target to fuzz `minimal_non_dust_custom`.

ACKs for top commit:
  apoelstra:
    ACK 08e0d4f0e53d97edd655ec1103d60a2b4715fb91; successfully ran local tests

Tree-SHA512: d9b0b94afc22babb914f89e2db133b0eb4b

21541 of 25953 relevant lines covered (83.0%)

16773.98 hits per line