Assertion `nRndPos1 < vRandom.size() && nRndPos2 < vRandom.size()' failed #22467

1. 
   vasild commented at 10:37 AM on July 16, 2021: contributor

   The problem was originally reported in #22450 (comment). This is a dedicated issue to track it independently because it is a different problem than the one reported in #22450 OP/description.

   fuzz: addrman.cpp:149: void CAddrMan::SwapRandom(unsigned int, unsigned int): Assertion `nRndPos1 < vRandom.size() && nRndPos2 < vRandom.size()' failed.
   

   To reproduce:

   FUZZ=addrman_deserialize ./src/test/fuzz/fuzz /tmp/crash-ef7d4a4f940b8fdb4999f6a92969260cf8e1405f.log
   

   crash-ef7d4a4f940b8fdb4999f6a92969260cf8e1405f.log

2. vasild added the label Bug on Jul 16, 2021
3. vasild referenced this in commit 8c8cd36329 on Jul 16, 2021
4. vasild cross-referenced this on Jul 16, 2021 from issue addrman: don't overwrite addr_info when resetting I2P ports by vasild
5. MarcoFalke added this to the milestone 22.0 on Jul 16, 2021
6. vasild cross-referenced this on Jul 16, 2021 from issue addrman: detect on-disk corrupted nNew and nTried during unserialization by vasild
7. 
   vasild commented at 4:40 PM on July 16, 2021: contributor

   Fix pending at https://github.com/bitcoin/bitcoin/pull/22468

8. 
   MarcoFalke commented at 7:26 AM on July 17, 2021: member

   OSS-Fuzz duplicate report from today: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=36255

9. jnewbery cross-referenced this on Jul 19, 2021 from issue addrman: Remove addrman hotfixes by jnewbery
10. vasild referenced this in commit fe67f03d0f on Jul 19, 2021
11. vasild cross-referenced this on Jul 19, 2021 from issue scripted-diff: remove ResetI2PPorts() (revert e0a2b390c14) by vasild
12. vasild referenced this in commit d4b67c8ebc on Jul 19, 2021
13. fanquake closed this on Jul 20, 2021

    ---

14. fanquake referenced this in commit 624a193330 on Jul 20, 2021
15. josibake referenced this in commit 4abcf2370c on Jul 21, 2021
16. hebasto referenced this in commit 90df9ac392 on Jul 22, 2021
17. sidhujag referenced this in commit 2a336a660b on Jul 23, 2021
18. fanquake referenced this in commit 5cf28d5203 on Aug 3, 2021
19. jnewbery cross-referenced this on Sep 13, 2021 from issue test: add addpeeraddress "tried", test addrman checks on restart with asmap by jonatack
20. janus referenced this in commit c5da7c0c29 on Nov 5, 2021
21. bitcoin locked this on Aug 18, 2022

Contributors

vasildMarcoFalke

Milestone
22.0