• binji's avatar
    Fix bug when transferring SharedArrayBuffer to multiple Workers. · dd7962bf
    binji authored
    Previously, the serialization code would call Externalize for every transferred
    ArrayBuffer or SharedArrayBuffer, but that function can only be called once. If
    the buffer is already externalized, we should call GetContents instead.
    
    Also fix use-after-free bug when transferring ArrayBuffers. The transferred
    ArrayBuffer must be internalized in the new isolate, or be managed by the
    Shell. The current code gives it to the isolate externalized and frees it
    immediately afterward when the SerializationData object is destroyed.
    
    BUG=chromium:497295
    R=jarin@chromium.org
    LOG=n
    
    Review URL: https://codereview.chromium.org/1215233004
    
    Cr-Commit-Position: refs/heads/master@{#29499}
    dd7962bf
Name
Last commit
Last update
benchmarks Loading commit data...
build Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party/binutils Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.gitignore Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
Makefile.nacl Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...