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

Issue 807754 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Indirect-leak in content::WebURLLoaderImpl::WebURLLoaderImpl

Project Member Reported by ClusterFuzz, Jan 31 2018

Issue description

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_lsan_chrome_mp
Platform Id: linux

Crash Type: Indirect-leak
Crash Address: 
Crash State:
  content::WebURLLoaderImpl::WebURLLoaderImpl
  content::WebURLLoaderImpl::WebURLLoaderImpl
  content::WorkerFetchContextImpl::URLLoaderFactoryImpl::CreateURLLoader
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_lsan_chrome_mp&range=520898:520900

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Jan 31 2018

Components: Blink>ServiceWorker Internals>Core
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, Jan 31 2018

Cc: apisa...@yandex-team.ru
Labels: Test-Predator-Auto-CC
Automatically adding ccs based on suspected regression changelists:

Suppressing false leaks caused by v8 fast shut in tests. by apisarev@yandex-team.ru - https://chromium.googlesource.com/chromium/src/+/a0ab4d619f5b412ceb73559d738f6b8f2c31200d

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label.
Cc: brajkumar@chromium.org
Labels: M-66
Owner: falken@chromium.org
Status: Assigned (was: Untriaged)
As per the above suspected CL, assigning to the reviewer of the file for more updates on this issue.

Thanks!
That CL just added a suppression and enabled some tests. It looks like the suppression wasn't wide enough. apisarev: can you add more suppressions?

Comment 5 by falken@chromium.org, Feb 15 2018

Status: Fixed (was: Assigned)
I put the wrong bug on a CL.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a913aa8664d7c509fc9466089e790ff2742dac94

commit a913aa8664d7c509fc9466089e790ff2742dac94
Author: Matt Falkenhagen <falken@chromium.org>
Date: Wed Feb 14 08:06:38 2018

Disable WebEmbeddedWorker tests again which are reported as leaks.

Probably the leak detector runs before the worker is fully shut down and all
related memory is freed. The tests were recently re-enabled with some
suppressions in r520900, but they are still reported as leaks in bug
808922.

R=dcheng, glider, kinuko

Bug: 586897, 808922 
Change-Id: I9c8a01303f38c39dcd3334ccb3c138bdac7d53db
Reviewed-on: https://chromium-review.googlesource.com/917844
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536676}
[modify] https://crrev.com/a913aa8664d7c509fc9466089e790ff2742dac94/third_party/WebKit/Source/modules/serviceworkers/WebEmbeddedWorkerImplTest.cpp

Sign in to add a comment