rust-bitcoin / rust-bitcoin / 24566284768

82%

master: 83%

Build Type

push

github

Committed by apoelstra

Commit Message

Merge rust-bitcoin/rust-bitcoin#5921: backport: consensus: Add CompactSize range check to deserialization

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/4a71193a953d9f7280dac2be9f0df83e79d05f7a">4a71193a9<a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/9d5cae957a522ce349d151e954b250db56aaf322"> Remove doc_auto_cfg (Tobin C. Harding)
<a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/91a6e640ad8f37b697669f9e993106afeab23fd7">91a6e640a</a><a href="https://github.com/rust-bitcoin/rust-bitcoin/commit/9d5cae957a522ce349d151e954b250db56aaf322"> consensus: Add CompactSize range check to deserialization (Tobin C. Harding)
</a><a class="double-link" href="https://github.com/rust-bitcoin/rust-bitcoin/commit/48ac68d7dfaa1b6166030d6ff47d4b6f2a1732e5">48ac68d7d</a> Update nightly toolchain version (Tobin C. Harding)

Pull request description:

  Manually backport #5697. On `master` the `VarInt` type is gone but the deserialization bug fixed by 5697 exists here none the less.
      
  Add the same range check to `Decodable for VarInt`.


ACKs for top commit:
  apoelstra:
    ACK 4a71193a953d9f7280dac2be9f0df83e79d05f7a; successfully ran local tests


Tree-SHA512: c46ed1b909631f45421ae7bb74a67fef6c5

Coverage Stats

22 of 25 new or added lines in 6 files covered. (88.0%)

9 existing lines in 3 files now uncovered.

16488 of 20153 relevant lines covered (81.81%)

17692.35 hits per line