Stack-overflow in blink::PrePaintTreeWalk::Walk |
||||||||||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=6741825858830336 Fuzzer: ochang_domfuzzer Job Type: linux_asan_chrome_mp Platform Id: linux Crash Type: Stack-overflow Crash Address: 0x7ffc812e8898 Crash State: blink::PrePaintTreeWalk::Walk Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_asan_chrome_mp&range=461467:461491 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6741825858830336 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Oct 19 2017
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/ca2623e17b5c45400485a2ceb08545baa9f09c4f (Optimize ClipRects in the PrePaintTreeWalk.). If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
,
Oct 19 2017
Not clear that we need to support 3000 level deep nested elements. I'd be happy WontFix on this.
,
Oct 26 2017
,
Nov 2 2017
ClusterFuzz testcase 6741825858830336 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.
,
Nov 3 2017
,
Nov 3 2017
This is a common crash in real world (top blink crash by far), judging from Opera crash reports. Some 370 nested calls to blink::PrePaintTreeWalk::Walk(blink::LayoutObject const&, blink::PrePaintTreeWalkContext const&) ending with a crash. Crash Reason EXCEPTION_STACK_OVERFLOW Something needs to not be recursive or use much less stack if it is recursive. Example of a common url triggering it: https://clients5.google.com/pagead/drt/dn/
,
Nov 3 2017
Updating the status to 'Assigned'.
,
Nov 3 2017
A note, what I said above was for Chromium 61. I can't see a similar crash in top 10 for 62 and 63 in Opera so maybe it used to be worse than it is now, and now is ok?
,
Nov 3 2017
Nevertheless I WontFixed this bug as this is not a security issue.
,
Nov 6 2017
This is tracked in bug 781301 so closing this one again.
,
Nov 7 2017
,
Nov 7 2017
,
Nov 18 2017
ClusterFuzz has detected this issue as fixed in range 514498:517698. Detailed report: https://clusterfuzz.com/testcase?key=6741825858830336 Fuzzer: ochang_domfuzzer Job Type: linux_asan_chrome_mp Platform Id: linux Crash Type: Stack-overflow Crash Address: 0x7ffc812e8898 Crash State: blink::PrePaintTreeWalk::Walk Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_asan_chrome_mp&range=461467:461491 Fixed: https://clusterfuzz.com/revisions?job=linux_asan_chrome_mp&range=514498:517698 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6741825858830336 See https://github.com/google/clusterfuzz-tools for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page. |
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by ClusterFuzz
, Oct 19 2017Labels: Test-Predator-AutoComponents