Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fuzz: simplify fuzzers dependencies in CIFuzz #1896

Merged
merged 1 commit into from
Mar 14, 2023

Commits on Mar 13, 2023

  1. fuzz: simplify fuzzers dependencies in CIFuzz

    CIFuzz (based on oss-fuzz) is the GitHub action/CI job that runs fuzz
    targets on pull requests. It only runs the fuzzers affected by a pull
    request/commit. Otherwise it will divide up the allotted fuzzing time
    among all fuzzers in the project.
    Since:
    * we have more than 20 fuzzers and most of them use the custom memory
    allocation functions (to force allocation failures) even if they are not
    strictly about DPI stuff;
    * we need to keep fuzzing time relatively small (to avoid waiting the CI
    results for a long time)
    
    it is important that fuzzers dependencies (which are based on *files*
    changed by the single commit/PR) are as small as possible.
    
    Bottom line: move all the low-level allocation callbacks to a dedicated
    file; this way most of the fuzzers don't depend anymore on `ndpi_main.c`
    file (which is touched by ever commit/PR).
    
    The goal is to have only the "most important" fuzzers running during (most
    of) the CI.
    IvanNardi committed Mar 13, 2023
    Configuration menu
    Copy the full SHA
    68dd5bc View commit details
    Browse the repository at this point in the history