New issue
Advanced search Search tips

Issue 609652 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Outdent command with visibility:collapsed crash

Project Member Reported by ClusterFuzz, May 5 2016

Issue description

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5079282161811456

Fuzzer: mbarbella_js_mutation_layout
Job Type: linux_debug_content_shell_drt
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  CHECK failed: isEndOfParagraph(endOfParagraphToMove). #text "\n   no area click 
  blink::CompositeEditCommand::moveParagraph
  blink::IndentOutdentCommand::outdentParagraph
  

Minimized Testcase (0.39 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv94iPuWqoFm3-HUqIxzg5xoc5377LFJELd1CjnO5HYieBKcfcTA3YgbvcP4BssGorVRzyPXS3Qpf-esc85hGVwQuv-oaJyFzsozfTBG4nsKzShyPmwA7vQnaHAHWNGOFK9-Qts-eY5emaCw3kLyDN7zynh0FeQ
<style>
   * {
    visibility: visible;
}
*:only-of-type {
    visibility: collapse;
  </style>
  <script>
onload = function() {
    document.designMode = 'on';
    var __v_0 = document.querySelector('blockquote');
    getSelection().collapse(__v_0, 2);
    document.execCommand('Outdent');
};
  </script>
  <blockquote>
   <table>
    <table>
</table>
<div>
   no area click seen yet
  </div>
  <div>


Filer: manoranjanr

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Components: Tools>Test>FindIt>NoResult Blink>Compositing
Labels: Te-Logged
Owner: koten...@yandex-team.ru
Status: Assigned (was: Available)
kotenkov@, could you please look into this change (https://chromium.googlesource.com/chromium/src/+/4021ae18b9410d496adc92077e00672253f3876d%5E%21/third_party/WebKit/Source/core/editing/commands/CompositeEditCommand.cpp) if possible?

Thank you!
Components: -Blink>Compositing Blink>Editing
Cc: yosin@chromium.org
Sorry for the delay. The change is not related to the crash itself, but it seems that changing ASSERTs to DCHECKs allowed clusterfuzz to detect the failure.
Owner: yosin@chromium.org

Comment 5 by yosin@chromium.org, Jun 10 2016

Cc: -yosin@chromium.org
Components: Blink>Editing>Command
Labels: -OS-Linux -Pri-1 OS-All Pri-2
Owner: ----
Status: Available (was: Assigned)
Summary: Outdent command with visibility:collapsed crash (was: CHECK failed: isEndOfParagraph(endOfParagraphToMove). #text "\n no area click )
Lower to Pri-2, since real world usage of Outdent command is low.
Project Member

Comment 6 by ClusterFuzz, Oct 7 2016

ClusterFuzz has detected this issue as fixed in range 423391:423439.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5079282161811456

Fuzzer: mbarbella_js_mutation_layout
Job Type: linux_debug_content_shell_drt
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  isEndOfParagraph(endOfParagraphToMove). #text "\n   no area click seen yet\n  "@
  blink::CompositeEditCommand::moveParagraph
  blink::IndentOutdentCommand::outdentParagraph
  
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=423391:423439

Minimized Testcase (0.39 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv95EC4DqaFzpznNZhFU8HQ7UCvfBUZNXwPAdif7JIBE65jONiaoOchO0dkGIJpvstrwZniCbBhmVWlUE11lGt6rGKLZ5rN8lriocLvrQY3YItsd82Led94mGOwyumxtBKptxujZSUurxYebtULggwsV3eO-5ng?testcase_id=5079282161811456
<style>
   * {
    visibility: visible;
}
*:only-of-type {
    visibility: collapse;
  </style>
  <script>
onload = function() {
    document.designMode = 'on';
    var __v_0 = document.querySelector('blockquote');
    getSelection().collapse(__v_0, 2);
    document.execCommand('Outdent');
};
</script>
  <blockquote>
   <table>
   <table>
   </table>
  <div>
   no area click seen yet
  </div>
  <div>


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 7 by ClusterFuzz, Oct 7 2016

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

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Components: -Tools>Test>FindIt>NoResult
Project Member

Comment 9 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