tests: Add fuzzing harness for AS-mapping (asmap) #18029

pull practicalswift wants to merge 2 commits into bitcoin:master from practicalswift:fuzzers-asmap changing 4 files +36 −1
  1. practicalswift commented at 4:11 pm on January 30, 2020: contributor

    Add fuzzing harness for AS-mapping (asmap).

    To test this PR:

    0$ make distclean
    1$ ./autogen.sh
    2$ CC=clang CXX=clang++ ./configure --enable-fuzz \
    3      --with-sanitizers=address,fuzzer,undefined
    4$ make
    5$ src/test/fuzz/asmap
    6
  2. tests: Add fuzzing harness for AS-mapping (asmap) 8d07706985
  3. tests: Add fuzzer asmap to FUZZERS_MISSING_CORPORA (temporarily) 4d2aceaad8
  4. DrahtBot added the label Build system on Jan 30, 2020
  5. DrahtBot added the label P2P on Jan 30, 2020
  6. DrahtBot added the label Tests on Jan 30, 2020
  7. MarcoFalke commented at 5:15 pm on January 30, 2020: member
    ACK 4d2aceaad8d28a54246b6639966e2278d2d795e3
  8. MarcoFalke removed the label Build system on Jan 30, 2020
  9. MarcoFalke removed the label P2P on Jan 30, 2020
  10. in src/netaddress.h:42 in 4d2aceaad8
    38@@ -39,7 +39,6 @@ class CNetAddr
    39         explicit CNetAddr(const struct in_addr& ipv4Addr);
    40         void SetIP(const CNetAddr& ip);
    41 
    42-    private:
    


    jonatack commented at 10:27 am on February 1, 2020:
    public: at line 48 now redundant, can be removed as well?
  11. jonatack commented at 10:39 am on February 1, 2020: member

    ACK 4d2aceaad8d28a54246b6639966e2278d2d795e3

    Code review, test. Fuzz output: https://gist.github.com/jonatack/5d78f682e0b245455192567d6af12b7e

    PR #17812 adds test coverage as well (review beg).

  12. jonatack commented at 6:16 am on February 2, 2020: member

    Tested #18023 for 18 hours and 72M execs with this fuzzer (before that patch it found a heap buffer overflow immediately, as shown in https://gist.github.com/jonatack/5d78f682e0b245455192567d6af12b7e):

    0[#72474660](/bitcoin-bitcoin/72474660/) REDUCE cov: 1198 ft: 3529 corp: 197/12792b exec/s: 1063 rss: 452Mb L: 41/1527 MS: 4 InsertByte-EraseBytes-InsertByte-PersAutoDict- DE: "\x00 \x00\x00\x00\x00\x00\x00"-
    
  13. laanwj referenced this in commit 554d89fb29 on Feb 5, 2020
  14. laanwj merged this on Feb 5, 2020
  15. laanwj closed this on Feb 5, 2020

  16. sidhujag referenced this in commit ff8f75b913 on Feb 9, 2020
  17. jasonbcox referenced this in commit 87080a256f on Nov 1, 2020
  18. sidhujag referenced this in commit 03b5a5b784 on Nov 10, 2020
  19. practicalswift deleted the branch on Apr 10, 2021
  20. random-zebra referenced this in commit 85f000ecbe on Jul 30, 2021
  21. kittywhiskers referenced this in commit bdcbd0fbdf on Feb 27, 2022
  22. kittywhiskers referenced this in commit e344d147ab on Feb 27, 2022
  23. kittywhiskers referenced this in commit 0111804cf8 on Feb 28, 2022
  24. kittywhiskers referenced this in commit 6f554a0458 on Feb 28, 2022
  25. kittywhiskers referenced this in commit f7ee0efc26 on Feb 28, 2022
  26. kittywhiskers referenced this in commit ad5c4d4651 on Mar 13, 2022
  27. kittywhiskers referenced this in commit c4491309dc on Mar 24, 2022
  28. DrahtBot locked this on Aug 16, 2022

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-11-17 12:12 UTC

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