Skip to content

OSS-Fuzz: Add new fuzzer targets ewf write operations - #219

Open
arthurscchan wants to merge 3 commits into
libyal:mainfrom
arthurscchan:libewf-write-fuzzer
Open

arthurscchan wants to merge 3 commits into
libyal:mainfrom
arthurscchan:libewf-write-fuzzer

Conversation

@arthurscchan

Copy link
Copy Markdown

This PR creates a new OSS-Fuzz fuzzer targets ewf write operations, together with google/oss-fuzz#16391 to enable a new fuzzer for libewf module of libyal.

@joachimmetz

Copy link
Copy Markdown
Member

@arthurscchan thanks for the proposed changes.

@joachimmetz

Copy link
Copy Markdown
Member

to have build_package/build_dpkg pass add the ossfuzz files to dpkg/source/local-options

@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 40.10%. Comparing base (b6f8123) to head (6966f89).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
+ Coverage   39.34%   40.10%   +0.76%     
==========================================
  Files         101      100       -1     
  Lines       41672    40773     -899     
  Branches     9547     9459      -88     
==========================================
- Hits        16394    16351      -43     
+ Misses      21835    20997     -838     
+ Partials     3443     3425      -18     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arthurscchan

Copy link
Copy Markdown
Author

I will take care of the changes and assign you to approve the OSS-Fuzz side PR later after this is merged.

@joachimmetz

Copy link
Copy Markdown
Member

@arthurscchan thanks, heads up there a change inbound to address the failing macos CI tests, just rebase/merge your PR once the commit is there

@joachimmetz

Copy link
Copy Markdown
Member

I will have the linux dist one retry. Looks like it could not find distribution files it was expecting, maybe due to a race condition in the test/packaging

@joachimmetz

Copy link
Copy Markdown
Member

@arthurscchan let me know if you need assistance, happy to to patch up the changes and get them merged. Note that I'm planning to get some additional changes to the OSS-Fuzz targets in in the near future.

@arthurscchan

Copy link
Copy Markdown
Author

I am really sorry I got stuck on some other things, I will back on this asap. But I don't mind if you could patch it for me and I will verify it later. Sorry again for your time.

@joachimmetz

Copy link
Copy Markdown
Member

no worries, know the "getting preempted by other priorities", let me take closer look tomorrow my morning

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
@joachimmetz

Copy link
Copy Markdown
Member

Heads up, other libyal project put "corpus files" in tests/data/ instead of ossfuzz/corpus/.

I'll get the tests to pass first and then do some restructuring, which might affect the pending oss-fuzz changes

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants