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

fuzzing: improve coverage #2495

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Conversation

IvanNardi
Copy link
Collaborator

Fix detection of WebDAV and Gnutella (over HTTP)
Fix detection of z3950

Add two fuzzers to test ndpi_memmem() and ndpi_strnstr()

Remove some dead code:

  • RTP: the same exact check is performed at the very beginning of the function
  • MQTT: use a better helper to exclude the protocol
  • Colletd: ndpi_hostname_sni_set() never fails

Update pl7m code (fix a Use-of-uninitialized-value error)

Fix detection of WebDAV and Gnutella (over HTTP)
Fix detection of z3950

Add two fuzzers to test `ndpi_memmem()` and `ndpi_strnstr()`

Remove some dead code:
* RTP: the same exact check is performed at the very beginning of the
function
* MQTT: use a better helper to exclude the protocol
* Colletd: `ndpi_hostname_sni_set()` never fails

Update pl7m code (fix a Use-of-uninitialized-value error)
@IvanNardi IvanNardi merged commit c3ba653 into ntop:dev Jul 12, 2024
35 checks passed
@IvanNardi IvanNardi deleted the fuzz-cov-improvements branch July 12, 2024 12:22
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.

1 participant