Do Not Clean Up Licenses As Aggressively
(This is a merge of http://go/wvgerrit/15430) Previously, after provisioning, any existing offline licenses would be deleted, as they were inevitably tied to the certificate that was just replaced. However, due to the way per-origin provisioning works on Android, this is no longer a safe assumption. Licenses from different origins are all stored together, so this behavior would delete all offline licenses across all origins every time any origin is provisioned. And it is not possible to delete only the licenses for specific origins without changing how licenses are stored. It is too late to change how licenses are stored in Android M, so we need to stop proactively cleaning up licenses in this edge case for now. Once b/23354606 is resolved, we can reinstate this clean-up and properly only clean up the licenses associated with the origin that was just provisioned. Bug: 23324167 Change-Id: Ic21db8c21bdb4243266cd49020ed52287eb21d9a
This commit is contained in:
committed by
Jeff Tinker
parent
9a714edc6f
commit
1d4339d0df
@@ -268,7 +268,6 @@ CdmResponseType CertificateProvisioning::HandleProvisioningResponse(
|
||||
LOGE("HandleProvisioningResponse: failed to save provisioning certificate");
|
||||
return CERT_PROVISIONING_RESPONSE_ERROR_8;
|
||||
}
|
||||
handle.DeleteAllLicenses();
|
||||
|
||||
return NO_ERROR;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user