fitzgen opened PR #1265 from limit-all-api-calls
to master
:
To avoid libfuzzer timeouts, limit the total number of API calls we generate in
theapi_calls
fuzz target. We were already limiting the number of exported
function calls we made, and this extends the limit to all API calls.
alexcrichton submitted PR Review.
alexcrichton created PR Review Comment:
I'm not too familiar with the
Arbitrary
trait, but is there perhaps a way that we can hook in here? Is there a way to indicate that we want at most a particular number of API calls?
alexcrichton submitted PR Review.
fitzgen submitted PR Review.
fitzgen created PR Review Comment:
We have
int_in_range
but this draws from the front of the byte string, rather than the end, like lengths do. The mechanism that lengths use to get an integer in a range from the end of the byte string isn't a public API, unfortunately. (Reason lengths should be drawn from the end of the byte string: https://github.com/rust-fuzz/libfuzzer-sys/blob/0c450753/libfuzzer/utils/FuzzedDataProvider.h#L92-L97)Overall, I don't think it really matters too much, though.
alexcrichton merged PR #1265.
fitzgen submitted PR Review.
fitzgen created PR Review Comment:
Filed https://github.com/rust-fuzz/arbitrary/issues/36 for posterity
Last updated: Dec 23 2024 at 12:05 UTC