• Andreas Haas's avatar
    Reland [wasm] Check the size of a function body before storing it · 641705e0
    Andreas Haas authored
    In the original CL I moved an error check backwards, unfortunately
    behind a vector lookup which should not happen when there is an error.
    Now I also move the vector lookup backwards.
    
    Original message:
    We stored the size of a function body before we check that
    these values are valid. This caused a failing DCHECK in the constructor
    of WireBytesRef which checked for integer overflows. With this CL we
    check the size of the function body before we create the WireBytesRef.
    
    R=clemensh@chromium.org
    
    Bug: chromium:738097
    Change-Id: Ie65b3cfcbcd6bdb3f04b0760673d9c7b7a0d1057
    Reviewed-on: https://chromium-review.googlesource.com/561519Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Andreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#46453}
    641705e0
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...