New issue
Advanced search Search tips

Issue 834623 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 832055
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: ----
Type: Bug-Security

Blocking:
issue 778507



Sign in to add a comment

TextOffsetMapping should skip anonymous LayoutBlockFlow with anonymous layout object

Project Member Reported by ClusterFuzz, Apr 19 2018

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=4927582869651456

Fuzzer: bj_broddelwerk
Job Type: linux_msan_content_shell_drt
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00560000007d
Crash State:
  blink::LayoutObject::NextInPreOrder
  blink::TextOffsetMapping::TextOffsetMapping
  blink::TextOffsetMapping::TextOffsetMapping
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=linux_msan_content_shell_drt&range=550102:550107

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4927582869651456

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by ClusterFuzz, Apr 19 2018

Components: Blink>Editing Blink>Layout
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 2 by ClusterFuzz, Apr 19 2018

Labels: Test-Predator-Auto-Owner
Owner: yosin@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/1324be868df000cb2e070199ce2d10cd0c5a2706 (Make NextWordPosition() to utilize TextOffsetMapping).

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label. If you aren't the correct owner for this issue, please unassign yourself as soon as possible so it can be re-triaged.

Comment 3 by yosin@chromium.org, Apr 19 2018

Blocking: 778507
Status: Started (was: Assigned)
Summary: TextOffsetMapping should skip anonymous LayoutBlockFlow with anonymous layout object (was: Crash in blink::LayoutObject::NextInPreOrder)
<select></select> creates anonymous LayoutBlockFlow with anonymous text

layer at (0,0) size 800x600
  LayoutBlockFlow {HTML} at (0,0) size 800x600
    LayoutBlockFlow {BODY} at (8,8) size 784x584
      LayoutText {#text} at (0,0) size 20x19
        text run at (0,0) width 20: "foo"
      LayoutMenuList {SELECT} at (20,0) size 22x20 [bgcolor=#FFFFFF] [border: (1px solid #A9A9A9)]
        LayoutBlockFlow (anonymous) at (1,1) size 20x18
          LayoutText (anonymous) at (4,1) size 4x16
            text run at (4,1) width 4: " "
      LayoutText {#text} at (42,0) size 20x19
        text run at (42,0) width 20: "bar"

Comment 4 by yosin@chromium.org, Apr 19 2018

Mergedinto: 832055
Status: Duplicate (was: Started)
Project Member

Comment 5 by ClusterFuzz, Apr 23 2018

ClusterFuzz has detected this issue as fixed in range 552626:552627.

Detailed report: https://clusterfuzz.com/testcase?key=4927582869651456

Fuzzer: bj_broddelwerk
Job Type: linux_msan_content_shell_drt
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00560000007d
Crash State:
  blink::LayoutObject::NextInPreOrder
  blink::TextOffsetMapping::TextOffsetMapping
  blink::TextOffsetMapping::TextOffsetMapping
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=linux_msan_content_shell_drt&range=550102:550107
Fixed: https://clusterfuzz.com/revisions?job=linux_msan_content_shell_drt&range=552626:552627

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4927582869651456

See https://github.com/google/clusterfuzz-tools 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 6 by sheriffbot@chromium.org, Jul 26

Labels: -Restrict-View-SecurityTeam allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment