Skip to content

Commit

Permalink
refactor(pallet-gear, wasm-gen): Remove stack limiter to decrease unn…
Browse files Browse the repository at this point in the history
…ecessary errors from wasmi (#3434)

Co-authored-by: Dmitry Novikov <[email protected]>
  • Loading branch information
techraed and breathx authored Oct 22, 2023
1 parent 6673ef0 commit ac01b11
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion pallets/gear/src/schedule.rs
Original file line number Diff line number Diff line change
Expand Up @@ -737,7 +737,8 @@ impl Default for Limits {
// To avoid potential stack overflow problems we have a panic in sandbox in case,
// execution is ended with stack overflow error. So, process queue execution will be
// stopped and we will be able to investigate the problem and decrease this constant if needed.
stack_height: Some(20_000),
// TODO #3435. Disabled stack height is a temp solution.
stack_height: cfg!(not(feature = "fuzz")).then_some(20_000),
globals: 256,
locals: 1024,
parameters: 128,
Expand Down
2 changes: 1 addition & 1 deletion scripts/src/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ run_fuzzer() {
cd $ROOT_DIR/utils/runtime-fuzzer

if [ "$3" = "wlogs" ]; then
LOG_TARGETS="debug,syscalls,gear_wasm_gen=trace,runtime_fuzzer=trace,gear_core_backend=trace"
LOG_TARGETS="debug,syscalls,runtime::sandbox=trace,gear_wasm_gen=trace,runtime_fuzzer=trace,gear_core_backend=trace"
else
LOG_TARGETS="off"
fi
Expand Down

0 comments on commit ac01b11

Please sign in to comment.