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

Issue 806620 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Stack-overflow in Parser::ParseExpression

Project Member Reported by ClusterFuzz, Jan 28 2018

Issue description

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

Fuzzer: libFuzzer_gn_parser_fuzzer
Job Type: libfuzzer_chrome_asan_debug
Platform Id: linux

Crash Type: Stack-overflow
Crash Address: 0x7ffd77affcb8
Crash State:
  Parser::ParseExpression
  Parser::Assignment
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan_debug&range=532258:532261

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Jan 28 2018

Labels: OS-Mac
Project Member

Comment 2 by ClusterFuzz, Jan 28 2018

Components: Build
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 3 by ClusterFuzz, Jan 28 2018

Cc: thomasanderson@chromium.org fdoray@chromium.org
Labels: Test-Predator-Auto-CC
Automatically adding ccs based on suspected regression changelists:

Remove libunwind arm dependency from libc++abi (src-side changes) by thomasanderson@chromium.org - https://chromium.googlesource.com/chromium/src/+/ea4edeec4194c6e751d3be2d5fa262a2475f6c47

Remove usage of SequencedWorkerPoolOwner. by fdoray@chromium.org - https://chromium.googlesource.com/chromium/src/+/c3b4d56a01fb89d9b290214a36beb2f1eafa5c5b

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label.
thomasanderson@/fdoray@ Could you please look into this issue.
Cc: dpranke@chromium.org
Owner: thomasanderson@chromium.org
Status: WontFix (was: Untriaged)
This is expected.  Clusterfuzz is generating code like "T=T=T=T=T=...".  Each "=" is another stack frame, so if you do that enough, eventually you'll run out of stack space.
Cc: penny...@chromium.org
+pennymac - didn't we do something that was supposed to prevent the fuzzers from going down this path?
https://bugs.chromium.org/p/chromium/issues/detail?id=780844

Just for reference.  If a new "open ended", repeating token is found by the fuzzer (by running out of memory), it can quickly be added to the sanity check code as seen here.

There shouldn't be many more missed!  :) 
Project Member

Comment 8 by ClusterFuzz, Feb 12 2018

Labels: Needs-Feedback
ClusterFuzz testcase 6317492428079104 is still reproducing on tip-of-tree build (trunk).

If this testcase was not reproducible locally or unworkable, ignore this notification and we will file another bug soon with hopefully a better and workable testcase.

Otherwise, if this is not intended to be fixed (e.g. this is an intentional crash), please add ClusterFuzz-Ignore label to prevent future bug filing with similar crash stacktrace.

Sign in to add a comment