test: Add missing suppression signed-integer-overflow:addrman.cpp #22925

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2109-testSupFuzz changing 1 files +2 −0
  1. MarcoFalke commented at 5:46 pm on September 8, 2021: member

    Steps to reproduce:

    crash-d5f88bd8d0d460ffbab217b856b8582600c00503.log

     0$ FUZZ=addrman ./src/test/fuzz/fuzz ./crash-d5f88bd8d0d460ffbab217b856b8582600c00503.log 
     1INFO: Running with entropic power schedule (0xFF, 100).
     2INFO: Seed: 1257085025
     3INFO: Loaded 1 modules   (379531 inline 8-bit counters): 379531 [0x562577b768a8, 0x562577bd3333), 
     4INFO: Loaded 1 PC tables (379531 PCs): 379531 [0x562577bd3338,0x56257819dbe8), 
     5./src/test/fuzz/fuzz: Running 1 inputs 1 time(s) each.
     6Running: ./crash-d5f88bd8d0d460ffbab217b856b8582600c00503.log
     7addrman.cpp:80:14: runtime error: signed integer overflow: 2105390 - -9223372036854775808 cannot be represented in type 'long'
     8    [#0](/bitcoin-bitcoin/0/) 0x5625752f0179 in CAddrInfo::IsTerrible(long) const addrman.cpp:80:14
     9    [#1](/bitcoin-bitcoin/1/) 0x56257531917d in CAddrMan::GetAddr_(std::vector<CAddress, std::allocator<CAddress> >&, unsigned long, unsigned long, std::optional<Network>) const addrman.cpp:874:16
    10    [#2](/bitcoin-bitcoin/2/) 0x562574f0251b in CAddrMan::GetAddr(unsigned long, unsigned long, std::optional<Network>) const ./addrman.h:259:9
    11    [#3](/bitcoin-bitcoin/3/) 0x562574eff7ad in addrman_fuzz_target(Span<unsigned char const>) test/fuzz/addrman.cpp:295:26
    12
    13
    14
    15SUMMARY: UndefinedBehaviorSanitizer: signed-integer-overflow addrman.cpp:80:14 in 
    
  2. test: Add missing suppression signed-integer-overflow:addrman.cpp facb534c37
  3. DrahtBot added the label Tests on Sep 8, 2021
  4. fanquake requested review from practicalswift on Sep 9, 2021
  5. practicalswift commented at 8:08 am on September 9, 2021: contributor
    cr ACK facb534c37725ca446fd56d781b70ba26508bd2a
  6. MarcoFalke merged this on Sep 9, 2021
  7. MarcoFalke closed this on Sep 9, 2021

  8. MarcoFalke deleted the branch on Sep 9, 2021
  9. sidhujag referenced this in commit bc3d945299 on Sep 11, 2021
  10. DrahtBot locked this on Oct 30, 2022


MarcoFalke practicalswift


practicalswift

Labels
Tests


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: 2024-12-19 09:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me