Skip to content

Conversation

@zvonkok
Copy link
Collaborator

@zvonkok zvonkok commented Dec 23, 2025

Added Fuzzing Support

Copilot AI review requested due to automatic review settings December 23, 2025 18:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds fuzzing support to the NVRC project to help identify crashes and edge cases in critical parsing functions. The implementation leverages libFuzzer via cargo-fuzz to test mount table parsing and kernel command-line parameter parsing.

  • Exposes library interface through src/lib.rs to enable fuzzing and testing
  • Adds two fuzz targets for mount parsing and kernel parameter parsing
  • Implements automated fuzzing workflow that runs nightly on GitHub Actions

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/lib.rs New library interface exposing all modules for external testing and fuzzing
src/mount.rs Changed is_mounted_in visibility from private to public to enable fuzzing
fuzz/Cargo.toml Cargo configuration for fuzzing workspace with libfuzzer-sys dependency and two fuzz targets
fuzz/fuzz_targets/mount_parsing.rs Fuzz target testing mount table parsing for panics with various inputs
fuzz/fuzz_targets/kernel_params.rs Fuzz target testing kernel command-line parameter parsing for crashes
.github/workflows/fuzzing.yaml GitHub Actions workflow running nightly fuzzing with 5-minute duration per target

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
@zvonkok zvonkok added the ok-to-test Ok to test label Dec 23, 2025
@zvonkok zvonkok merged commit 73b9d73 into NVIDIA:main Dec 24, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Ok to test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant