• Clemens Backes's avatar
    Hash all strings as unsigned values · caf005e8
    Clemens Backes authored
    A previous CL (https://crrev.com/c/1926769) changed hashing to always
    treat the input as signed values. This causes problems, since the hash
    of a one-byte string differs the hash of the identical two-byte string.
    Hence this CL switches to treating all values as unsigned in hashing.
    
    The bug cannot easily be reproduced in v8 alone, since we would need to
    create an internalized two-byte string, which contains one-byte data.
    Blink manages to create such a string via external strings.
    
    R=jkummerow@chromium.org
    
    Bug: chromium:1025184, chromium:1027131
    Change-Id: Id41aa0e463691c02099a08c6e9d837a079c872df
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1930615Reviewed-by: 's avatarJakob Kummerow <jkummerow@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#65113}
    caf005e8
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...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...