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

Measure code coverage during fuzzing #2

Open
kmcallister opened this issue Apr 2, 2015 · 3 comments
Open

Measure code coverage during fuzzing #2

kmcallister opened this issue Apr 2, 2015 · 3 comments

Comments

@kmcallister
Copy link
Contributor

Measure the code paths & conditions that occur. Don't stop a fuzzing run until we've hit a quota for each.

@mozfreddyb
Copy link

mozfreddyb commented Jul 31, 2019

I would like to give this a poke by porting the existing fuzzer to cargo-fuzz. The idea is to do this in the following steps:

  • move existing code into a fuzz directory (under src?) and wrap in cargo-fuzz fuzz-target
  • expand existing code to include functions that are not covered. A quick glance shows me only 8 match different cases whereas cargo-geiger tells me there are 22 methods using unsafe.
  • (optional) split existing code into separate fuzzing targets, so we can fuzz functions individually

Makes sense?

@mozfreddyb
Copy link

I would like to give this a poke by porting the existing fuzzer to cargo-fuzz. The idea is to do this in the following steps:

* [ ]  move existing code into a `fuzz` directory  (under `src`?) and wrap in cargo-fuzz `fuzz-target`

* [ ]  expand existing code to include functions that are not covered. A quick glance shows me [only 8 match different cases](https://github.com/servo/tendril/blob/master/examples/fuzz.rs#L35) whereas cargo-geiger tells me there are 22 methods using unsafe.

* [ ]  (optional) split existing code into separate fuzzing targets, so we can fuzz functions individually

Makes sense?

@kmcallister @SimonSapin Any kind of feedback would be appreciated :-)

@jdm
Copy link
Member

jdm commented Aug 26, 2019

@mozfreddyb That plan sounds fine.

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

No branches or pull requests

3 participants