base: Stop overloading ReadFileToString()

components/policy/core/common/cloud/user_cloud_policy_store.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/policy/core/common/cloud/user_cloud_policy_store.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 68 matching lines @@
   policy::PolicyLoadResult result;
   // If the backing file does not exist, just return. We don't verify the key
   // path here, because the key is optional (the validation code will fail if
   // the key does not exist but the loaded policy is unsigned).
   if (!base::PathExists(policy_path)) {
     result.status = policy::LOAD_RESULT_NO_POLICY_FILE;
     return result;
   }
   std::string data;
 
-  if (!base::ReadFileToString(policy_path, &data, kPolicySizeLimit) ||
+  if (!base::ReadFileToStringWithMaxSize(policy_path, &data,
+                                         kPolicySizeLimit) ||
       !result.policy.ParseFromString(data)) {
     LOG(WARNING) << "Failed to read or parse policy data from "
                  << policy_path.value();
     result.status = policy::LOAD_RESULT_LOAD_ERROR;
     return result;
   }
 
-  if (!base::ReadFileToString(key_path, &data, kKeySizeLimit) ||
+  if (!base::ReadFileToStringWithMaxSize(key_path, &data, kKeySizeLimit) ||
       !result.key.ParseFromString(data)) {
     // Log an error on missing key data, but do not trigger a load failure
     // for now since there are still old unsigned cached policy blobs in the
     // wild with no associated key (see kMetricPolicyHasVerifiedCachedKey UMA
     // stat below).
     LOG(ERROR) << "Failed to read or parse key data from " << key_path.value();
     result.key.clear_signing_key();
   }
 
   // Track the occurrence of valid cached keys - when this ratio gets high
@@ skipping 343 matching lines @@
                 std::move(validator->payload()));
 
   // If the key was rotated, update our local cache of the key.
   if (validator->policy()->has_new_public_key())
     policy_key_ = validator->policy()->new_public_key();
   status_ = STATUS_OK;
   NotifyStoreLoaded();
 }
 
 }  // namespace policy