New issue
Advanced search Search tips

Issue 836375 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Circular ownership on CdmAdapter

Project Member Reported by xhw...@chromium.org, Apr 24 2018

Issue description

CdmAdapter -> CdmPromiseAdapter -> CdmInitializedPromise -> CdmAdapter

// |cdm| ownership passed to the promise.
cdm->Initialize(std::make_unique<CdmInitializedPromise>(cdm_created_cb, cdm));

So if the initialization doesn't finish properly, e.g. the CDM never calls OnInitialized(), we'll leak the CdmAdapter and the CDM instance.
 
Labels: -M-69

Sign in to add a comment