Skip to content

fuzz: add additional harnesses to be consumed by OSS-Fuzz#464

Closed
DavidKorczynski wants to merge 1 commit intoOISF:0.5.xfrom
DavidKorczynski:fuzz-additions
Closed

fuzz: add additional harnesses to be consumed by OSS-Fuzz#464
DavidKorczynski wants to merge 1 commit intoOISF:0.5.xfrom
DavidKorczynski:fuzz-additions

Conversation

@DavidKorczynski
Copy link
Copy Markdown
Contributor

The goal is to increase code coverage relative to the latest code coverage report
https://storage.googleapis.com/oss-fuzz-coverage/libhtp/reports/20260504/linux/src/report.html

Can confirm these new harnesses increase covered code, based on local runs.

Some adjustments are needed in OSS-Fuzz once these are merged.

The goal is to increase code coverage relative to the latest code
coverage report
https://storage.googleapis.com/oss-fuzz-coverage/libhtp/reports/20260504/linux/src/report.html

Can confirm these new harnesses increase covered code, based on local
runs.

Some adjustments are needed in OSS-Fuzz once these are merged.

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

Thanks @DavidKorczynski

I am not sure we are so interested in this : Suricata now only uses this libhtp C (it now uses an embedded libhtp in rust) for its main7 branch which is becoming EOL in the summer...
And Suricata does not use the multipart code that is in libhtp

That is why we do not bother for #402

@DavidKorczynski
Copy link
Copy Markdown
Contributor Author

Sounds good, thanks for the quick reply @catenacyber

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants