bytecodealliance / wizer

The WebAssembly Pre-Initializer
Apache License 2.0
959 stars 58 forks source link

Update all dependencies, wasmtime18, replace wasi-cap-std-sync with wasi-common #99

Closed Gentle closed 8 months ago

Gentle commented 8 months ago

I've updated all dependencies and wasmtime to their latest versions

wasi-cap-std-sync was replaced with wasi-common, some code had to be adjusted

the tests are still green and it works in my test project

I haven't touched the fuzzing code yet, does anyone knwo how to run that so I can make sure it also survived the updates?

Gentle commented 8 months ago

fuzzing is green again

Gentle commented 8 months ago

you were right, wasm_smith::Module::new needed a let-else return, now it goes much further :)

max_funcs and max_exports had to be added or else this assert fails

now it crashes after some minutes with

thread '<unnamed>' panicked at fuzz/fuzz_targets/same_result.rs:79:46:
called `Result::unwrap()` on an `Err` value: failed to parse WebAssembly module

Caused by:
    Unsupported feature: unsupported init expr in element section: GlobalGet { global_index: 0 }

which is at let module = wasmtime::Module::new(&engine, &wasm).unwrap();

should I also return on Err here?