This PR is a (quite late) follow-up to #19237 (https://github.com/bitcoin/bitcoin/pull/19237#issuecomment-642203251). Looking at CPubKey::Serialize and CPubKey::Unserialize I can't think of a scenario where the roundtrip (serialization/deserialization) equality wouldn't hold.
fuzz: Assert roundtrip equality for `CPubKey` #22271
pull theStack wants to merge 1 commits into bitcoin:master from theStack:202106-fuzz-assert-CPubKey-deser-roundtrip changing 1 files +1 −2-
theStack commented at 3:20 PM on June 17, 2021: contributor
-
fuzz: Assert roundtrip equality for `CPubKey` 9550dffa0c
-
theStack commented at 3:20 PM on June 17, 2021: contributor
ping @practicalswift
- DrahtBot added the label Tests on Jun 17, 2021
-
jamesob commented at 4:00 PM on June 17, 2021: member
- MarcoFalke merged this on Jun 17, 2021
- MarcoFalke closed this on Jun 17, 2021
-
practicalswift commented at 7:46 PM on June 17, 2021: contributor
Post-merge cr ACK 9550dffa0c61df6d1591c62d09629b4c5731e1b7
Thanks for fixing this @theStack!
- practicalswift cross-referenced this on Jun 17, 2021 from issue Serialization-deserialization roundtrip of CPubKey does not necessarily result in an equal object by practicalswift
- sidhujag referenced this in commit 1d67f858c6 on Jun 18, 2021
- theStack deleted the branch on Jul 31, 2021
- gwillen referenced this in commit 94bc5cb7f5 on Jun 1, 2022
- bitcoin locked this on Aug 16, 2022
