alexcrichton opened PR #3664 from fuzz-limits
to main
:
This PR has two commits, the first being the primary one, so solve a recent fuzz bug that came up where a module was timing out while fuzzing. The commits themselves have many more details as to what's going on here. The high-level summary is:
- All fuzz targets are refactored to have one mode of wasm-smith-based configuration.
- Some fuzz targets were shuffled around.
- The cranelift debug verifier is disabled for modules that might have more than 10 functions.
alexcrichton requested fitzgen for a review on PR #3664.
alexcrichton updated PR #3664 from fuzz-limits
to main
.
cfallin submitted PR review.
fitzgen submitted PR review.
fitzgen submitted PR review.
fitzgen created PR review comment:
This new fuzz target never initializes the logger or calls
log_wasm
on the generated bytes, so debugging failures for it will be annoying. Care to add a call tolog_wasm
here? (Which will internally initialize theenv_logger
)
fitzgen created PR review comment:
Ditto here.
alexcrichton updated PR #3664 from fuzz-limits
to main
.
alexcrichton updated PR #3664 from fuzz-limits
to main
.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
Oops excellent point
alexcrichton updated PR #3664 from fuzz-limits
to main
.
alexcrichton merged PR #3664.
Last updated: Dec 23 2024 at 12:05 UTC