Issue metadata
Sign in to add a comment
|
Crash in blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets |
||||||||||||||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=5833957778391040 Fuzzer: bj_broddelwerk Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: UNKNOWN READ Crash Address: 0x000000000508 Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::TextIteratorAlgorithm<blink::EditingAlgorithm<blink::NodeTraversal> >::ra Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=209699:209703 Minimized Testcase (1.60 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97_rnM2w-rtV3y4gdcP4kvCkwHaTulX45VL1yae410Kp_uK8jHUm0vsar2Kz-kVPwHQJAFq_23l2QYhxAMR7uTaPO7rrLG5B1OuBCjDdea5nWZdA-LyXBlaVXkIRHpOOnXAh617NV1PERP9JhOEN3MLuviYzA Filer: ajha See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
May 31 2016
,
Jun 9 2016
Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4767447295197184 Fuzzer: inferno_layout_test_unmodified Job Type: windows_syzyasan_content_shell Platform Id: windows Crash Type: UNKNOWN Crash Address: 0x0000038f Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::Internals::updateLayoutIgnorePendingStylesheetsAndRunPostLayoutTasks Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_content_shell&range=398496:398502 Minimized Testcase (0.19 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94mWlQsblmwKNNBCqKLebBn2jTFamXJ8npO0NZ8DHveIUCi4QHZsB1FVyPu6cjFvYdZ_G_GhqVyLKMYnzDEj0pDnZmZOS7y-zQHeAHyrAfyWxAglpHHiCixQX5qew9Cj8wtksx_kb2nQufVhCDQ6CXeCmpLIA Additional requirements: Requires Gestures Filer: mummareddy See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jun 10 2016
,
Jun 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce commit 874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce Author: yosin <yosin@chromium.org> Date: Fri Jun 10 10:38:44 2016 Make flat tree version RenderedPosition constructor to work on flat tree This patch makes flat tree version of |RenderedPosition| constructor to work on flat tree rather than DOM tree to calculate inline box position for flat tree position correctly. Before this patch, flat tree version of |RenderedPosition| constructor uses |toPositionInDOMTree()|, but id doesn't work well for children of shadow root and insertion point, e.g. CONENT elements and SLOT elements. In attached layout test, |RenderedPosition| constructor attempts to compute for CONTENT element in DETAIL element. BUG= 616070 , 618421 TEST=LayoutTests/editing/selection/select_all/select_all_details_crash.html Review-Url: https://codereview.chromium.org/2059663002 Cr-Commit-Position: refs/heads/master@{#399150} [add] https://crrev.com/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce/third_party/WebKit/LayoutTests/editing/selection/select_all/select_all_details_crash.html [modify] https://crrev.com/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce/third_party/WebKit/Source/core/editing/RenderedPosition.cpp
,
Jun 13 2016
,
Jun 14 2016
Looks like the issue still persists and impacting the Head as detected by Clusterfuzz. yosin@ : Could you please take a look into this.
,
Jun 14 2016
Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5572962707308544 Fuzzer: bj_broddelwerk Job Type: windows_syzyasan_chrome Platform Id: windows Crash Type: UNKNOWN Crash Address: 0x00000387 Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::TextIteratorAlgorithm<blink::EditingAlgorithm<blink::NodeTraversal> >::ra Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_chrome&range=399164:399271 Minimized Testcase (2.46 Kb): https://cluster-fuzz.appspot.com/download/AMIfv95Hvc31x71uaHu18aXEs2M8O3P_FnXnZ_Y2IUL0ty7jZ4LivCQi3a0fEbq0LoFg3wxC3z4QuxZp5fV7pjr_CdmJLwI9DDBMKK4Gmw08LSDqZVGQeqOLR21xBeqUjTmeecn_12B344V0xBGTWOzaG-YNaWr31A Filer: durga.behera See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Jun 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce commit 874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce Author: yosin <yosin@chromium.org> Date: Fri Jun 10 10:38:44 2016 Make flat tree version RenderedPosition constructor to work on flat tree This patch makes flat tree version of |RenderedPosition| constructor to work on flat tree rather than DOM tree to calculate inline box position for flat tree position correctly. Before this patch, flat tree version of |RenderedPosition| constructor uses |toPositionInDOMTree()|, but id doesn't work well for children of shadow root and insertion point, e.g. CONENT elements and SLOT elements. In attached layout test, |RenderedPosition| constructor attempts to compute for CONTENT element in DETAIL element. BUG= 616070 , 618421 TEST=LayoutTests/editing/selection/select_all/select_all_details_crash.html Review-Url: https://codereview.chromium.org/2059663002 Cr-Commit-Position: refs/heads/master@{#399150} [add] https://crrev.com/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce/third_party/WebKit/LayoutTests/editing/selection/select_all/select_all_details_crash.html [modify] https://crrev.com/874ac2203bcbf8d00ccb7c447ad2d307cb3dd7ce/third_party/WebKit/Source/core/editing/RenderedPosition.cpp
,
Jun 27 2016
,
Aug 25 2016
ClusterFuzz has detected this issue as fixed in range 413791:414128. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5833957778391040 Fuzzer: bj_broddelwerk Job Type: linux_lsan_chrome_mp Platform Id: linux Crash Type: UNKNOWN READ Crash Address: 0x000000000508 Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::TextIteratorAlgorithm<blink::EditingAlgorithm<blink::NodeTraversal> >::ra Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=371187:371278 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=413791:414128 Minimized Testcase (1.59 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97cVr46d0jcJibrf5Q8SBxChn2NvhnIVLn30TdxlYq8jJ4vBIDRf7sH8l-49XqKgj8e8Ti1hP-UHCC7unHBRMnr0oHDt0_Jmo8iGDufRoEYuIq__zJxs9AuG-0lSpkXuEeMges8xlA42GLCw2Yb6V2npo6IIQ?testcase_id=5833957778391040 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.
,
Nov 22 2016
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ajha@chromium.org
, May 31 2016Components: Blink>DOM
Labels: -Type-Bug Te-Logged M-51 Type-Bug-Regression