New issue
Advanced search Search tips

Issue 676888 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Out-of-memory in v8_wasm_asmjs_fuzzer

Project Member Reported by ClusterFuzz, Dec 24 2016

Issue description

google-starttls-percentages.csv
22.0 KB View Download
Components: Blink>JavaScript
Labels: Test-Predator-Wrong M-57
Owner: ahaas@chromium.org
Status: Assigned (was: Untriaged)
Andreas, please take a look, maybe there is a better way of reporting an OOM.
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/94266b7d86cfb7201461a9af2bfe2ec3268a70e3

commit 94266b7d86cfb7201461a9af2bfe2ec3268a70e3
Author: ahaas <ahaas@chromium.org>
Date: Mon Jan 23 12:09:52 2017

[wasm] Change the constant kV8MaxWasmMemoryPages to a command line flag.

The hardcoded constant caused a problem for the wasm fuzzer because
when the maximum memory was allocated in a test case, clusterfuzz ran
out of memory. with the command line flag we can set a lower limit
for the fuzzer.

The flag has the value of the constant as its default value, so that
for everything but the fuzzers nothing should change.

R=titzer@chromium.org
BUG= chromium:676888 

Review-Url: https://codereview.chromium.org/2626313003
Cr-Commit-Position: refs/heads/master@{#42599}

[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/compiler/wasm-compiler.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/flag-definitions.h
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/flags.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/wasm/module-decoder.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/wasm/wasm-interpreter.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/wasm/wasm-js.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/wasm/wasm-limits.h
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/src/wasm/wasm-module.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/test/fuzzer/wasm-asmjs.cc
[modify] https://crrev.com/94266b7d86cfb7201461a9af2bfe2ec3268a70e3/test/fuzzer/wasm.cc

Project Member

Comment 6 by ClusterFuzz, Jan 24 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase 6487262947966976 is verified as fixed, so closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Sign in to add a comment