Add const qualification to clarify verifier is not modified

project-oak · May 9, 2024 · 122538c · 122538c
1 parent f964100
commit 122538c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/cc/client/client.cc b/cc/client/client.cc
@@ -37,7 +37,6 @@
 
 namespace oak::client {
 
-namespace {
 using ::oak::attestation::v1::AttestationResults;
 using ::oak::attestation::verification::AttestationVerifier;
 using ::oak::crypto::ClientEncryptor;
@@ -46,13 +45,12 @@ using ::oak::crypto::v1::EncryptedRequest;
 using ::oak::crypto::v1::EncryptedResponse;
 using ::oak::session::v1::EndorsedEvidence;
 using ::oak::transport::TransportWrapper;
-}  // namespace
 
 constexpr absl::string_view kEmptyAssociatedData = "";
 
 absl::StatusOr<std::unique_ptr<OakClient>> OakClient::Create(
     std::unique_ptr<TransportWrapper> transport,
-    AttestationVerifier& verifier) {
+    const AttestationVerifier& verifier) {
   absl::StatusOr<EndorsedEvidence> endorsed_evidence =
       transport->GetEndorsedEvidence();
   if (!endorsed_evidence.ok()) {

diff --git a/cc/client/client.h b/cc/client/client.h
@@ -39,7 +39,7 @@ class OakClient {
   // and creating an encrypted channel.
   static absl::StatusOr<std::unique_ptr<OakClient>> Create(
       std::unique_ptr<::oak::transport::TransportWrapper> transport,
-      ::oak::attestation::verification::AttestationVerifier& verifier);
+      const ::oak::attestation::verification::AttestationVerifier& verifier);
 
   // Invoke the provided method by fetching and verifying the attested enclave
   // public key, and then using it to encrypt the request body.