Skip to content

Extend fuzzing harness to add more reachability#264

Merged
aous72 merged 2 commits intoaous72:masterfrom
DavidKorczynski:patch-2
Apr 5, 2026
Merged

Extend fuzzing harness to add more reachability#264
aous72 merged 2 commits intoaous72:masterfrom
DavidKorczynski:patch-2

Conversation

@DavidKorczynski
Copy link
Copy Markdown
Contributor

Goal is to expand code coverage

Current code coverage is available at: https://storage.googleapis.com/oss-fuzz-coverage/openjph/reports/20260326/linux/src/report.html

Goal is to expand code coverage
@palemieux
Copy link
Copy Markdown
Contributor

Could we use the opportunity to add a fuzzing harness for encoding?

@DavidKorczynski
Copy link
Copy Markdown
Contributor Author

yeah, ill be happy to do that. Same PR or a new one?

@palemieux
Copy link
Copy Markdown
Contributor

palemieux commented Mar 27, 2026 via email

Signed-off-by: David Korczynski <david@adalogics.com>
@aous72
Copy link
Copy Markdown
Owner

aous72 commented Mar 30, 2026

Dear @DavidKorczynski and @palemieux,

Thank you for this very useful PR.
Sorry for being too busy with another matter.
I just need to confirm one thing with you guys regarding planar settings -- you may have taken care of this, but I am just checking.
If we need to employ colour transform, we cannot use planar.
If we do need colour transform, planar is optional.

I am happy to merge this PR if the planar option is setup correctly.

Thank you again.

Kind regards,
Aous.

@DavidKorczynski
Copy link
Copy Markdown
Contributor Author

If we need to employ colour transform, we cannot use planar.

Yes: https://github.com/aous72/OpenJPH/pull/264/changes#diff-0d532342cc7ce994ce6a641390e660dd11c72b3dd85244a3f7a3941c592e11deR75

  if (color_transform) planar = false;

@aous72 aous72 merged commit 9b576c6 into aous72:master Apr 5, 2026
17 checks passed
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.

3 participants