Crash in blink::PaintLayerScrollableArea::ShouldPerformScrollAnchoring |
||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=6641970190745600 Fuzzer: ifratric-browserfuzzer-v3 Job Type: mac_asan_chrome Platform Id: mac Crash Type: UNKNOWN READ Crash Address: 0x0000000000e8 Crash State: blink::PaintLayerScrollableArea::ShouldPerformScrollAnchoring blink::LayoutBlock::GetLayout blink::LayoutBlockFlow::PositionAndLayoutOnceIfNeeded Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=418712:418732 Reproducer Testcase: https://clusterfuzz.com/download/AMIfv952VYvkkEU8sRagHT9sohuKN7hMWjT85bofuxBpc0mG5W3jxuPTXPvLuHeEQ25FLn3niyVxEsV3qIaYpYFg9ADT2K2FvFLsa5FU6NvwUl4TzEY0wHQM3p6-M414gZkOahnegK9bSDPj0v5EMPbwM8Rx2tiwH3b81fpF91owUlH6nM2oaEOTeQGDBP1gcKeTM-hAu7wFYZr9pti6pUEoHuOWRoX68b5T47ZTroaljfCnno6GRZ2GKtR-WKPpVK8QoewTr6tuIGGJ_MQYfSVBSBmoYI3xk3HlybvPaO4wkibkQimx7g2pEf4nIJkVP7a9xkpDpKE3OFSZUa0Xjmxk39l7QLDwH-T5LLFF8sjxhnVM72MpKCPrPHkdJJhQVtCyppTtFRxWclKouoHMUX7_M5u4HbnbUA?testcase_id=6641970190745600 Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Apr 11 2017
,
Apr 11 2017
,
Apr 29 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c566301a304c90ca1c1b05c9ed96e09b4c0392da commit c566301a304c90ca1c1b05c9ed96e09b4c0392da Author: skobes <skobes@chromium.org> Date: Sat Apr 29 09:09:23 2017 Remove LayoutObject::LayerCreationAllowedForSubtree. This method disallows PaintLayer creation inside a LayoutSVGHiddenContainer, even for overflow-clipping elements that would normally require a PaintLayer. This wreaks havoc in the layout code which assumes in numerous places that HasOverflowClip() implies GetScrollableArea() != nullptr. It was added in http://crrev.com/88e9d056 to fix a paint bug, but it seems unnecessary now. The svg/foreignObject layout tests all pass and the original test case on http://wkbug.com/41386 still renders correctly after this change. BUG= 710087 Review-Url: https://codereview.chromium.org/2851763004 Cr-Commit-Position: refs/heads/master@{#468233} [add] https://crrev.com/c566301a304c90ca1c1b05c9ed96e09b4c0392da/third_party/WebKit/LayoutTests/svg/foreignObject/overflow-clip-in-hidden-container-crash-expected.txt [add] https://crrev.com/c566301a304c90ca1c1b05c9ed96e09b4c0392da/third_party/WebKit/LayoutTests/svg/foreignObject/overflow-clip-in-hidden-container-crash.html [modify] https://crrev.com/c566301a304c90ca1c1b05c9ed96e09b4c0392da/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp [modify] https://crrev.com/c566301a304c90ca1c1b05c9ed96e09b4c0392da/third_party/WebKit/Source/core/layout/LayoutObject.cpp [modify] https://crrev.com/c566301a304c90ca1c1b05c9ed96e09b4c0392da/third_party/WebKit/Source/core/layout/LayoutObject.h
,
May 1 2017
ClusterFuzz has detected this issue as fixed in range 468227:468252. Detailed report: https://clusterfuzz.com/testcase?key=6641970190745600 Fuzzer: ifratric-browserfuzzer-v3 Job Type: mac_asan_chrome Platform Id: mac Crash Type: UNKNOWN READ Crash Address: 0x0000000000e8 Crash State: blink::PaintLayerScrollableArea::ShouldPerformScrollAnchoring blink::LayoutBlock::GetLayout blink::LayoutBlockFlow::PositionAndLayoutOnceIfNeeded Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=418712:418732 Fixed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=468227:468252 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=6641970190745600 See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information. If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
,
May 1 2017
ClusterFuzz testcase 6641970190745600 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 |
||||
Comment 1 by mummare...@chromium.org
, Apr 11 2017Components: Blink>Layout
Labels: Test-Layout Test-Predator-Wrong M-58