widevine-partner/android
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Close CDM sessions only if opened

Browse Source 
[ Merge of http://go/wvgerrit/49822 ]

This avoids logging an unnecessary error, when the session is not found.

Bug: 79210873
Test: Wv unit/integration test, GtsMediaDrmTest, playback with Play
      Movies and Netflix.

Change-Id: Ifef99d1380d763670ad0fa89c885fb5fd41567e2
This commit is contained in:
Rahul Frias
2018-05-08 00:30:46 -07:00
parent 5b169b1722
commit 9c95aba4f5
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
libwvdrmengine/mediadrm/src_hidl/WVDrmPlugin.cpp
View File

@@ -142,14 +142,16 @@ WVDrmPlugin::~WVDrmPlugin() {
} }
} }
mCryptoSessions.clear(); mCryptoSessions.clear();
CdmIdentifier identifier; if (mCdmIdentifierBuilder.is_sealed()) {
Status status = mCdmIdentifierBuilder.getCdmIdentifier(&identifier); CdmIdentifier identifier;
if (status != Status::OK) { Status status = mCdmIdentifierBuilder.getCdmIdentifier(&identifier);
ALOGE("Failed to get cdm identifier %d", status);
} else {
status = mapCdmResponseType(mCDM->CloseCdm(identifier));
if (status != Status::OK) { if (status != Status::OK) {
ALOGE("Failed to close cdm. status %d", status); ALOGE("Failed to get cdm identifier %d", status);
} else {
status = mapCdmResponseType(mCDM->CloseCdm(identifier));
if (status != Status::OK) {
ALOGE("Failed to close cdm. status %d", status);
}
} }
} }
} }