New issue
Advanced search Search tips

Issue 651372 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Crash in blink::Database::runTransaction

Project Member Reported by ClusterFuzz, Sep 29 2016

Issue description

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

Fuzzer: lcamtuf_cross_fuzz
Job Type: linux_asan_chrome_mp
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000000
Crash State:
  blink::Database::runTransaction
  blink::Database::changeVersion
  blink::DatabaseV8Internal::changeVersionMethodCallback
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_mp&range=420502:420630

Minimized Testcase (5.61 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96EJL_UrUR5M-4vilMfSyKrcnyCI0K3gtCv_PLV6yVyBroeGz7GKfidZYKa3kwv7znN0RMAyQu95E_QEcZxC9z20ZXIOIaDL-Y1ZcHruUNkxgEJK-5cEkuquXl-eR4lB1Q5NSbtRxQ1xr7mnP5STcblCal7cg?testcase_id=4577243131281408

Issue manually filed by: brajkumar

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Components: Blink
Labels: -Type-Bug findit-wrong Te-Logged Type-Bug-Regression
Owner: jsb...@chromium.org
Status: Assigned (was: Untriaged)
Through code search on file "Database.cpp", suspecting the below change
Review URL: https://codereview.chromium.org/2375733003

jsbell@ - Observed some recent changes on this file, so assigning to you. Could you please check if this is caused with respect to this change, if not please help us in reassign the issue to the right owner.
Project Member

Comment 2 by ClusterFuzz, Sep 29 2016

ClusterFuzz has detected this issue as fixed in range 421488:421508.

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

Fuzzer: lcamtuf_cross_fuzz
Job Type: linux_asan_chrome_mp
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000000
Crash State:
  blink::Database::runTransaction
  blink::Database::changeVersion
  blink::DatabaseV8Internal::changeVersionMethodCallback
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_mp&range=420502:420630
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_mp&range=421488:421508

Minimized Testcase (5.61 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96EJL_UrUR5M-4vilMfSyKrcnyCI0K3gtCv_PLV6yVyBroeGz7GKfidZYKa3kwv7znN0RMAyQu95E_QEcZxC9z20ZXIOIaDL-Y1ZcHruUNkxgEJK-5cEkuquXl-eR4lB1Q5NSbtRxQ1xr7mnP5STcblCal7cg?testcase_id=4577243131281408

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 3 by ClusterFuzz, Sep 29 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.

Comment 4 by jsb...@chromium.org, Sep 29 2016

This looks like another missing null check for getExecutionContext() - probably racy rather than actually fixed.
Project Member

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