T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fuzz: Don't panic on module compilation errors (#875)

Browse Source 
Let's avoid having two phases of checks and just ignore the module
compilation errors during the instantiate oracle, only relying on one check.
This commit is contained in:
Alex Crichton
2020-01-30 22:22:05 +01:00
committed by GitHub
parent f2382db461
commit c6438d0d44
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
crates/fuzzing/src/oracles.rs
View File

@@ -23,10 +23,6 @@ use wasmtime::*;
/// ///
/// You can control which compiler is used via passing a `Strategy`. /// You can control which compiler is used via passing a `Strategy`.
pub fn instantiate(wasm: &[u8], strategy: Strategy) { pub fn instantiate(wasm: &[u8], strategy: Strategy) {
if wasmparser::validate(wasm, None).is_err() {
return;
}
let mut config = Config::new(); let mut config = Config::new();
config config
.strategy(strategy) .strategy(strategy)
@@ -34,7 +30,10 @@ pub fn instantiate(wasm: &[u8], strategy: Strategy) {
let engine = Engine::new(&config); let engine = Engine::new(&config);
let store = Store::new(&engine); let store = Store::new(&engine);
let module = Module::new(&store, wasm).expect("Failed to compile a valid Wasm module!"); let module = match Module::new(&store, wasm) {
Ok(module) => module,
Err(_) => return,
};
let imports = match dummy_imports(&store, module.imports()) { let imports = match dummy_imports(&store, module.imports()) {
Ok(imps) => imps, Ok(imps) => imps,