• Clemens Backes's avatar
    [wasm] Crash on failed growing in correctness fuzzing · 801575ef
    Clemens Backes authored
    Different platforms have different limits for growing memory, thus the
    correctness fuzzer should crash instead of failing to grow. This will
    make the fuzzer ignore the test case.
    
    Instead of using the minimum of {wasm::max_mem_pages()} and the declared
    maximum as the limit for growing, we can just use the declared limit.
    {wasm::max_mem_pages()} will already be checked in the called methods.
    All we need is a check for the --correctness-fuzzer-suppressions flag if
    growing actually fails (either because of the platform-specific limit,
    or because of an actual OOM).
    
    Drive-by: unify the duplicated call to
    {BackingStore::GrowWasmMemoryInPlace}.
    
    R=ahaas@chromium.org
    
    Bug: chromium:1213320
    Change-Id: I7f219e1f93824225946d8a2136f15874c091e234
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2931815Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#74917}
    801575ef
wasm-objects.cc 91.6 KB