New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 761784 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Timeout in v8_wasm_async_fuzzer

Project Member Reported by ClusterFuzz, Sep 4 2017

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=6263613064216576

Fuzzer: libFuzzer_v8_wasm_async_fuzzer
Job Type: libfuzzer_chrome_asan
Platform Id: linux

Crash Type: Timeout (exceeds 25 secs)
Crash Address: 
Crash State:
  v8_wasm_async_fuzzer
  
Sanitizer: address (ASAN)

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6263613064216576

Issue filed automatically.

See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reproducing.md for more information.

Note: This crash might not be reproducible with the provided testcase. That said, for the past 14 days we've been seeing this crash frequently. If you are unable to reproduce this, please try a speculative fix based on the crash stacktrace in the report. The fix can be verified by looking at the crash statistics in the report, a day after the fix is deployed. If the fix resolved the issue, please close the bug by marking as Fixed.
 
Cc: msrchandra@chromium.org
Components: Blink>JavaScript
Labels: Test-Predator-Correct-CLs
Owner: clemensh@chromium.org
Status: Assigned (was: Untriaged)
Assigning to concern owner from Predator results --
Regression information is not available. The result is the blame information. 

Author: jochen
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/c7aace4d43603cc03d90277e7bcf3cc538af1990
Time: Mon Nov 23 08:09:34 2015
The CL last changed line 145 of file execution.cc, which is stack frame 5. 

Author: bmeurer
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/634d1d86d8e98584097c6cb548a68ecf705c44f7
Time: Wed Sep 23 05:43:57 2015
The CL last changed line 181 of file execution.cc, which is stack frame 6. 

Author: ahaas
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/1521fe9c3c88128cea5188b28068736a87838f1c
Time: Mon Sep 12 12:26:37 2016
The CL last changed line 133 of file wasm-module-runner.cc, which is stack frame 7. 

Author: Andreas Haas
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/6828887b85fc9e0e9cfc2d5839c6c4f40281787b
Time: Wed Jun 21 09:24:03 2017
The CL last changed line 48 of file wasm-module-runner.cc, which is stack frame 8. 

Author: Clemens Hammacher
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/bf9196493f7fbbf5a11c71236c201533eb422475
Time: Fri Sep 01 13:20:46 2017
The CL last changed line 72 of file wasm-async.cc, which is stack frame 9. 

Author: verwaest
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/5c73b25ff58e32bcd69153de15a786210769426c
Time: Thu Mar 10 12:14:46 2016
The CL last changed line 25 of file api-arguments.cc, which is stack frame 10. 

Author: yangguo
Project: chromium-v8
Changelist: https://chromium.googlesource.com/v8/v8.git/+/a4bd96a6e2f6b0f19f8fd9379ba42bd6e75190be
Time: Mon Jul 25 19:15:01 2016
The CL last changed line 112 of file builtins-api.cc, which is stack frame 11.

Suspecting Commit#
https://chromium.googlesource.com/v8/v8.git/+/bf9196493f7fbbf5a11c71236c201533eb422475

@clemensh -- Could you please look into the issue, kindly re-assign if it has nothing to do with your changes.
Thank You.

Owner: ahaas@chromium.org
Suspected CL is not changing any semantics.
Assigning to ahaas@, the owner of all (wasm) fuzzers :)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 7 2017

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

commit 7b53a0e010d0e53fbf59620423d8ae644335ae11
Author: Andreas Haas <ahaas@chromium.org>
Date: Thu Sep 07 12:35:45 2017

[wasm] Avoid executing infinite loops in the wasm fuzzers

The wasm-async fuzzer uses the bytes provided by the fuzzer engine
directly as wasm module bytes, compiles them with async compilation, and
then tries to execute the "main" function of the module. This "main"
can have an infinite loop which causes a timeout in the fuzzer. With
this CL the "main" function is first executed with the interpreter. If
the execution in the interpreter finishes within 16k steps, which means
that there is no infinite loop, also the compiled code is executed.

I added the raw fuzzer input as a test case because in this case I
really want to test the fuzzer and not V8.

R=clemensh@chromium.org

Bug:  chromium:761784 
Change-Id: Id1fe5da0da8670ec821ab9979fdb9454dbde1162
Reviewed-on: https://chromium-review.googlesource.com/651046
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Reviewed-by: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47874}
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/src/wasm/wasm-objects.h
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/common/wasm/wasm-module-runner.cc
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/common/wasm/wasm-module-runner.h
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/fuzzer.gyp
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/wasm-async.cc
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/wasm-fuzzer-common.cc
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/wasm-fuzzer-common.h
[modify] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/wasm.cc
[add] https://crrev.com/7b53a0e010d0e53fbf59620423d8ae644335ae11/test/fuzzer/wasm_async/regression-761784.wasm

Comment 4 by ahaas@chromium.org, Sep 11 2017

Status: Fixed (was: Assigned)

Comment 5 by ahaas@chromium.org, Sep 11 2017

Cc: ahaas@chromium.org
 Issue 754428  has been merged into this issue.

Sign in to add a comment