New issue
Advanced search Search tips

Issue 822742 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug
Team-Accessibility



Sign in to add a comment

Null-dereference READ in blink::AXMenuList::DidUpdateActiveOption

Project Member Reported by ClusterFuzz, Mar 16 2018

Issue description

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

Fuzzer: inferno_twister
Job Type: linux_ubsan_vptr_content_shell_drt
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x000000000000
Crash State:
  blink::AXMenuList::DidUpdateActiveOption
  blink::LayoutMenuList::UpdateFromElement
  blink::HTMLSelectElement::SelectOption
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_vptr_content_shell_drt&range=527199:527221

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Mar 16 2018

Components: Blink>Accessibility 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, Mar 16 2018

Cc: chrishtr@chromium.org l...@chromium.org shend@chromium.org
Labels: Test-Predator-Auto-CC
Automatically adding ccs based on suspected regression changelists:

Reduce maximum number of fragments to 500. by chrishtr@chromium.org - https://chromium.googlesource.com/chromium/src/+/644fbe520dc19514a80d86a61655ff81ea4b00d5

[css-typed-om] Fix crash by always considering css wide keywords. by shend@chromium.org - https://chromium.googlesource.com/chromium/src/+/64bd68a928aebf4399f03eaf53ee15785e78fe54

DevTools: ignore hash in url when searching for associated uiSourceCode by luoe@chromium.org - https://chromium.googlesource.com/chromium/src/+/3b46beb71bae6552e11b09ccba1b0b61ed9709e6

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label.

Comment 3 by e...@chromium.org, Mar 19 2018

Cc: bsep@chromium.org
Components: -Blink>Layout Blink>Forms
The two blink changes lookm unrelated. r527204 https://chromium-review.googlesource.com/848232 changes forms which might be applicable.

Comment 4 by bsep@chromium.org, Mar 20 2018

My CL changes Views radio buttons in the native UI, not HTML forms, so I would be surprised if it were related. (Unless HTML form controls are implemented in Views?)

Comment 5 by tkent@chromium.org, Mar 20 2018

Components: -Blink>Forms
AXObject::AXObjectCache() returns a null object. I think this is an issue of AX code.

Project Member

Comment 6 by ClusterFuzz, Apr 8 2018

ClusterFuzz has detected this issue as fixed in range 549059:549062.

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

Fuzzer: inferno_twister
Job Type: linux_ubsan_vptr_content_shell_drt
Platform Id: linux

Crash Type: Null-dereference READ
Crash Address: 0x000000000000
Crash State:
  blink::AXMenuList::DidUpdateActiveOption
  blink::LayoutMenuList::UpdateFromElement
  blink::HTMLSelectElement::SelectOption
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_vptr_content_shell_drt&range=527199:527221
Fixed: https://clusterfuzz.com/revisions?job=linux_ubsan_vptr_content_shell_drt&range=549059:549062

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

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 7 by ClusterFuzz, Apr 8 2018

Labels: ClusterFuzz-Verified
Status: Verified (was: Untriaged)
ClusterFuzz testcase 6016941471563776 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