fuzz: Check for addrv1 compatibility before using addrv1 serializer/deserializer on CSubNet #20355

pull practicalswift wants to merge 2 commits into bitcoin:master from practicalswift:fix-sub_net_deserialize changing 1 files +28 −10
  1. practicalswift commented at 3:36 PM on November 9, 2020: contributor

    Check for addrv1 compatibility before using addrv1 serializer/deserializer on CSubNet. As requested by MarcoFalke in #20289 (comment).

    Assert roundtrip equality for both addrv1 and addrv2 versions of CService.

  2. fuzz: Check for addrv1 compatibility before using addrv1 serializer/deserializer on CSubNet 0e3a78a8ab
  3. fuzz: Assert roundtrip equality for both addrv1 and addrv2 versions of CService 79b8f8d574
  4. practicalswift cross-referenced this on Nov 9, 2020 from issue fuzz: Check for addrv1 compatibility before using addrv1 serializer/deserializer on CService by practicalswift
  5. MarcoFalke commented at 4:49 PM on November 9, 2020: member

    review ACK 79b8f8d5749fc676d9eeca0296172b7cdbbf2238

  6. DrahtBot added the label Tests on Nov 9, 2020
  7. MarcoFalke merged this on Nov 10, 2020
  8. MarcoFalke closed this on Nov 10, 2020

  9. sidhujag referenced this in commit 929abcfe2d on Nov 10, 2020
  10. practicalswift deleted the branch on Apr 10, 2021
  11. bitcoin locked this on Aug 18, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-20 06:54 UTC