blob: 04d9680c4d7b4e984bad53d0d96e283dc827d7ce [file] [log] [blame]
Alex Deymoaea4c1c2015-08-19 20:24:43 -07001//
2// Copyright (C) 2014 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
Alex Deymoc705cc82014-02-19 11:15:00 -080016
Alex Deymo63784a52014-05-28 10:46:14 -070017#include "update_engine/update_manager/chromeos_policy.h"
Alex Deymo0d11c602014-04-23 20:12:20 -070018
Gilad Arnolde1218812014-05-07 12:21:36 -070019#include <algorithm>
Gilad Arnold0adbc942014-05-12 10:35:43 -070020#include <set>
Alex Deymoc705cc82014-02-19 11:15:00 -080021#include <string>
Amin Hassani186ff6a2018-02-27 11:06:03 -080022#include <vector>
Alex Deymoc705cc82014-02-19 11:15:00 -080023
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070024#include <base/logging.h>
Gilad Arnoldb3b05442014-05-30 14:25:05 -070025#include <base/strings/string_util.h>
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070026#include <base/time/time.h>
27
Alex Deymo39910dc2015-11-09 17:04:30 -080028#include "update_engine/common/error_code.h"
Alex Deymoe88e9fe2016-02-03 16:38:00 -080029#include "update_engine/common/error_code_utils.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080030#include "update_engine/common/utils.h"
Alex Deymo63784a52014-05-28 10:46:14 -070031#include "update_engine/update_manager/device_policy_provider.h"
Amin Hassani186ff6a2018-02-27 11:06:03 -080032#include "update_engine/update_manager/enough_slots_ab_updates_policy_impl.h"
33#include "update_engine/update_manager/enterprise_device_policy_impl.h"
34#include "update_engine/update_manager/interactive_update_policy_impl.h"
35#include "update_engine/update_manager/official_build_check_policy_impl.h"
36#include "update_engine/update_manager/out_of_box_experience_policy_impl.h"
Alex Deymo63784a52014-05-28 10:46:14 -070037#include "update_engine/update_manager/policy_utils.h"
38#include "update_engine/update_manager/shill_provider.h"
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070039
Alex Deymo0d11c602014-04-23 20:12:20 -070040using base::Time;
41using base::TimeDelta;
Sen Jiang255e22b2016-05-20 16:15:29 -070042using chromeos_update_engine::ConnectionTethering;
43using chromeos_update_engine::ConnectionType;
Gilad Arnoldb3b05442014-05-30 14:25:05 -070044using chromeos_update_engine::ErrorCode;
Aaron Wood23bd3392017-10-06 14:48:25 -070045using chromeos_update_engine::InstallPlan;
Gilad Arnolddc4bb262014-07-23 10:45:19 -070046using std::get;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070047using std::min;
Gilad Arnold0adbc942014-05-12 10:35:43 -070048using std::set;
Alex Deymoc705cc82014-02-19 11:15:00 -080049using std::string;
Amin Hassani186ff6a2018-02-27 11:06:03 -080050using std::vector;
Alex Deymoc705cc82014-02-19 11:15:00 -080051
Gilad Arnoldb3b05442014-05-30 14:25:05 -070052namespace {
53
Gilad Arnolddc4bb262014-07-23 10:45:19 -070054// Examines |err_code| and decides whether the URL index needs to be advanced,
55// the error count for the URL incremented, or none of the above. In the first
56// case, returns true; in the second case, increments |*url_num_error_p| and
57// returns false; otherwise just returns false.
Gilad Arnoldb3b05442014-05-30 14:25:05 -070058//
59// TODO(garnold) Adapted from PayloadState::UpdateFailed() (to be retired).
Gilad Arnolddc4bb262014-07-23 10:45:19 -070060bool HandleErrorCode(ErrorCode err_code, int* url_num_error_p) {
Gilad Arnoldb3b05442014-05-30 14:25:05 -070061 err_code = chromeos_update_engine::utils::GetBaseErrorCode(err_code);
62 switch (err_code) {
63 // Errors which are good indicators of a problem with a particular URL or
64 // the protocol used in the URL or entities in the communication channel
65 // (e.g. proxies). We should try the next available URL in the next update
66 // check to quickly recover from these errors.
67 case ErrorCode::kPayloadHashMismatchError:
68 case ErrorCode::kPayloadSizeMismatchError:
69 case ErrorCode::kDownloadPayloadVerificationError:
70 case ErrorCode::kDownloadPayloadPubKeyVerificationError:
71 case ErrorCode::kSignedDeltaPayloadExpectedError:
72 case ErrorCode::kDownloadInvalidMetadataMagicString:
73 case ErrorCode::kDownloadSignatureMissingInManifest:
74 case ErrorCode::kDownloadManifestParseError:
75 case ErrorCode::kDownloadMetadataSignatureError:
76 case ErrorCode::kDownloadMetadataSignatureVerificationError:
77 case ErrorCode::kDownloadMetadataSignatureMismatch:
78 case ErrorCode::kDownloadOperationHashVerificationError:
79 case ErrorCode::kDownloadOperationExecutionError:
80 case ErrorCode::kDownloadOperationHashMismatch:
81 case ErrorCode::kDownloadInvalidMetadataSize:
82 case ErrorCode::kDownloadInvalidMetadataSignature:
83 case ErrorCode::kDownloadOperationHashMissingError:
84 case ErrorCode::kDownloadMetadataSignatureMissingError:
85 case ErrorCode::kPayloadMismatchedType:
86 case ErrorCode::kUnsupportedMajorPayloadVersion:
87 case ErrorCode::kUnsupportedMinorPayloadVersion:
88 LOG(INFO) << "Advancing download URL due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -080089 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -070090 << " (" << static_cast<int>(err_code) << ")";
Gilad Arnoldb3b05442014-05-30 14:25:05 -070091 return true;
92
93 // Errors which seem to be just transient network/communication related
94 // failures and do not indicate any inherent problem with the URL itself.
95 // So, we should keep the current URL but just increment the
96 // failure count to give it more chances. This way, while we maximize our
97 // chances of downloading from the URLs that appear earlier in the response
98 // (because download from a local server URL that appears earlier in a
99 // response is preferable than downloading from the next URL which could be
Alex Vakulenko072359c2014-07-18 11:41:07 -0700100 // an Internet URL and thus could be more expensive).
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700101 case ErrorCode::kError:
102 case ErrorCode::kDownloadTransferError:
103 case ErrorCode::kDownloadWriteError:
104 case ErrorCode::kDownloadStateInitializationError:
Gilad Arnold684219d2014-07-07 14:54:57 -0700105 case ErrorCode::kOmahaErrorInHTTPResponse: // Aggregate for HTTP errors.
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700106 LOG(INFO) << "Incrementing URL failure count due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800107 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700108 << " (" << static_cast<int>(err_code) << ")";
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700109 *url_num_error_p += 1;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700110 return false;
111
112 // Errors which are not specific to a URL and hence shouldn't result in
113 // the URL being penalized. This can happen in two cases:
114 // 1. We haven't started downloading anything: These errors don't cost us
115 // anything in terms of actual payload bytes, so we should just do the
116 // regular retries at the next update check.
117 // 2. We have successfully downloaded the payload: In this case, the
118 // payload attempt number would have been incremented and would take care
Alex Vakulenko072359c2014-07-18 11:41:07 -0700119 // of the back-off at the next update check.
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700120 // In either case, there's no need to update URL index or failure count.
121 case ErrorCode::kOmahaRequestError:
122 case ErrorCode::kOmahaResponseHandlerError:
123 case ErrorCode::kPostinstallRunnerError:
124 case ErrorCode::kFilesystemCopierError:
125 case ErrorCode::kInstallDeviceOpenError:
126 case ErrorCode::kKernelDeviceOpenError:
127 case ErrorCode::kDownloadNewPartitionInfoError:
128 case ErrorCode::kNewRootfsVerificationError:
129 case ErrorCode::kNewKernelVerificationError:
130 case ErrorCode::kPostinstallBootedFromFirmwareB:
131 case ErrorCode::kPostinstallFirmwareRONotUpdatable:
132 case ErrorCode::kOmahaRequestEmptyResponseError:
133 case ErrorCode::kOmahaRequestXMLParseError:
134 case ErrorCode::kOmahaResponseInvalid:
135 case ErrorCode::kOmahaUpdateIgnoredPerPolicy:
136 case ErrorCode::kOmahaUpdateDeferredPerPolicy:
Kevin Cernekee2494e282016-03-29 18:03:53 -0700137 case ErrorCode::kNonCriticalUpdateInOOBE:
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700138 case ErrorCode::kOmahaUpdateDeferredForBackoff:
139 case ErrorCode::kPostinstallPowerwashError:
140 case ErrorCode::kUpdateCanceledByChannelChange:
David Zeuthenf3e28012014-08-26 18:23:52 -0400141 case ErrorCode::kOmahaRequestXMLHasEntityDecl:
Allie Woodeb9e6d82015-04-17 13:55:30 -0700142 case ErrorCode::kFilesystemVerifierError:
Alex Deymo1f19dcc2016-02-03 09:22:17 -0800143 case ErrorCode::kUserCanceled:
Weidong Guo421ff332017-04-17 10:08:38 -0700144 case ErrorCode::kOmahaUpdateIgnoredOverCellular:
Sen Jiang02c49422017-10-31 15:14:11 -0700145 case ErrorCode::kUpdatedButNotActive:
Sen Jiang3978ddd2018-03-22 18:05:44 -0700146 case ErrorCode::kNoUpdate:
Marton Hunyady199152d2018-05-07 19:08:48 +0200147 case ErrorCode::kRollbackNotPossible:
Amin Hassani80f4d4c2018-05-16 13:34:00 -0700148 case ErrorCode::kFirstActiveOmahaPingSentPersistenceError:
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700149 LOG(INFO) << "Not changing URL index or failure count due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800150 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700151 << " (" << static_cast<int>(err_code) << ")";
152 return false;
153
154 case ErrorCode::kSuccess: // success code
155 case ErrorCode::kUmaReportedMax: // not an error code
156 case ErrorCode::kOmahaRequestHTTPResponseBase: // aggregated already
Alex Deymo1f19dcc2016-02-03 09:22:17 -0800157 case ErrorCode::kDevModeFlag: // not an error code
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700158 case ErrorCode::kResumedFlag: // not an error code
159 case ErrorCode::kTestImageFlag: // not an error code
160 case ErrorCode::kTestOmahaUrlFlag: // not an error code
161 case ErrorCode::kSpecialFlags: // not an error code
162 // These shouldn't happen. Enumerating these explicitly here so that we
163 // can let the compiler warn about new error codes that are added to
164 // action_processor.h but not added here.
165 LOG(WARNING) << "Unexpected error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800166 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700167 << " (" << static_cast<int>(err_code) << ")";
168 // Note: Not adding a default here so as to let the compiler warn us of
169 // any new enums that were added in the .h but not listed in this switch.
170 }
171 return false;
172}
173
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700174// Checks whether |url| can be used under given download restrictions.
175bool IsUrlUsable(const string& url, bool http_allowed) {
Alex Vakulenko0103c362016-01-20 07:56:15 -0800176 return http_allowed ||
177 !base::StartsWith(url, "http://",
178 base::CompareCase::INSENSITIVE_ASCII);
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700179}
180
181} // namespace
182
Alex Deymo63784a52014-05-28 10:46:14 -0700183namespace chromeos_update_manager {
Alex Deymoc705cc82014-02-19 11:15:00 -0800184
Amin Hassani186ff6a2018-02-27 11:06:03 -0800185const NextUpdateCheckPolicyConstants
186 ChromeOSPolicy::kNextUpdateCheckPolicyConstants = {
187 .timeout_initial_interval = 7 * 60,
188 .timeout_periodic_interval = 45 * 60,
189 .timeout_max_backoff_interval = 4 * 60 * 60,
190 .timeout_regular_fuzz = 10 * 60,
191 .attempt_backoff_max_interval_in_days = 16,
192 .attempt_backoff_fuzz_in_hours = 12,
193};
Alex Deymo14e7dde2015-10-20 14:46:33 -0700194
Gilad Arnold349ac832014-10-06 14:20:28 -0700195const int ChromeOSPolicy::kMaxP2PAttempts = 10;
196const int ChromeOSPolicy::kMaxP2PAttemptsPeriodInSeconds = 5 * 24 * 60 * 60;
Gilad Arnolda2e8eaa2014-09-24 13:12:33 -0700197
Alex Deymo0d11c602014-04-23 20:12:20 -0700198EvalStatus ChromeOSPolicy::UpdateCheckAllowed(
199 EvaluationContext* ec, State* state, string* error,
200 UpdateCheckParams* result) const {
Gilad Arnold42f253b2014-06-25 12:39:17 -0700201 // Set the default return values.
202 result->updates_enabled = true;
203 result->target_channel.clear();
Gilad Arnoldd4b30322014-07-21 15:35:27 -0700204 result->target_version_prefix.clear();
Marton Hunyadyba51c3f2018-04-25 15:18:10 +0200205 result->rollback_allowed = false;
Marton Hunyady0e0e3542018-02-21 18:51:39 +0100206 result->rollback_allowed_milestones = -1;
Gilad Arnold44dc3bf2014-07-18 23:39:38 -0700207 result->is_interactive = false;
Gilad Arnold42f253b2014-06-25 12:39:17 -0700208
Amin Hassani186ff6a2018-02-27 11:06:03 -0800209 EnoughSlotsAbUpdatesPolicyImpl enough_slots_ab_updates_policy;
210 EnterpriseDevicePolicyImpl enterprise_device_policy;
211 OnlyUpdateOfficialBuildsPolicyImpl only_update_official_builds_policy;
212 InteractiveUpdatePolicyImpl interactive_update_policy;
213 OobePolicyImpl oobe_policy;
214 NextUpdateCheckTimePolicyImpl next_update_check_time_policy(
215 kNextUpdateCheckPolicyConstants);
Gilad Arnolda1eabcd2014-07-09 15:42:40 -0700216
Amin Hassani186ff6a2018-02-27 11:06:03 -0800217 vector<Policy const*> policies_to_consult = {
218 // Do not perform any updates if there are not enough slots to do A/B
219 // updates.
220 &enough_slots_ab_updates_policy,
221
222 // Check to see if Enterprise-managed (has DevicePolicy) and/or
223 // Kiosk-mode. If so, then defer to those settings.
224 &enterprise_device_policy,
225
226 // Check to see if an interactive update was requested.
227 &interactive_update_policy,
228
229 // Unofficial builds should not perform periodic update checks.
230 &only_update_official_builds_policy,
231
232 // If OOBE is enabled, wait until it is completed.
233 &oobe_policy,
234
235 // Ensure that periodic update checks are timed properly.
236 &next_update_check_time_policy,
237 };
238
239 // Now that the list of policy implementations, and the order to consult them,
240 // has been setup, consult the policies. If none of the policies make a
241 // definitive decisions about whether or not to check for updates, then allow
242 // the update check to happen.
243 EvalStatus status = ConsultPolicies(policies_to_consult,
244 &Policy::UpdateCheckAllowed,
245 ec,
246 state,
247 error,
248 result);
249 if (EvalStatus::kContinue != status) {
250 return status;
251 } else {
252 // It is time to check for an update.
253 LOG(INFO) << "Allowing update check.";
Gilad Arnoldbfc44f72014-07-09 14:41:39 -0700254 return EvalStatus::kSucceeded;
255 }
Alex Deymoc705cc82014-02-19 11:15:00 -0800256}
257
Aaron Wood23bd3392017-10-06 14:48:25 -0700258EvalStatus ChromeOSPolicy::UpdateCanBeApplied(EvaluationContext* ec,
259 State* state,
260 std::string* error,
261 ErrorCode* result,
262 InstallPlan* install_plan) const {
263 *result = ErrorCode::kSuccess;
264 return EvalStatus::kSucceeded;
265}
266
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700267EvalStatus ChromeOSPolicy::UpdateCanStart(
268 EvaluationContext* ec,
269 State* state,
270 string* error,
Gilad Arnold42f253b2014-06-25 12:39:17 -0700271 UpdateDownloadParams* result,
Gilad Arnoldd78caf92014-09-24 09:28:14 -0700272 const UpdateState update_state) const {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700273 // Set the default return values. Note that we set persisted values (backoff,
274 // scattering) to the same values presented in the update state. The reason is
275 // that preemptive returns, such as the case where an update check is due,
276 // should not clear off the said values; rather, it is the deliberate
277 // inference of new values that should cause them to be reset.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700278 result->update_can_start = false;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700279 result->cannot_start_reason = UpdateCannotStartReason::kUndefined;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700280 result->download_url_idx = -1;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700281 result->download_url_allowed = true;
282 result->download_url_num_errors = 0;
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700283 result->p2p_downloading_allowed = false;
284 result->p2p_sharing_allowed = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700285 result->do_increment_failures = false;
286 result->backoff_expiry = update_state.backoff_expiry;
287 result->scatter_wait_period = update_state.scatter_wait_period;
288 result->scatter_check_threshold = update_state.scatter_check_threshold;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700289
290 // Make sure that we're not due for an update check.
291 UpdateCheckParams check_result;
292 EvalStatus check_status = UpdateCheckAllowed(ec, state, error, &check_result);
293 if (check_status == EvalStatus::kFailed)
294 return EvalStatus::kFailed;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700295 bool is_check_due = (check_status == EvalStatus::kSucceeded &&
296 check_result.updates_enabled == true);
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700297
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700298 // Check whether backoff applies, and if not then which URL can be used for
299 // downloading. These require scanning the download error log, and so they are
300 // done together.
301 UpdateBackoffAndDownloadUrlResult backoff_url_result;
302 EvalStatus backoff_url_status = UpdateBackoffAndDownloadUrl(
303 ec, state, error, &backoff_url_result, update_state);
Gilad Arnold14a9e702014-10-08 08:09:09 -0700304 if (backoff_url_status == EvalStatus::kFailed)
305 return EvalStatus::kFailed;
306 result->download_url_idx = backoff_url_result.url_idx;
307 result->download_url_num_errors = backoff_url_result.url_num_errors;
308 result->do_increment_failures = backoff_url_result.do_increment_failures;
309 result->backoff_expiry = backoff_url_result.backoff_expiry;
310 bool is_backoff_active =
311 (backoff_url_status == EvalStatus::kAskMeAgainLater) ||
312 !backoff_url_result.backoff_expiry.is_null();
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700313
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700314 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Gilad Arnold14a9e702014-10-08 08:09:09 -0700315 bool is_scattering_active = false;
316 EvalStatus scattering_status = EvalStatus::kSucceeded;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700317
318 const bool* device_policy_is_loaded_p = ec->GetValue(
319 dp_provider->var_device_policy_is_loaded());
320 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Gilad Arnold76a11f62014-05-20 09:02:12 -0700321 // Check whether scattering applies to this update attempt. We should not be
322 // scattering if this is an interactive update check, or if OOBE is enabled
323 // but not completed.
324 //
325 // Note: current code further suppresses scattering if a "deadline"
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700326 // attribute is found in the Omaha response. However, it appears that the
Gilad Arnold76a11f62014-05-20 09:02:12 -0700327 // presence of this attribute is merely indicative of an OOBE update, during
328 // which we suppress scattering anyway.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700329 bool is_scattering_applicable = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700330 result->scatter_wait_period = kZeroInterval;
331 result->scatter_check_threshold = 0;
332 if (!update_state.is_interactive) {
Gilad Arnold76a11f62014-05-20 09:02:12 -0700333 const bool* is_oobe_enabled_p = ec->GetValue(
334 state->config_provider()->var_is_oobe_enabled());
335 if (is_oobe_enabled_p && !(*is_oobe_enabled_p)) {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700336 is_scattering_applicable = true;
Gilad Arnold76a11f62014-05-20 09:02:12 -0700337 } else {
338 const bool* is_oobe_complete_p = ec->GetValue(
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700339 state->system_provider()->var_is_oobe_complete());
Gilad Arnold14a9e702014-10-08 08:09:09 -0700340 is_scattering_applicable = (is_oobe_complete_p && *is_oobe_complete_p);
Gilad Arnold76a11f62014-05-20 09:02:12 -0700341 }
342 }
343
344 // Compute scattering values.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700345 if (is_scattering_applicable) {
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700346 UpdateScatteringResult scatter_result;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700347 scattering_status = UpdateScattering(ec, state, error, &scatter_result,
348 update_state);
349 if (scattering_status == EvalStatus::kFailed) {
350 return EvalStatus::kFailed;
351 } else {
352 result->scatter_wait_period = scatter_result.wait_period;
353 result->scatter_check_threshold = scatter_result.check_threshold;
354 if (scattering_status == EvalStatus::kAskMeAgainLater ||
355 scatter_result.is_scattering)
356 is_scattering_active = true;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700357 }
358 }
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700359 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700360
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700361 // Find out whether P2P is globally enabled.
362 bool p2p_enabled;
363 EvalStatus p2p_enabled_status = P2PEnabled(ec, state, error, &p2p_enabled);
364 if (p2p_enabled_status != EvalStatus::kSucceeded)
365 return EvalStatus::kFailed;
366
367 // Is P2P is enabled, consider allowing it for downloading and/or sharing.
368 if (p2p_enabled) {
369 // Sharing via P2P is allowed if not disabled by Omaha.
370 if (update_state.p2p_sharing_disabled) {
371 LOG(INFO) << "Blocked P2P sharing because it is disabled by Omaha.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700372 } else {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700373 result->p2p_sharing_allowed = true;
Gilad Arnoldef8d0872014-10-03 14:14:06 -0700374 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700375
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700376 // Downloading via P2P is allowed if not disabled by Omaha, an update is not
377 // interactive, and other limits haven't been reached.
378 if (update_state.p2p_downloading_disabled) {
379 LOG(INFO) << "Blocked P2P downloading because it is disabled by Omaha.";
380 } else if (update_state.is_interactive) {
381 LOG(INFO) << "Blocked P2P downloading because update is interactive.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700382 } else if (update_state.p2p_num_attempts >= kMaxP2PAttempts) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700383 LOG(INFO) << "Blocked P2P downloading as it was attempted too many "
384 "times.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700385 } else if (!update_state.p2p_first_attempted.is_null() &&
386 ec->IsWallclockTimeGreaterThan(
387 update_state.p2p_first_attempted +
388 TimeDelta::FromSeconds(kMaxP2PAttemptsPeriodInSeconds))) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700389 LOG(INFO) << "Blocked P2P downloading as its usage timespan exceeds "
390 "limit.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700391 } else {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700392 // P2P download is allowed; if backoff or scattering are active, be sure
393 // to suppress them, yet prevent any download URL from being used.
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700394 result->p2p_downloading_allowed = true;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700395 if (is_backoff_active || is_scattering_active) {
396 is_backoff_active = is_scattering_active = false;
397 result->download_url_allowed = false;
398 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700399 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700400 }
401
Gilad Arnold14a9e702014-10-08 08:09:09 -0700402 // Check for various deterrents.
403 if (is_check_due) {
404 result->cannot_start_reason = UpdateCannotStartReason::kCheckDue;
405 return EvalStatus::kSucceeded;
406 }
407 if (is_backoff_active) {
408 result->cannot_start_reason = UpdateCannotStartReason::kBackoff;
409 return backoff_url_status;
410 }
411 if (is_scattering_active) {
412 result->cannot_start_reason = UpdateCannotStartReason::kScattering;
413 return scattering_status;
414 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700415 if (result->download_url_idx < 0 && !result->p2p_downloading_allowed) {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700416 result->cannot_start_reason = UpdateCannotStartReason::kCannotDownload;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700417 return EvalStatus::kSucceeded;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700418 }
419
Gilad Arnold14a9e702014-10-08 08:09:09 -0700420 // Update is good to go.
421 result->update_can_start = true;
Gilad Arnoldaf2f6ae2014-04-28 14:14:52 -0700422 return EvalStatus::kSucceeded;
423}
424
Gilad Arnolda8262e22014-06-02 13:54:27 -0700425// TODO(garnold) Logic in this method is based on
426// ConnectionManager::IsUpdateAllowedOver(); be sure to deprecate the latter.
427//
428// TODO(garnold) The current logic generally treats the list of allowed
429// connections coming from the device policy as a whitelist, meaning that it
430// can only be used for enabling connections, but not disable them. Further,
431// certain connection types (like Bluetooth) cannot be enabled even by policy.
432// In effect, the only thing that device policy can change is to enable
433// updates over a cellular network (disabled by default). We may want to
434// revisit this semantics, allowing greater flexibility in defining specific
435// permissions over all types of networks.
Gilad Arnold684219d2014-07-07 14:54:57 -0700436EvalStatus ChromeOSPolicy::UpdateDownloadAllowed(
Gilad Arnolda8262e22014-06-02 13:54:27 -0700437 EvaluationContext* ec,
438 State* state,
439 string* error,
440 bool* result) const {
441 // Get the current connection type.
442 ShillProvider* const shill_provider = state->shill_provider();
443 const ConnectionType* conn_type_p = ec->GetValue(
444 shill_provider->var_conn_type());
445 POLICY_CHECK_VALUE_AND_FAIL(conn_type_p, error);
446 ConnectionType conn_type = *conn_type_p;
447
448 // If we're tethering, treat it as a cellular connection.
449 if (conn_type != ConnectionType::kCellular) {
450 const ConnectionTethering* conn_tethering_p = ec->GetValue(
451 shill_provider->var_conn_tethering());
452 POLICY_CHECK_VALUE_AND_FAIL(conn_tethering_p, error);
453 if (*conn_tethering_p == ConnectionTethering::kConfirmed)
454 conn_type = ConnectionType::kCellular;
455 }
456
457 // By default, we allow updates for all connection types, with exceptions as
458 // noted below. This also determines whether a device policy can override the
459 // default.
460 *result = true;
461 bool device_policy_can_override = false;
462 switch (conn_type) {
463 case ConnectionType::kBluetooth:
464 *result = false;
465 break;
466
467 case ConnectionType::kCellular:
468 *result = false;
469 device_policy_can_override = true;
470 break;
471
472 case ConnectionType::kUnknown:
473 if (error)
474 *error = "Unknown connection type";
475 return EvalStatus::kFailed;
476
477 default:
478 break; // Nothing to do.
479 }
480
481 // If update is allowed, we're done.
482 if (*result)
483 return EvalStatus::kSucceeded;
484
485 // Check whether the device policy specifically allows this connection.
Gilad Arnolda8262e22014-06-02 13:54:27 -0700486 if (device_policy_can_override) {
487 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
488 const bool* device_policy_is_loaded_p = ec->GetValue(
489 dp_provider->var_device_policy_is_loaded());
490 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
491 const set<ConnectionType>* allowed_conn_types_p = ec->GetValue(
492 dp_provider->var_allowed_connection_types_for_update());
493 if (allowed_conn_types_p) {
494 if (allowed_conn_types_p->count(conn_type)) {
495 *result = true;
496 return EvalStatus::kSucceeded;
497 }
Gilad Arnold28d6be62014-06-30 14:04:04 -0700498 } else if (conn_type == ConnectionType::kCellular) {
499 // Local user settings can allow updates over cellular iff a policy was
500 // loaded but no allowed connections were specified in it.
501 const bool* update_over_cellular_allowed_p = ec->GetValue(
502 state->updater_provider()->var_cellular_enabled());
503 if (update_over_cellular_allowed_p && *update_over_cellular_allowed_p)
504 *result = true;
Gilad Arnolda8262e22014-06-02 13:54:27 -0700505 }
506 }
507 }
508
Gilad Arnold28d6be62014-06-30 14:04:04 -0700509 return (*result ? EvalStatus::kSucceeded : EvalStatus::kAskMeAgainLater);
Gilad Arnolda8262e22014-06-02 13:54:27 -0700510}
511
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700512EvalStatus ChromeOSPolicy::P2PEnabled(EvaluationContext* ec,
513 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800514 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700515 bool* result) const {
516 bool enabled = false;
517
518 // Determine whether use of P2P is allowed by policy. Even if P2P is not
519 // explicitly allowed, we allow it if the device is enterprise enrolled (that
520 // is, missing or empty owner string).
521 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
522 const bool* device_policy_is_loaded_p = ec->GetValue(
523 dp_provider->var_device_policy_is_loaded());
524 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
525 const bool* policy_au_p2p_enabled_p = ec->GetValue(
526 dp_provider->var_au_p2p_enabled());
527 if (policy_au_p2p_enabled_p) {
528 enabled = *policy_au_p2p_enabled_p;
529 } else {
530 const string* policy_owner_p = ec->GetValue(dp_provider->var_owner());
531 if (!policy_owner_p || policy_owner_p->empty())
532 enabled = true;
533 }
534 }
535
536 // Enable P2P, if so mandated by the updater configuration. This is additive
537 // to whether or not P2P is enabled by device policy.
538 if (!enabled) {
539 const bool* updater_p2p_enabled_p = ec->GetValue(
540 state->updater_provider()->var_p2p_enabled());
541 enabled = updater_p2p_enabled_p && *updater_p2p_enabled_p;
542 }
543
544 *result = enabled;
545 return EvalStatus::kSucceeded;
546}
547
548EvalStatus ChromeOSPolicy::P2PEnabledChanged(EvaluationContext* ec,
549 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800550 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700551 bool* result,
552 bool prev_result) const {
553 EvalStatus status = P2PEnabled(ec, state, error, result);
554 if (status == EvalStatus::kSucceeded && *result == prev_result)
555 return EvalStatus::kAskMeAgainLater;
556 return status;
557}
558
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700559EvalStatus ChromeOSPolicy::UpdateBackoffAndDownloadUrl(
Alex Deymof329b932014-10-30 01:37:48 -0700560 EvaluationContext* ec, State* state, string* error,
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700561 UpdateBackoffAndDownloadUrlResult* result,
562 const UpdateState& update_state) const {
563 // Sanity checks.
564 DCHECK_GE(update_state.download_errors_max, 0);
565
566 // Set default result values.
567 result->do_increment_failures = false;
568 result->backoff_expiry = update_state.backoff_expiry;
569 result->url_idx = -1;
570 result->url_num_errors = 0;
571
572 const bool* is_official_build_p = ec->GetValue(
573 state->system_provider()->var_is_official_build());
574 bool is_official_build = (is_official_build_p ? *is_official_build_p : true);
575
576 // Check whether backoff is enabled.
577 bool may_backoff = false;
578 if (update_state.is_backoff_disabled) {
579 LOG(INFO) << "Backoff disabled by Omaha.";
580 } else if (update_state.is_interactive) {
581 LOG(INFO) << "No backoff for interactive updates.";
582 } else if (update_state.is_delta_payload) {
583 LOG(INFO) << "No backoff for delta payloads.";
584 } else if (!is_official_build) {
585 LOG(INFO) << "No backoff for unofficial builds.";
586 } else {
587 may_backoff = true;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700588 }
589
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700590 // If previous backoff still in effect, block.
591 if (may_backoff && !update_state.backoff_expiry.is_null() &&
592 !ec->IsWallclockTimeGreaterThan(update_state.backoff_expiry)) {
593 LOG(INFO) << "Previous backoff has not expired, waiting.";
594 return EvalStatus::kAskMeAgainLater;
595 }
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700596
597 // Determine whether HTTP downloads are forbidden by policy. This only
598 // applies to official system builds; otherwise, HTTP is always enabled.
599 bool http_allowed = true;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700600 if (is_official_build) {
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700601 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
602 const bool* device_policy_is_loaded_p = ec->GetValue(
603 dp_provider->var_device_policy_is_loaded());
604 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
605 const bool* policy_http_downloads_enabled_p = ec->GetValue(
606 dp_provider->var_http_downloads_enabled());
607 http_allowed = (!policy_http_downloads_enabled_p ||
608 *policy_http_downloads_enabled_p);
609 }
610 }
611
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700612 int url_idx = update_state.last_download_url_idx;
613 if (url_idx < 0)
614 url_idx = -1;
615 bool do_advance_url = false;
616 bool is_failure_occurred = false;
617 Time err_time;
618
619 // Scan the relevant part of the download error log, tracking which URLs are
620 // being used, and accounting the number of errors for each URL. Note that
621 // this process may not traverse all errors provided, as it may decide to bail
622 // out midway depending on the particular errors exhibited, the number of
623 // failures allowed, etc. When this ends, |url_idx| will point to the last URL
624 // used (-1 if starting fresh), |do_advance_url| will determine whether the
625 // URL needs to be advanced, and |err_time| the point in time when the last
626 // reported error occurred. Additionally, if the error log indicates that an
627 // update attempt has failed (abnormal), then |is_failure_occurred| will be
628 // set to true.
629 const int num_urls = update_state.download_urls.size();
630 int prev_url_idx = -1;
631 int url_num_errors = update_state.last_download_url_num_errors;
632 Time prev_err_time;
633 bool is_first = true;
634 for (const auto& err_tuple : update_state.download_errors) {
635 // Do some sanity checks.
636 int used_url_idx = get<0>(err_tuple);
637 if (is_first && url_idx >= 0 && used_url_idx != url_idx) {
638 LOG(WARNING) << "First URL in error log (" << used_url_idx
639 << ") not as expected (" << url_idx << ")";
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700640 }
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700641 is_first = false;
642 url_idx = used_url_idx;
643 if (url_idx < 0 || url_idx >= num_urls) {
644 LOG(ERROR) << "Download error log contains an invalid URL index ("
645 << url_idx << ")";
646 return EvalStatus::kFailed;
647 }
648 err_time = get<2>(err_tuple);
649 if (!(prev_err_time.is_null() || err_time >= prev_err_time)) {
650 // TODO(garnold) Monotonicity cannot really be assumed when dealing with
651 // wallclock-based timestamps. However, we're making a simplifying
652 // assumption so as to keep the policy implementation straightforward, for
653 // now. In general, we should convert all timestamp handling in the
654 // UpdateManager to use monotonic time (instead of wallclock), including
655 // the computation of various expiration times (backoff, scattering, etc).
656 // The client will do whatever conversions necessary when
657 // persisting/retrieving these values across reboots. See chromium:408794.
658 LOG(ERROR) << "Download error timestamps not monotonically increasing.";
659 return EvalStatus::kFailed;
660 }
661 prev_err_time = err_time;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700662
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700663 // Ignore errors that happened before the last known failed attempt.
664 if (!update_state.failures_last_updated.is_null() &&
665 err_time <= update_state.failures_last_updated)
666 continue;
667
668 if (prev_url_idx >= 0) {
669 if (url_idx < prev_url_idx) {
670 LOG(ERROR) << "The URLs in the download error log have wrapped around ("
671 << prev_url_idx << "->" << url_idx
672 << "). This should not have happened and means that there's "
673 "a bug. To be conservative, we record a failed attempt "
674 "(invalidating the rest of the error log) and resume "
675 "download from the first usable URL.";
676 url_idx = -1;
677 is_failure_occurred = true;
678 break;
679 }
680
681 if (url_idx > prev_url_idx) {
682 url_num_errors = 0;
683 do_advance_url = false;
684 }
685 }
686
687 if (HandleErrorCode(get<1>(err_tuple), &url_num_errors) ||
688 url_num_errors > update_state.download_errors_max)
689 do_advance_url = true;
690
691 prev_url_idx = url_idx;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700692 }
693
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700694 // If required, advance to the next usable URL. If the URLs wraparound, we
695 // mark an update attempt failure. Also be sure to set the download error
696 // count to zero.
697 if (url_idx < 0 || do_advance_url) {
698 url_num_errors = 0;
699 int start_url_idx = -1;
700 do {
701 if (++url_idx == num_urls) {
702 url_idx = 0;
703 // We only mark failure if an actual advancing of a URL was required.
704 if (do_advance_url)
705 is_failure_occurred = true;
706 }
707
708 if (start_url_idx < 0)
709 start_url_idx = url_idx;
710 else if (url_idx == start_url_idx)
711 url_idx = -1; // No usable URL.
712 } while (url_idx >= 0 &&
713 !IsUrlUsable(update_state.download_urls[url_idx], http_allowed));
714 }
715
716 // If we have a download URL but a failure was observed, compute a new backoff
717 // expiry (if allowed). The backoff period is generally 2 ^ (num_failures - 1)
718 // days, bounded by the size of int and kAttemptBackoffMaxIntervalInDays, and
719 // fuzzed by kAttemptBackoffFuzzInHours hours. Backoff expiry is computed from
720 // the latest recorded time of error.
721 Time backoff_expiry;
722 if (url_idx >= 0 && is_failure_occurred && may_backoff) {
723 CHECK(!err_time.is_null())
724 << "We must have an error timestamp if a failure occurred!";
725 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
726 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
727 PRNG prng(*seed);
Alex Deymof329b932014-10-30 01:37:48 -0700728 int exp = min(update_state.num_failures,
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700729 static_cast<int>(sizeof(int)) * 8 - 2);
Amin Hassani186ff6a2018-02-27 11:06:03 -0800730 TimeDelta backoff_interval = TimeDelta::FromDays(min(
731 1 << exp,
732 kNextUpdateCheckPolicyConstants.attempt_backoff_max_interval_in_days));
733 TimeDelta backoff_fuzz = TimeDelta::FromHours(
734 kNextUpdateCheckPolicyConstants.attempt_backoff_fuzz_in_hours);
735 TimeDelta wait_period = NextUpdateCheckTimePolicyImpl::FuzzedInterval(
736 &prng, backoff_interval.InSeconds(), backoff_fuzz.InSeconds());
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700737 backoff_expiry = err_time + wait_period;
738
739 // If the newly computed backoff already expired, nullify it.
740 if (ec->IsWallclockTimeGreaterThan(backoff_expiry))
741 backoff_expiry = Time();
742 }
743
744 result->do_increment_failures = is_failure_occurred;
745 result->backoff_expiry = backoff_expiry;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700746 result->url_idx = url_idx;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700747 result->url_num_errors = url_num_errors;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700748 return EvalStatus::kSucceeded;
749}
750
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700751EvalStatus ChromeOSPolicy::UpdateScattering(
752 EvaluationContext* ec,
753 State* state,
754 string* error,
755 UpdateScatteringResult* result,
756 const UpdateState& update_state) const {
757 // Preconditions. These stem from the postconditions and usage contract.
758 DCHECK(update_state.scatter_wait_period >= kZeroInterval);
759 DCHECK_GE(update_state.scatter_check_threshold, 0);
760
761 // Set default result values.
762 result->is_scattering = false;
763 result->wait_period = kZeroInterval;
764 result->check_threshold = 0;
765
766 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
767
768 // Ensure that a device policy is loaded.
769 const bool* device_policy_is_loaded_p = ec->GetValue(
770 dp_provider->var_device_policy_is_loaded());
771 if (!(device_policy_is_loaded_p && *device_policy_is_loaded_p))
772 return EvalStatus::kSucceeded;
773
774 // Is scattering enabled by policy?
775 const TimeDelta* scatter_factor_p = ec->GetValue(
776 dp_provider->var_scatter_factor());
777 if (!scatter_factor_p || *scatter_factor_p == kZeroInterval)
778 return EvalStatus::kSucceeded;
779
780 // Obtain a pseudo-random number generator.
781 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
782 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
783 PRNG prng(*seed);
784
785 // Step 1: Maintain the scattering wait period.
786 //
787 // If no wait period was previously determined, or it no longer fits in the
788 // scatter factor, then generate a new one. Otherwise, keep the one we have.
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700789 TimeDelta wait_period = update_state.scatter_wait_period;
790 if (wait_period == kZeroInterval || wait_period > *scatter_factor_p) {
791 wait_period = TimeDelta::FromSeconds(
792 prng.RandMinMax(1, scatter_factor_p->InSeconds()));
793 }
794
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700795 // If we surpassed the wait period or the max scatter period associated with
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700796 // the update, then no wait is needed.
797 Time wait_expires = (update_state.first_seen +
798 min(wait_period, update_state.scatter_wait_period_max));
Gilad Arnolda65fced2014-07-23 09:01:31 -0700799 if (ec->IsWallclockTimeGreaterThan(wait_expires))
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700800 wait_period = kZeroInterval;
801
802 // Step 2: Maintain the update check threshold count.
803 //
804 // If an update check threshold is not specified then generate a new
805 // one.
806 int check_threshold = update_state.scatter_check_threshold;
807 if (check_threshold == 0) {
808 check_threshold = prng.RandMinMax(
809 update_state.scatter_check_threshold_min,
810 update_state.scatter_check_threshold_max);
811 }
812
813 // If the update check threshold is not within allowed range then nullify it.
814 // TODO(garnold) This is compliant with current logic found in
815 // OmahaRequestAction::IsUpdateCheckCountBasedWaitingSatisfied(). We may want
816 // to change it so that it behaves similarly to the wait period case, namely
817 // if the current value exceeds the maximum, we set a new one within range.
818 if (check_threshold > update_state.scatter_check_threshold_max)
819 check_threshold = 0;
820
821 // If the update check threshold is non-zero and satisfied, then nullify it.
822 if (check_threshold > 0 && update_state.num_checks >= check_threshold)
823 check_threshold = 0;
824
825 bool is_scattering = (wait_period != kZeroInterval || check_threshold);
826 EvalStatus ret = EvalStatus::kSucceeded;
827 if (is_scattering && wait_period == update_state.scatter_wait_period &&
828 check_threshold == update_state.scatter_check_threshold)
829 ret = EvalStatus::kAskMeAgainLater;
830 result->is_scattering = is_scattering;
831 result->wait_period = wait_period;
832 result->check_threshold = check_threshold;
833 return ret;
834}
835
Alex Deymo63784a52014-05-28 10:46:14 -0700836} // namespace chromeos_update_manager