CAddrMan::Deserialize handle corrupt serializations better. #7932

pull pstratem wants to merge 1 commits into bitcoin:master from pstratem:2016-04-23-addrman-deserialize-limits changing 1 files +8 −0

pstratem commented at 10:01 AM on April 24, 2016: contributor

No description provided.
CAddrMan::Deserialize handle corrupt serializations better. fb26bf0ea3
jonasschnelli added the label P2P on Apr 24, 2016
laanwj commented at 10:47 AM on April 25, 2016: member

utACK https://github.com/bitcoin/bitcoin/pull/7932/commits/fb26bf0ea3822638b10a783f054c280fc053a2b5
sipa commented at 5:11 PM on May 5, 2016: member

Does this interact with #7696?
pstratem commented at 1:07 AM on May 17, 2016: contributor

@sipa I believe these are mutually beneficial and not conflicting.
sipa commented at 8:34 AM on May 17, 2016: member

utACK fb26bf0ea3822638b10a783f054c280fc053a2b5
laanwj merged this on May 18, 2016
laanwj closed this on May 18, 2016
laanwj referenced this in commit ed749bdb64 on May 18, 2016
sickpig referenced this in commit 6239eb6a90 on May 2, 2017
sickpig cross-referenced this on May 2, 2017 from issue [ported] Core PR 8070 and PR 7932 by sickpig
dgenr8 cross-referenced this on May 18, 2017 from issue Networking improvements by dgenr8
codablock referenced this in commit 211e0acae9 on Sep 16, 2017
codablock referenced this in commit b379f99932 on Sep 19, 2017
codablock referenced this in commit 0450dfdeb8 on Dec 21, 2017
furszy cross-referenced this on Jun 7, 2021 from issue Road to Tor v3 support (BIP155) by furszy
furszy cross-referenced this on Jul 15, 2021 from issue [Net] asmap to improve IP bucketing in addrman - backports by furszy
random-zebra referenced this in commit 85f000ecbe on Jul 30, 2021
random-zebra referenced this in commit b4751e10ce on Aug 11, 2021
bitcoin locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#7696 Fix de-serialization bug where AddrMan is left corrupted