Remove unnecessary copies identified by clang-tidy

Change-Id: Iee57c8dfbefdfce6f785f4ae936084f62fc32836
This commit is contained in:
Ian Benz
2023-10-09 20:03:15 +00:00
committed by Robert Shih
parent 1548fe5c98
commit b1ac15d021

View File

@@ -529,7 +529,7 @@ CdmResponseType CertificateProvisioning::HandleProvisioning40Response(
const SignedProvisioningMessage& signed_response, std::string* cert,
std::string* wrapped_key) {
ProvisioningResponse provisioning_response;
const std::string response_message = signed_response.message();
const std::string& response_message = signed_response.message();
if (response_message.empty() ||
!provisioning_response.ParseFromString(response_message)) {
return CdmResponseType(PROVISIONING_4_RESPONSE_FAILED_TO_PARSE_MESSAGE);
@@ -578,8 +578,8 @@ CdmResponseType CertificateProvisioning::HandleProvisioning40Response(
// Get wrapped private key for cast cert
CryptoWrappedKey cast_cert_private_key;
const std::string signature = signed_response.signature();
const std::string core_message = signed_response.oemcrypto_core_message();
const std::string& signature = signed_response.signature();
const std::string& core_message = signed_response.oemcrypto_core_message();
status = crypto_session_->LoadProvisioning(response_message, core_message,
signature,
&cast_cert_private_key.key());