New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 638714 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
OOO until NaN
Closed: Aug 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

pos + n <= text_selection->text.length(). The text can not fully cover range in

Project Member Reported by ClusterFuzz, Aug 17 2016

Issue description

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

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.
 
Components: Tools>Test>FindIt>NoResult
Labels: Te-Logged M-53
Owner: ekaramad@chromium.org
Status: Assigned (was: Untriaged)
Through code search on render_widget_host_view_aura.cc, suspected CL is 
https://chromium.googlesource.com/chromium/src/+/0b1041f656728be7aba9923c9ac21fd4448b0c15
ekaramad@. could you please take a look?
I will take a look. The DCHECK was about to be removed anyway. I will investigate why the range values are incorrect however.
I cannot reproduce the crash following the steps. I downloaded the build which comes with the report. I will keep poking at it.
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.
Cc: dglazkov@chromium.org
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.
Cc: -dglazkov@chromium.org
Labels: -Pri-1 Pri-2
Owner: dglazkov@chromium.org
Sure thing. Will take a look.
Project Member

Comment 7 by ClusterFuzz, 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.
Project Member

Comment 8 by ClusterFuzz, Aug 19 2016

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase is verified as fixed, closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Well dang, I was just going to go look at it :)
Components: -Tools>Test>FindIt>NoResult
Project Member

Comment 11 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
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