Signed Integer-overflow in CAddrInfo::IsTerrible #22931

issue MarcoFalke opened this issue on September 9, 2021
  1. MarcoFalke commented at 2:59 PM on September 9, 2021: member

    This may happen when peers.dat is deserialized from disk.

    See:

  2. MarcoFalke added the label Bug on Sep 9, 2021
  3. MarcoFalke added the label P2P on Sep 9, 2021
  4. mzumsande commented at 7:17 PM on September 9, 2021: contributor

    Would be fixed by running Check() after deserialization, as in your PR #22734 .

  5. MarcoFalke cross-referenced this on Sep 10, 2021 from issue fuzz: Reset addrman when consistency check fails by MarcoFalke
  6. MarcoFalke cross-referenced this on Sep 15, 2021 from issue addrman: Avoid crash on corrupt data, Force Check after deserialize by MarcoFalke
  7. MarcoFalke referenced this in commit a8a272ac32 on Sep 21, 2021
  8. MarcoFalke closed this on Sep 21, 2021
  9. sidhujag referenced this in commit 43bc87779b on Sep 21, 2021
  10. bitcoin locked this on Oct 30, 2022
Contributors
MarcoFalkemzumsande
Labels
BugP2P
Linked (view graph)
#22734 addrman: Avoid crash on corrupt data, Force Check after deserialize#22925 test: Add missing suppression signed-integer-overflow:addrman.cpp#22939 fuzz: Reset addrman when consistency check fails
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-19 06:53 UTC