Issue metadata
Sign in to add a comment
|
FrameSelection::SelectAll should bail if selectstart event handler removes the frame |
||||||||||||||||||||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=5012923471888384 Fuzzer: ifratric-browserfuzzer-v3 Job Type: mac_asan_chrome Platform Id: mac Crash Type: UNKNOWN READ Crash Address: 0x0000000005b0 Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::FrameSelection::setFocusedNodeIfNeeded Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=450347:450401 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5012923471888384 Issue filed automatically. See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Apr 27 2017
,
Apr 27 2017
Lower to Pri-2, since this issue caused by unusual HTML.
,
Apr 29 2017
Minimized repro case:
<iframe id="frame"></iframe>
<script>
frame.contentDocument.onselectstart = () => frame.remove();
frame.contentDocument.execCommand("selectAll");
</script>
,
Apr 29 2017
,
May 1 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/73fe6ba4ceb2db91a43ee2305f69ffde08001f62 commit 73fe6ba4ceb2db91a43ee2305f69ffde08001f62 Author: xiaochengh <xiaochengh@chromium.org> Date: Mon May 01 05:42:36 2017 Make FrameSelection::SelectAll bail if 'selectstart' detaches frame BUG= 715387 TEST=LayoutTests/editing/selection/selectstart_detaches_frame.html Review-Url: https://codereview.chromium.org/2848993002 Cr-Commit-Position: refs/heads/master@{#468292} [add] https://crrev.com/73fe6ba4ceb2db91a43ee2305f69ffde08001f62/third_party/WebKit/LayoutTests/editing/selection/selectstart_detaches_frame.html [modify] https://crrev.com/73fe6ba4ceb2db91a43ee2305f69ffde08001f62/third_party/WebKit/Source/core/editing/FrameSelection.cpp
,
May 1 2017
ClusterFuzz has detected this issue as fixed in range 468288:468298. Detailed report: https://clusterfuzz.com/testcase?key=5012923471888384 Fuzzer: ifratric-browserfuzzer-v3 Job Type: mac_asan_chrome Platform Id: mac Crash Type: UNKNOWN READ Crash Address: 0x0000000005b0 Crash State: blink::Document::updateStyleAndLayoutTreeIgnorePendingStylesheets blink::Document::updateStyleAndLayoutIgnorePendingStylesheets blink::FrameSelection::setFocusedNodeIfNeeded Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=450347:450401 Fixed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=468288:468298 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5012923471888384 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 5012923471888384 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 msrchandra@chromium.org
, Apr 26 2017Components: Blink>Editing
Labels: M-60 Test-Predator-Correct-CLs
Owner: yosin@chromium.org
Status: Assigned (was: Untriaged)