New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 611689 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

Crash in blink::LocalFrame::detach

Project Member Reported by ClusterFuzz, May 13 2016

Issue description

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

Fuzzer: inferno_twister_custom_bundle
Job Type: linux_asan_chrome_chromeos
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000000
Crash State:
  blink::LocalFrame::detach
  blink::Page::willBeDestroyed
  blink::SVGImage::~SVGImage
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_chromeos&range=393144:393183

Minimized Testcase (0.72 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97AH2HDxKmFcdrm9l72DY6FrzQ_O99a_F5rjZQPNNE7RDQ9DlJuE1Iq0Sz10MvLLozBUatkoOW5kqPzBsdTPPYEyFYNCKkZ4pqD98DuC30UwaDna6BEbuNDsgQ_4oy_TKgiAxIozuFvyC0S3XIAH2UKP11O4A

Additional requirements: Requires Gestures

Filer: rnimmagadda

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: dcheng@chromium.org
Components: Blink>SVG
Labels: -Pri-1 findit-wrong Te-Logged M-52 Pri-2
Owner: tkent@chromium.org
Status: Assigned (was: Available)
Suspecting below:
=================

Author: tkent
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/bc591303022b835a568bcdb4b82e5678c5bf6435
Time: Tue Sep 29 11:15:04 2015
The CL last changed line 96 of file HashCountedSet.h, which is stack frame 4.

---------------------------------------------------------------------------------------------------------

Author: dcheng
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/a4bcdcb1f8df4f7427e208201501a9d8e41e386b
Time: Thu Feb 04 19:17:21 2016
The CL last changed line 112 of file HTMLFrameOwnerElement.h, which is stack frame 6.

@tkent/dcheng: Could you please look into the issue, and if it has nothing to do with your changes and if possible please do assign it to the concerned owner.

Thank you.

Comment 2 by tkent@chromium.org, May 15 2016

Cc: sigbjo...@opera.com
Owner: dcheng@chromium.org
My CL has no production code change.

This looks to be in dcheng's area, or sigbjornf's.

Comment 3 by sigbjo...@opera.com, May 16 2016

I wouldn't be too surprised if this is during Blink shutdown and with LSan, re-assign if it is.. :)
Project Member

Comment 4 by ClusterFuzz, May 16 2016

ClusterFuzz has detected this issue as fixed in range 393799:393810.

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

Fuzzer: inferno_twister_custom_bundle
Job Type: linux_asan_chrome_chromeos
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000000
Crash State:
  blink::LocalFrame::detach
  blink::Page::willBeDestroyed
  blink::SVGImage::~SVGImage
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_chromeos&range=393144:393183
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_chromeos&range=393799:393810

Minimized Testcase (0.72 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97AH2HDxKmFcdrm9l72DY6FrzQ_O99a_F5rjZQPNNE7RDQ9DlJuE1Iq0Sz10MvLLozBUatkoOW5kqPzBsdTPPYEyFYNCKkZ4pqD98DuC30UwaDna6BEbuNDsgQ_4oy_TKgiAxIozuFvyC0S3XIAH2UKP11O4A

Additional requirements: Requires Gestures

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.

Comment 5 by dcheng@chromium.org, May 16 2016

Owner: sigbjo...@opera.com
Looks like this was resolved by https://chromium.googlesource.com/chromium/src/+/a2917a54ac168cd4ccb8bf453a44cadd38ec0c6b. Assigning to sigbjornf to confirm.

Comment 6 by sigbjo...@opera.com, May 17 2016

Cc: -sigbjo...@opera.com
Status: Fixed (was: Assigned)
Yes, floating garbage (the SVGImage) that wasn't swept & finalized before initiating the extra heap scrubbing that LSan requires (=> all static locals/persistents are cleared, including the one that SubframeLoadingDisabler uses, which the testcase here would otherwise trip up on when touching after it having been cleared.)
Project Member

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