pos + n <= text_selection->text.length(). The text can not fully cover range in |
||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=6661115457306624 Fuzzer: inferno_layout_test_unmodified Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: pos + n <= text_selection->text.length(). The text can not fully cover range in content::RenderWidgetHostViewAura::OnTextSelectionChanged content::TextInputManager::SelectionChanged Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=268656:269696 Minimized Testcase (0.15 Kb): Download: https://cluster-fuzz.appspot.com/download/AMIfv97H-S8znJmA1lN2nhNdI29rxDeGJ49sK3zpB-GAx4YIQjZSw72bBKGn3giP28cQR9NYtWA_t-9GomnKPU0CdGHJSmGcjED81qb8pFE1kQ3srlhKRcF8mGV_Yv9U1YZpewqv6ZlRlKcf0TAWUFL-jtSBBg-SZg?testcase_id=6661115457306624 <div contenteditable="true"> <table id="table"> <script> var sel = window.getSelection(); var parent = table.parentNode; sel.collapse(parent); </script> Additional requirements: Requires Gestures Issue manually filed by: mummareddy See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Aug 17 2016
I will take a look. The DCHECK was about to be removed anyway. I will investigate why the range values are incorrect however.
,
Aug 17 2016
I cannot reproduce the crash following the steps. I downloaded the build which comes with the report. I will keep poking at it.
,
Aug 17 2016
I should also add that the CL mentioned above should not be a cause of this. These values are reported from the renderer side so they issue could be at the origin of the data.
,
Aug 17 2016
Adding dglazkov@ for input since he recently had a CL on updating text selection info. The crash is due to invalid text selection values which directly come from the renderer side. dglazkov@: Can you please take a look? also any suggestions on repro steps are much appreciated. I could not get a crash by following the steps in the docs.
,
Aug 17 2016
Sure thing. Will take a look.
,
Aug 19 2016
ClusterFuzz has detected this issue as fixed in range 412915:412948. Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6661115457306624 Fuzzer: inferno_layout_test_unmodified Job Type: linux_debug_content_shell_drt Platform Id: linux Crash Type: CHECK failure Crash Address: Crash State: pos + n <= text_selection->text.length(). The text can not fully cover range in content::RenderWidgetHostViewAura::OnTextSelectionChanged content::TextInputManager::SelectionChanged Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=268656:269696 Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_debug_content_shell_drt&range=412915:412948 Minimized Testcase (0.15 Kb): Download: https://cluster-fuzz.appspot.com/download/AMIfv97H-S8znJmA1lN2nhNdI29rxDeGJ49sK3zpB-GAx4YIQjZSw72bBKGn3giP28cQR9NYtWA_t-9GomnKPU0CdGHJSmGcjED81qb8pFE1kQ3srlhKRcF8mGV_Yv9U1YZpewqv6ZlRlKcf0TAWUFL-jtSBBg-SZg?testcase_id=6661115457306624 <div contenteditable="true"> <table id="table"> <script> var sel = window.getSelection(); var parent = table.parentNode; sel.collapse(parent); </script> Additional requirements: Requires Gestures 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.
,
Aug 19 2016
ClusterFuzz testcase is verified as fixed, closing issue. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Aug 19 2016
Well dang, I was just going to go look at it :)
,
Oct 18 2016
,
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 mummare...@chromium.org
, Aug 17 2016Labels: Te-Logged M-53
Owner: ekaramad@chromium.org
Status: Assigned (was: Untriaged)