fuzz: Avoid OOM in system fuzz target #22649

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2108-fuzzSystemOOM changing 2 files +5 −1
  1. MarcoFalke commented at 10:33 am on August 6, 2021: member

    If the inputs size is unlimited, the target may consume unlimited memory, because the argsmanager stores the argument names. Limiting the size should fix this issue.

    Should fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=36906

  2. fuzz: Avoid OOM in system fuzz target fa7718344d
  3. MarcoFalke commented at 10:33 am on August 6, 2021: member
    The macro idea is stolen from #22508
  4. DrahtBot added the label Tests on Aug 6, 2021
  5. fanquake requested review from practicalswift on Aug 12, 2021
  6. practicalswift commented at 6:17 pm on August 15, 2021: contributor

    cr ACK fa7718344d2879bb3f3c00a4185c5445390c017d

    Two non-blocking explicitness nits for the LIMITED_WHILE macro if a touch up is made for other reasons:

    • Explicit unsigned int _count instead of equivalent but potentially confusing unsigned _count
    • Explicit _count != 0 instead of equivalent implicit int-to-bool conversion

    :)

  7. practicalswift approved
  8. MarcoFalke merged this on Aug 16, 2021
  9. MarcoFalke closed this on Aug 16, 2021

  10. MarcoFalke deleted the branch on Aug 16, 2021
  11. sidhujag referenced this in commit ec91dd71ba on Aug 20, 2021
  12. DrahtBot locked this on Aug 18, 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 15:12 UTC

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