New issue
Advanced search Search tips

Issue 789325 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Dec 21
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Null-dereference READ in blink::InlineBox::LogicalLeft

Project Member Reported by ClusterFuzz, Nov 28 2017

Issue description

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

Fuzzer: ifratric-browserfuzzer-v3
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x00000000003d
Crash State:
  blink::InlineBox::LogicalLeft
  blink::RootInlineBox::ClosestLeafChildForLogicalLeftPosition
  blink::LayoutBlockFlow::PositionForPoint
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=519668:519676

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Nov 28 2017

Components: 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, Nov 28 2017

Labels: Test-Predator-Auto-Owner
Owner: jfernan...@igalia.com
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/534ecb7b67e43f206cf8ad645ff8d69286261db0 ([css-grid] Treat 'auto' as 0px when compute non-laid out item's margins).

If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
Project Member

Comment 3 by ClusterFuzz, Nov 28 2017

Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/534ecb7b67e43f206cf8ad645ff8d69286261db0 ([css-grid] Treat 'auto' as 0px when compute non-laid out item's margins).

If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
I'll take a look.
Labels: Test-Predator-Wrong-CLs
Owner: ----
The crash is reproducible with the reduced test case even here:

- refs/heads/master@{#517797}

That version is way before the suspected commit landed. I already had doubts such commit could be the cause of the crash, but this fact confirms it.

Comment 6 by r...@igalia.com, Nov 29 2017

BTW, it crashes here in stable (62.0.3202.94).

Comment 7 by e...@chromium.org, Nov 30 2017

Cc: cbiesin...@chromium.org
Status: Available (was: Assigned)
Cc: jfernan...@igalia.com

Comment 9 by e...@chromium.org, Jan 8 2018

Labels: -Pri-1 Pri-2
Still reproduces.
Project Member

Comment 11 by ClusterFuzz, Dec 21

ClusterFuzz has detected this issue as fixed in range 618280:618281.

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

Fuzzer: ifratric-browserfuzzer-v3
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x00000000003d
Crash State:
  blink::InlineBox::LogicalLeft
  blink::RootInlineBox::ClosestLeafChildForLogicalLeftPosition
  blink::LayoutBlockFlow::PositionForPoint
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=508786:508814
Fixed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=618280:618281

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

See https://github.com/google/clusterfuzz-tools for instructions to reproduce this bug locally.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
Project Member

Comment 12 by ClusterFuzz, Dec 21

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

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Sign in to add a comment