New issue
Advanced search Search tips

Issue 842490 link

Starred by 4 users

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Stack-overflow in LayoutIfNeeded

Project Member Reported by ClusterFuzz, May 13 2018

Issue description

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

Fuzzer: attekett_surku_fuzzer
Job Type: linux_lsan_chrome_mp
Platform Id: linux

Crash Type: Stack-overflow
Crash Address: 0x7ffcc2044eb8
Crash State:
  LayoutIfNeeded
  blink::SVGLayoutSupport::LayoutChildren
  blink::LayoutSVGContainer::UpdateLayout
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_lsan_chrome_mp&range=523898:523900

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

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by ClusterFuzz, May 13 2018

Components: Blink>DOM
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.

Comment 2 by fergal@chromium.org, May 14 2018

Components: -Blink>DOM Blink>SVG
Opening the repro SVG gives me an error about excessive node nesting. So it seems like we should catch this rather than hitting a stack overflow.


Cc: brajkumar@chromium.org
Labels: -Pri-1 M-67 CF-NeedsTriage Pri-2
> Predator was unable to identify any culprit changelists for this test case. 

> Unable to find actual suspect through code search and also observing no possible suspect CL under regression range, hence requesting someone from blink team to look in to it.

Thanks!
Labels: -Pri-2 Pri-3
Status: Available (was: Untriaged)

Comment 5 by f...@opera.com, May 14 2018

Maybe something should be done to this:

  // FIXME: HTMLConstructionSite has a limit of 512, should these match?
  static const unsigned kMaxXMLTreeDepth = 5000;

Cc: -brajkumar@chromium.org
Labels: -M-67 -CF-NeedsTriage M-71 M-70
Gentle Ping! Just wondering do we have any further update on this?

Thank you!
I suspect adjusting the limit from c#5 is still the most reasonable thing to do here.

Sign in to add a comment