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

[elfutils] build zlib with MSan #7401

Merged
merged 1 commit into from
Mar 24, 2022

Commits on Mar 23, 2022

  1. [elfutils] build zlib with MSan

    Unlike fuzz-dwfl-core, the new fuzz targets actually use zlib so
    instead of just linking against zlib to make them compile they
    should use the library instrumented with MSan. Without it OSS-Fuzz
    reports bogus issues like https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=45630
    https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=45631 and
    https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=45633.
    
    To hopefully make it easier to figure out how to add new fuzz targets
    going forward I also added the following comment to the build script
    ```
    When new fuzz targets are added it usually makes sense to notify the maintainers of
    the elfutils project using the mailing list: elfutils-devel@sourceware.org. There
    fuzz targets can be reviewed properly (to make sure they don't fail to compile with -Werror
    for example), their names can be chosen accordingly (so as not to spam the mailing
    list with bogus bug reports that are opened and closed once they are renamed) and so
    on. Also since a lot of bug reports coming out of the blue aren't exactly helpful
    fuzz targets should probably be added one at a time to make it easier to keep track
    of them.
    ```
    
    It's a follow-up to google#7395
    and google#7393.
    evverx committed Mar 23, 2022
    Configuration menu
    Copy the full SHA
    59131cd View commit details
    Browse the repository at this point in the history