Skip to content

Add fuzzing#914

Open
JarLob wants to merge 1 commit into
M2Team:mainfrom
JarLob:fuzz
Open

Add fuzzing#914
JarLob wants to merge 1 commit into
M2Team:mainfrom
JarLob:fuzz

Conversation

@JarLob

@JarLob JarLob commented May 26, 2026

Copy link
Copy Markdown
Contributor

This PR adds fuzzing support for NanaZip specific formats as promised.
There is some overlap with #890 because the fuzzing is done with Asan instrumented build to catch memory access violations, but overall Asan build of NanaZip can be built on its own and used to run tests for example.

@JarLob

JarLob commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

The fuzzing / build-base is expected to fail in this PR since Asan build is still not in the main branch. Once accepted it will run fuzzing for 10 minutes for new PRs that make changes in relevant files.

@MouriNaruto

Copy link
Copy Markdown
Member

Thanks. But I will make some changes and merge manually on the code side.

Also, for the test assets side, I need to know some opinions from @dinhngtu to know the assets store side.

But at least, I think we can say:

  • ASAN mode PR and Fuzzing tests PR will be finally merged via multiple commits. (use co-authored-by to note you in all related commits)
  • ASAN mode PR and Fuzzing tests PR will not be ported back to the 6.0 stable branch. But it should be OK because all source code changes in the NanaZip stable branch will be cherry-picked from the main branch, which smells like Linux kernel style.

Kenji Mouri

@JarLob

JarLob commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

Ok, let me know how it goes. I'm really curious because I basically vibe coded a replacement for https://google.github.io/clusterfuzzlite/ that runs on Windows.
Working in my repo, but if I notice any bugs I'll push new commits here.

@MouriNaruto

MouriNaruto commented May 26, 2026

Copy link
Copy Markdown
Member

@JarLob

According to #913, you will know:

So, I also hope you can help me to review these headers/sources changes, which will be used in NanaZip, and it seems your PR will also have similar operation functions.

Kenji Mouri

@JarLob

JarLob commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

Will take a look when time permits.

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