blob: 262987e5c7d7f29a0d6ca101fa8674e62abcec13 [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>
Amin Hassanifbb600f2019-08-14 19:52:30 -070020#include <memory>
Gilad Arnold0adbc942014-05-12 10:35:43 -070021#include <set>
Alex Deymoc705cc82014-02-19 11:15:00 -080022#include <string>
Amin Hassani186ff6a2018-02-27 11:06:03 -080023#include <vector>
Alex Deymoc705cc82014-02-19 11:15:00 -080024
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070025#include <base/logging.h>
Gilad Arnoldb3b05442014-05-30 14:25:05 -070026#include <base/strings/string_util.h>
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070027#include <base/time/time.h>
28
Alex Deymo39910dc2015-11-09 17:04:30 -080029#include "update_engine/common/error_code.h"
Alex Deymoe88e9fe2016-02-03 16:38:00 -080030#include "update_engine/common/error_code_utils.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080031#include "update_engine/common/utils.h"
Alex Deymo63784a52014-05-28 10:46:14 -070032#include "update_engine/update_manager/device_policy_provider.h"
Amin Hassani186ff6a2018-02-27 11:06:03 -080033#include "update_engine/update_manager/enough_slots_ab_updates_policy_impl.h"
34#include "update_engine/update_manager/enterprise_device_policy_impl.h"
35#include "update_engine/update_manager/interactive_update_policy_impl.h"
36#include "update_engine/update_manager/official_build_check_policy_impl.h"
37#include "update_engine/update_manager/out_of_box_experience_policy_impl.h"
Alex Deymo63784a52014-05-28 10:46:14 -070038#include "update_engine/update_manager/policy_utils.h"
39#include "update_engine/update_manager/shill_provider.h"
Adolfo Victoria94ffe132018-06-28 16:14:56 -070040#include "update_engine/update_manager/update_time_restrictions_policy_impl.h"
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070041
Alex Deymo0d11c602014-04-23 20:12:20 -070042using base::Time;
43using base::TimeDelta;
Sen Jiang255e22b2016-05-20 16:15:29 -070044using chromeos_update_engine::ConnectionTethering;
45using chromeos_update_engine::ConnectionType;
Gilad Arnoldb3b05442014-05-30 14:25:05 -070046using chromeos_update_engine::ErrorCode;
Aaron Wood23bd3392017-10-06 14:48:25 -070047using chromeos_update_engine::InstallPlan;
Gilad Arnolddc4bb262014-07-23 10:45:19 -070048using std::get;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -070049using std::min;
Gilad Arnold0adbc942014-05-12 10:35:43 -070050using std::set;
Alex Deymoc705cc82014-02-19 11:15:00 -080051using std::string;
Amin Hassanifbb600f2019-08-14 19:52:30 -070052using std::unique_ptr;
Amin Hassani186ff6a2018-02-27 11:06:03 -080053using std::vector;
Alex Deymoc705cc82014-02-19 11:15:00 -080054
Gilad Arnoldb3b05442014-05-30 14:25:05 -070055namespace {
56
Gilad Arnolddc4bb262014-07-23 10:45:19 -070057// Examines |err_code| and decides whether the URL index needs to be advanced,
58// the error count for the URL incremented, or none of the above. In the first
59// case, returns true; in the second case, increments |*url_num_error_p| and
60// returns false; otherwise just returns false.
Gilad Arnoldb3b05442014-05-30 14:25:05 -070061//
62// TODO(garnold) Adapted from PayloadState::UpdateFailed() (to be retired).
Gilad Arnolddc4bb262014-07-23 10:45:19 -070063bool HandleErrorCode(ErrorCode err_code, int* url_num_error_p) {
Gilad Arnoldb3b05442014-05-30 14:25:05 -070064 err_code = chromeos_update_engine::utils::GetBaseErrorCode(err_code);
65 switch (err_code) {
66 // Errors which are good indicators of a problem with a particular URL or
67 // the protocol used in the URL or entities in the communication channel
68 // (e.g. proxies). We should try the next available URL in the next update
69 // check to quickly recover from these errors.
70 case ErrorCode::kPayloadHashMismatchError:
71 case ErrorCode::kPayloadSizeMismatchError:
72 case ErrorCode::kDownloadPayloadVerificationError:
73 case ErrorCode::kDownloadPayloadPubKeyVerificationError:
74 case ErrorCode::kSignedDeltaPayloadExpectedError:
75 case ErrorCode::kDownloadInvalidMetadataMagicString:
76 case ErrorCode::kDownloadSignatureMissingInManifest:
77 case ErrorCode::kDownloadManifestParseError:
78 case ErrorCode::kDownloadMetadataSignatureError:
79 case ErrorCode::kDownloadMetadataSignatureVerificationError:
80 case ErrorCode::kDownloadMetadataSignatureMismatch:
81 case ErrorCode::kDownloadOperationHashVerificationError:
82 case ErrorCode::kDownloadOperationExecutionError:
83 case ErrorCode::kDownloadOperationHashMismatch:
84 case ErrorCode::kDownloadInvalidMetadataSize:
85 case ErrorCode::kDownloadInvalidMetadataSignature:
86 case ErrorCode::kDownloadOperationHashMissingError:
87 case ErrorCode::kDownloadMetadataSignatureMissingError:
88 case ErrorCode::kPayloadMismatchedType:
89 case ErrorCode::kUnsupportedMajorPayloadVersion:
90 case ErrorCode::kUnsupportedMinorPayloadVersion:
Sen Jiang8e768e92017-06-28 17:13:19 -070091 case ErrorCode::kPayloadTimestampError:
Sen Jiang57f91802017-11-14 17:42:13 -080092 case ErrorCode::kVerityCalculationError:
Gilad Arnoldb3b05442014-05-30 14:25:05 -070093 LOG(INFO) << "Advancing download URL due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -080094 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -070095 << " (" << static_cast<int>(err_code) << ")";
Gilad Arnoldb3b05442014-05-30 14:25:05 -070096 return true;
97
98 // Errors which seem to be just transient network/communication related
99 // failures and do not indicate any inherent problem with the URL itself.
100 // So, we should keep the current URL but just increment the
101 // failure count to give it more chances. This way, while we maximize our
102 // chances of downloading from the URLs that appear earlier in the response
103 // (because download from a local server URL that appears earlier in a
104 // response is preferable than downloading from the next URL which could be
Alex Vakulenko072359c2014-07-18 11:41:07 -0700105 // an Internet URL and thus could be more expensive).
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700106 case ErrorCode::kError:
107 case ErrorCode::kDownloadTransferError:
108 case ErrorCode::kDownloadWriteError:
109 case ErrorCode::kDownloadStateInitializationError:
Gilad Arnold684219d2014-07-07 14:54:57 -0700110 case ErrorCode::kOmahaErrorInHTTPResponse: // Aggregate for HTTP errors.
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700111 LOG(INFO) << "Incrementing URL failure count due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800112 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700113 << " (" << static_cast<int>(err_code) << ")";
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700114 *url_num_error_p += 1;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700115 return false;
116
117 // Errors which are not specific to a URL and hence shouldn't result in
118 // the URL being penalized. This can happen in two cases:
119 // 1. We haven't started downloading anything: These errors don't cost us
120 // anything in terms of actual payload bytes, so we should just do the
121 // regular retries at the next update check.
122 // 2. We have successfully downloaded the payload: In this case, the
123 // payload attempt number would have been incremented and would take care
Alex Vakulenko072359c2014-07-18 11:41:07 -0700124 // of the back-off at the next update check.
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700125 // In either case, there's no need to update URL index or failure count.
126 case ErrorCode::kOmahaRequestError:
127 case ErrorCode::kOmahaResponseHandlerError:
128 case ErrorCode::kPostinstallRunnerError:
129 case ErrorCode::kFilesystemCopierError:
130 case ErrorCode::kInstallDeviceOpenError:
131 case ErrorCode::kKernelDeviceOpenError:
132 case ErrorCode::kDownloadNewPartitionInfoError:
133 case ErrorCode::kNewRootfsVerificationError:
134 case ErrorCode::kNewKernelVerificationError:
135 case ErrorCode::kPostinstallBootedFromFirmwareB:
136 case ErrorCode::kPostinstallFirmwareRONotUpdatable:
137 case ErrorCode::kOmahaRequestEmptyResponseError:
138 case ErrorCode::kOmahaRequestXMLParseError:
139 case ErrorCode::kOmahaResponseInvalid:
140 case ErrorCode::kOmahaUpdateIgnoredPerPolicy:
141 case ErrorCode::kOmahaUpdateDeferredPerPolicy:
Kevin Cernekee2494e282016-03-29 18:03:53 -0700142 case ErrorCode::kNonCriticalUpdateInOOBE:
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700143 case ErrorCode::kOmahaUpdateDeferredForBackoff:
144 case ErrorCode::kPostinstallPowerwashError:
145 case ErrorCode::kUpdateCanceledByChannelChange:
David Zeuthenf3e28012014-08-26 18:23:52 -0400146 case ErrorCode::kOmahaRequestXMLHasEntityDecl:
Allie Woodeb9e6d82015-04-17 13:55:30 -0700147 case ErrorCode::kFilesystemVerifierError:
Alex Deymo1f19dcc2016-02-03 09:22:17 -0800148 case ErrorCode::kUserCanceled:
Weidong Guo421ff332017-04-17 10:08:38 -0700149 case ErrorCode::kOmahaUpdateIgnoredOverCellular:
Sen Jiangfe522822017-10-31 15:14:11 -0700150 case ErrorCode::kUpdatedButNotActive:
Sen Jiang89e24c12018-03-22 18:05:44 -0700151 case ErrorCode::kNoUpdate:
Marton Hunyady199152d2018-05-07 19:08:48 +0200152 case ErrorCode::kRollbackNotPossible:
Amin Hassani80f4d4c2018-05-16 13:34:00 -0700153 case ErrorCode::kFirstActiveOmahaPingSentPersistenceError:
Amin Hassanid3d84212019-08-17 00:27:44 -0700154 case ErrorCode::kInternalLibCurlError:
155 case ErrorCode::kUnresolvedHostError:
156 case ErrorCode::kUnresolvedHostRecovered:
Yifan Hong46e37d42019-12-13 12:01:42 -0800157 case ErrorCode::kNotEnoughSpace:
Yifan Hong77279902019-12-17 16:38:21 -0800158 case ErrorCode::kDeviceCorrupted:
Jae Hoon Kim3e69b4c2020-06-16 09:23:39 -0700159 case ErrorCode::kPackageExcludedFromUpdate:
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700160 LOG(INFO) << "Not changing URL index or failure count due to error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800161 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700162 << " (" << static_cast<int>(err_code) << ")";
163 return false;
164
Amin Hassani4b717432019-01-14 16:24:20 -0800165 case ErrorCode::kSuccess: // success code
166 case ErrorCode::kUmaReportedMax: // not an error code
167 case ErrorCode::kOmahaRequestHTTPResponseBase: // aggregated already
168 case ErrorCode::kDevModeFlag: // not an error code
169 case ErrorCode::kResumedFlag: // not an error code
170 case ErrorCode::kTestImageFlag: // not an error code
171 case ErrorCode::kTestOmahaUrlFlag: // not an error code
172 case ErrorCode::kSpecialFlags: // not an error code
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700173 // These shouldn't happen. Enumerating these explicitly here so that we
174 // can let the compiler warn about new error codes that are added to
175 // action_processor.h but not added here.
176 LOG(WARNING) << "Unexpected error "
Alex Deymoe88e9fe2016-02-03 16:38:00 -0800177 << chromeos_update_engine::utils::ErrorCodeToString(err_code)
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700178 << " (" << static_cast<int>(err_code) << ")";
Amin Hassani4b717432019-01-14 16:24:20 -0800179 // Note: Not adding a default here so as to let the compiler warn us of
180 // any new enums that were added in the .h but not listed in this switch.
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700181 }
182 return false;
183}
184
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700185// Checks whether |url| can be used under given download restrictions.
186bool IsUrlUsable(const string& url, bool http_allowed) {
Alex Vakulenko0103c362016-01-20 07:56:15 -0800187 return http_allowed ||
Amin Hassani4b717432019-01-14 16:24:20 -0800188 !base::StartsWith(
189 url, "http://", base::CompareCase::INSENSITIVE_ASCII);
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700190}
191
192} // namespace
193
Alex Deymo63784a52014-05-28 10:46:14 -0700194namespace chromeos_update_manager {
Alex Deymoc705cc82014-02-19 11:15:00 -0800195
Amin Hassanifbb600f2019-08-14 19:52:30 -0700196unique_ptr<Policy> GetSystemPolicy() {
197 return std::make_unique<ChromeOSPolicy>();
198}
199
Amin Hassani186ff6a2018-02-27 11:06:03 -0800200const NextUpdateCheckPolicyConstants
201 ChromeOSPolicy::kNextUpdateCheckPolicyConstants = {
202 .timeout_initial_interval = 7 * 60,
203 .timeout_periodic_interval = 45 * 60,
204 .timeout_max_backoff_interval = 4 * 60 * 60,
205 .timeout_regular_fuzz = 10 * 60,
206 .attempt_backoff_max_interval_in_days = 16,
207 .attempt_backoff_fuzz_in_hours = 12,
208};
Alex Deymo14e7dde2015-10-20 14:46:33 -0700209
Gilad Arnold349ac832014-10-06 14:20:28 -0700210const int ChromeOSPolicy::kMaxP2PAttempts = 10;
211const int ChromeOSPolicy::kMaxP2PAttemptsPeriodInSeconds = 5 * 24 * 60 * 60;
Gilad Arnolda2e8eaa2014-09-24 13:12:33 -0700212
Amin Hassani4b717432019-01-14 16:24:20 -0800213EvalStatus ChromeOSPolicy::UpdateCheckAllowed(EvaluationContext* ec,
214 State* state,
215 string* error,
216 UpdateCheckParams* result) const {
Gilad Arnold42f253b2014-06-25 12:39:17 -0700217 // Set the default return values.
218 result->updates_enabled = true;
219 result->target_channel.clear();
Amin Hassani37b67232020-08-13 09:29:48 -0700220 result->lts_tag.clear();
Gilad Arnoldd4b30322014-07-21 15:35:27 -0700221 result->target_version_prefix.clear();
Marton Hunyadyba51c3f2018-04-25 15:18:10 +0200222 result->rollback_allowed = false;
Marton Hunyady0e0e3542018-02-21 18:51:39 +0100223 result->rollback_allowed_milestones = -1;
Miriam Polzeraff72002020-08-27 08:20:39 +0200224 result->rollback_on_channel_downgrade = false;
Amin Hassanied37d682018-04-06 13:22:00 -0700225 result->interactive = false;
Gilad Arnold42f253b2014-06-25 12:39:17 -0700226
Amin Hassani186ff6a2018-02-27 11:06:03 -0800227 EnoughSlotsAbUpdatesPolicyImpl enough_slots_ab_updates_policy;
228 EnterpriseDevicePolicyImpl enterprise_device_policy;
229 OnlyUpdateOfficialBuildsPolicyImpl only_update_official_builds_policy;
230 InteractiveUpdatePolicyImpl interactive_update_policy;
231 OobePolicyImpl oobe_policy;
232 NextUpdateCheckTimePolicyImpl next_update_check_time_policy(
233 kNextUpdateCheckPolicyConstants);
Gilad Arnolda1eabcd2014-07-09 15:42:40 -0700234
Amin Hassani186ff6a2018-02-27 11:06:03 -0800235 vector<Policy const*> policies_to_consult = {
236 // Do not perform any updates if there are not enough slots to do A/B
237 // updates.
238 &enough_slots_ab_updates_policy,
239
240 // Check to see if Enterprise-managed (has DevicePolicy) and/or
241 // Kiosk-mode. If so, then defer to those settings.
242 &enterprise_device_policy,
243
244 // Check to see if an interactive update was requested.
245 &interactive_update_policy,
246
247 // Unofficial builds should not perform periodic update checks.
248 &only_update_official_builds_policy,
249
250 // If OOBE is enabled, wait until it is completed.
251 &oobe_policy,
252
253 // Ensure that periodic update checks are timed properly.
254 &next_update_check_time_policy,
255 };
256
257 // Now that the list of policy implementations, and the order to consult them,
258 // has been setup, consult the policies. If none of the policies make a
259 // definitive decisions about whether or not to check for updates, then allow
260 // the update check to happen.
261 EvalStatus status = ConsultPolicies(policies_to_consult,
262 &Policy::UpdateCheckAllowed,
263 ec,
264 state,
265 error,
266 result);
267 if (EvalStatus::kContinue != status) {
268 return status;
269 } else {
270 // It is time to check for an update.
271 LOG(INFO) << "Allowing update check.";
Gilad Arnoldbfc44f72014-07-09 14:41:39 -0700272 return EvalStatus::kSucceeded;
273 }
Alex Deymoc705cc82014-02-19 11:15:00 -0800274}
275
Aaron Wood23bd3392017-10-06 14:48:25 -0700276EvalStatus ChromeOSPolicy::UpdateCanBeApplied(EvaluationContext* ec,
277 State* state,
278 std::string* error,
279 ErrorCode* result,
280 InstallPlan* install_plan) const {
Adolfo Victoria0dbf1f92018-07-19 14:18:11 -0700281 UpdateTimeRestrictionsPolicyImpl update_time_restrictions_policy;
282 InteractiveUpdatePolicyImpl interactive_update_policy;
283
284 vector<Policy const*> policies_to_consult = {
285 // Check to see if an interactive update has been requested.
286 &interactive_update_policy,
287
288 // Do not apply or download an update if we are inside one of the
289 // restricted times.
290 &update_time_restrictions_policy,
291 };
292
293 EvalStatus status = ConsultPolicies(policies_to_consult,
294 &Policy::UpdateCanBeApplied,
295 ec,
296 state,
297 error,
298 result,
299 install_plan);
300 if (EvalStatus::kContinue != status) {
301 return status;
302 } else {
303 // The update can proceed.
304 LOG(INFO) << "Allowing update to be applied.";
305 *result = ErrorCode::kSuccess;
306 return EvalStatus::kSucceeded;
307 }
Aaron Wood23bd3392017-10-06 14:48:25 -0700308}
309
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700310EvalStatus ChromeOSPolicy::UpdateCanStart(
311 EvaluationContext* ec,
312 State* state,
313 string* error,
Gilad Arnold42f253b2014-06-25 12:39:17 -0700314 UpdateDownloadParams* result,
Gilad Arnoldd78caf92014-09-24 09:28:14 -0700315 const UpdateState update_state) const {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700316 // Set the default return values. Note that we set persisted values (backoff,
317 // scattering) to the same values presented in the update state. The reason is
318 // that preemptive returns, such as the case where an update check is due,
319 // should not clear off the said values; rather, it is the deliberate
320 // inference of new values that should cause them to be reset.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700321 result->update_can_start = false;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700322 result->cannot_start_reason = UpdateCannotStartReason::kUndefined;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700323 result->download_url_idx = -1;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700324 result->download_url_allowed = true;
325 result->download_url_num_errors = 0;
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700326 result->p2p_downloading_allowed = false;
327 result->p2p_sharing_allowed = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700328 result->do_increment_failures = false;
329 result->backoff_expiry = update_state.backoff_expiry;
330 result->scatter_wait_period = update_state.scatter_wait_period;
331 result->scatter_check_threshold = update_state.scatter_check_threshold;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700332
333 // Make sure that we're not due for an update check.
334 UpdateCheckParams check_result;
335 EvalStatus check_status = UpdateCheckAllowed(ec, state, error, &check_result);
336 if (check_status == EvalStatus::kFailed)
337 return EvalStatus::kFailed;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700338 bool is_check_due = (check_status == EvalStatus::kSucceeded &&
339 check_result.updates_enabled == true);
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700340
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700341 // Check whether backoff applies, and if not then which URL can be used for
342 // downloading. These require scanning the download error log, and so they are
343 // done together.
344 UpdateBackoffAndDownloadUrlResult backoff_url_result;
345 EvalStatus backoff_url_status = UpdateBackoffAndDownloadUrl(
346 ec, state, error, &backoff_url_result, update_state);
Gilad Arnold14a9e702014-10-08 08:09:09 -0700347 if (backoff_url_status == EvalStatus::kFailed)
348 return EvalStatus::kFailed;
349 result->download_url_idx = backoff_url_result.url_idx;
350 result->download_url_num_errors = backoff_url_result.url_num_errors;
351 result->do_increment_failures = backoff_url_result.do_increment_failures;
352 result->backoff_expiry = backoff_url_result.backoff_expiry;
353 bool is_backoff_active =
354 (backoff_url_status == EvalStatus::kAskMeAgainLater) ||
355 !backoff_url_result.backoff_expiry.is_null();
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700356
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700357 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Gilad Arnold14a9e702014-10-08 08:09:09 -0700358 bool is_scattering_active = false;
359 EvalStatus scattering_status = EvalStatus::kSucceeded;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700360
Amin Hassani4b717432019-01-14 16:24:20 -0800361 const bool* device_policy_is_loaded_p =
362 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700363 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Gilad Arnold76a11f62014-05-20 09:02:12 -0700364 // Check whether scattering applies to this update attempt. We should not be
365 // scattering if this is an interactive update check, or if OOBE is enabled
366 // but not completed.
367 //
368 // Note: current code further suppresses scattering if a "deadline"
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700369 // attribute is found in the Omaha response. However, it appears that the
Gilad Arnold76a11f62014-05-20 09:02:12 -0700370 // presence of this attribute is merely indicative of an OOBE update, during
371 // which we suppress scattering anyway.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700372 bool is_scattering_applicable = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700373 result->scatter_wait_period = kZeroInterval;
374 result->scatter_check_threshold = 0;
Amin Hassanied37d682018-04-06 13:22:00 -0700375 if (!update_state.interactive) {
Amin Hassani4b717432019-01-14 16:24:20 -0800376 const bool* is_oobe_enabled_p =
377 ec->GetValue(state->config_provider()->var_is_oobe_enabled());
Gilad Arnold76a11f62014-05-20 09:02:12 -0700378 if (is_oobe_enabled_p && !(*is_oobe_enabled_p)) {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700379 is_scattering_applicable = true;
Gilad Arnold76a11f62014-05-20 09:02:12 -0700380 } else {
Amin Hassani4b717432019-01-14 16:24:20 -0800381 const bool* is_oobe_complete_p =
382 ec->GetValue(state->system_provider()->var_is_oobe_complete());
Gilad Arnold14a9e702014-10-08 08:09:09 -0700383 is_scattering_applicable = (is_oobe_complete_p && *is_oobe_complete_p);
Gilad Arnold76a11f62014-05-20 09:02:12 -0700384 }
385 }
386
387 // Compute scattering values.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700388 if (is_scattering_applicable) {
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700389 UpdateScatteringResult scatter_result;
Amin Hassani4b717432019-01-14 16:24:20 -0800390 scattering_status =
391 UpdateScattering(ec, state, error, &scatter_result, update_state);
Gilad Arnold14a9e702014-10-08 08:09:09 -0700392 if (scattering_status == EvalStatus::kFailed) {
393 return EvalStatus::kFailed;
394 } else {
395 result->scatter_wait_period = scatter_result.wait_period;
396 result->scatter_check_threshold = scatter_result.check_threshold;
397 if (scattering_status == EvalStatus::kAskMeAgainLater ||
398 scatter_result.is_scattering)
399 is_scattering_active = true;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700400 }
401 }
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700402 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700403
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700404 // Find out whether P2P is globally enabled.
405 bool p2p_enabled;
406 EvalStatus p2p_enabled_status = P2PEnabled(ec, state, error, &p2p_enabled);
407 if (p2p_enabled_status != EvalStatus::kSucceeded)
408 return EvalStatus::kFailed;
409
410 // Is P2P is enabled, consider allowing it for downloading and/or sharing.
411 if (p2p_enabled) {
412 // Sharing via P2P is allowed if not disabled by Omaha.
413 if (update_state.p2p_sharing_disabled) {
414 LOG(INFO) << "Blocked P2P sharing because it is disabled by Omaha.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700415 } else {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700416 result->p2p_sharing_allowed = true;
Gilad Arnoldef8d0872014-10-03 14:14:06 -0700417 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700418
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700419 // Downloading via P2P is allowed if not disabled by Omaha, an update is not
420 // interactive, and other limits haven't been reached.
421 if (update_state.p2p_downloading_disabled) {
422 LOG(INFO) << "Blocked P2P downloading because it is disabled by Omaha.";
Amin Hassanied37d682018-04-06 13:22:00 -0700423 } else if (update_state.interactive) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700424 LOG(INFO) << "Blocked P2P downloading because update is interactive.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700425 } else if (update_state.p2p_num_attempts >= kMaxP2PAttempts) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700426 LOG(INFO) << "Blocked P2P downloading as it was attempted too many "
427 "times.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700428 } else if (!update_state.p2p_first_attempted.is_null() &&
429 ec->IsWallclockTimeGreaterThan(
430 update_state.p2p_first_attempted +
431 TimeDelta::FromSeconds(kMaxP2PAttemptsPeriodInSeconds))) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700432 LOG(INFO) << "Blocked P2P downloading as its usage timespan exceeds "
433 "limit.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700434 } else {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700435 // P2P download is allowed; if backoff or scattering are active, be sure
436 // to suppress them, yet prevent any download URL from being used.
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700437 result->p2p_downloading_allowed = true;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700438 if (is_backoff_active || is_scattering_active) {
439 is_backoff_active = is_scattering_active = false;
440 result->download_url_allowed = false;
441 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700442 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700443 }
444
Gilad Arnold14a9e702014-10-08 08:09:09 -0700445 // Check for various deterrents.
446 if (is_check_due) {
447 result->cannot_start_reason = UpdateCannotStartReason::kCheckDue;
448 return EvalStatus::kSucceeded;
449 }
450 if (is_backoff_active) {
451 result->cannot_start_reason = UpdateCannotStartReason::kBackoff;
452 return backoff_url_status;
453 }
454 if (is_scattering_active) {
455 result->cannot_start_reason = UpdateCannotStartReason::kScattering;
456 return scattering_status;
457 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700458 if (result->download_url_idx < 0 && !result->p2p_downloading_allowed) {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700459 result->cannot_start_reason = UpdateCannotStartReason::kCannotDownload;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700460 return EvalStatus::kSucceeded;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700461 }
462
Gilad Arnold14a9e702014-10-08 08:09:09 -0700463 // Update is good to go.
464 result->update_can_start = true;
Gilad Arnoldaf2f6ae2014-04-28 14:14:52 -0700465 return EvalStatus::kSucceeded;
466}
467
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700468EvalStatus ChromeOSPolicy::P2PEnabled(EvaluationContext* ec,
469 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800470 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700471 bool* result) const {
472 bool enabled = false;
473
474 // Determine whether use of P2P is allowed by policy. Even if P2P is not
475 // explicitly allowed, we allow it if the device is enterprise enrolled (that
476 // is, missing or empty owner string).
477 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Amin Hassani4b717432019-01-14 16:24:20 -0800478 const bool* device_policy_is_loaded_p =
479 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700480 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Amin Hassani4b717432019-01-14 16:24:20 -0800481 const bool* policy_au_p2p_enabled_p =
482 ec->GetValue(dp_provider->var_au_p2p_enabled());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700483 if (policy_au_p2p_enabled_p) {
484 enabled = *policy_au_p2p_enabled_p;
485 } else {
Jae Hoon Kim9d9492f2019-06-17 14:52:48 -0700486 const bool* policy_has_owner_p =
487 ec->GetValue(dp_provider->var_has_owner());
488 if (!policy_has_owner_p || !*policy_has_owner_p)
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700489 enabled = true;
490 }
491 }
492
493 // Enable P2P, if so mandated by the updater configuration. This is additive
494 // to whether or not P2P is enabled by device policy.
495 if (!enabled) {
Amin Hassani4b717432019-01-14 16:24:20 -0800496 const bool* updater_p2p_enabled_p =
497 ec->GetValue(state->updater_provider()->var_p2p_enabled());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700498 enabled = updater_p2p_enabled_p && *updater_p2p_enabled_p;
499 }
500
501 *result = enabled;
502 return EvalStatus::kSucceeded;
503}
504
505EvalStatus ChromeOSPolicy::P2PEnabledChanged(EvaluationContext* ec,
506 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800507 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700508 bool* result,
509 bool prev_result) const {
510 EvalStatus status = P2PEnabled(ec, state, error, result);
511 if (status == EvalStatus::kSucceeded && *result == prev_result)
512 return EvalStatus::kAskMeAgainLater;
513 return status;
514}
515
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700516EvalStatus ChromeOSPolicy::UpdateBackoffAndDownloadUrl(
Amin Hassani4b717432019-01-14 16:24:20 -0800517 EvaluationContext* ec,
518 State* state,
519 string* error,
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700520 UpdateBackoffAndDownloadUrlResult* result,
521 const UpdateState& update_state) const {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700522 DCHECK_GE(update_state.download_errors_max, 0);
523
524 // Set default result values.
525 result->do_increment_failures = false;
526 result->backoff_expiry = update_state.backoff_expiry;
527 result->url_idx = -1;
528 result->url_num_errors = 0;
529
Amin Hassani4b717432019-01-14 16:24:20 -0800530 const bool* is_official_build_p =
531 ec->GetValue(state->system_provider()->var_is_official_build());
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700532 bool is_official_build = (is_official_build_p ? *is_official_build_p : true);
533
534 // Check whether backoff is enabled.
535 bool may_backoff = false;
536 if (update_state.is_backoff_disabled) {
537 LOG(INFO) << "Backoff disabled by Omaha.";
Amin Hassanied37d682018-04-06 13:22:00 -0700538 } else if (update_state.interactive) {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700539 LOG(INFO) << "No backoff for interactive updates.";
540 } else if (update_state.is_delta_payload) {
541 LOG(INFO) << "No backoff for delta payloads.";
542 } else if (!is_official_build) {
543 LOG(INFO) << "No backoff for unofficial builds.";
544 } else {
545 may_backoff = true;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700546 }
547
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700548 // If previous backoff still in effect, block.
549 if (may_backoff && !update_state.backoff_expiry.is_null() &&
550 !ec->IsWallclockTimeGreaterThan(update_state.backoff_expiry)) {
551 LOG(INFO) << "Previous backoff has not expired, waiting.";
552 return EvalStatus::kAskMeAgainLater;
553 }
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700554
555 // Determine whether HTTP downloads are forbidden by policy. This only
556 // applies to official system builds; otherwise, HTTP is always enabled.
557 bool http_allowed = true;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700558 if (is_official_build) {
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700559 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Amin Hassani4b717432019-01-14 16:24:20 -0800560 const bool* device_policy_is_loaded_p =
561 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700562 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Amin Hassani4b717432019-01-14 16:24:20 -0800563 const bool* policy_http_downloads_enabled_p =
564 ec->GetValue(dp_provider->var_http_downloads_enabled());
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700565 http_allowed = (!policy_http_downloads_enabled_p ||
566 *policy_http_downloads_enabled_p);
567 }
568 }
569
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700570 int url_idx = update_state.last_download_url_idx;
571 if (url_idx < 0)
572 url_idx = -1;
573 bool do_advance_url = false;
574 bool is_failure_occurred = false;
575 Time err_time;
576
577 // Scan the relevant part of the download error log, tracking which URLs are
578 // being used, and accounting the number of errors for each URL. Note that
579 // this process may not traverse all errors provided, as it may decide to bail
580 // out midway depending on the particular errors exhibited, the number of
581 // failures allowed, etc. When this ends, |url_idx| will point to the last URL
582 // used (-1 if starting fresh), |do_advance_url| will determine whether the
583 // URL needs to be advanced, and |err_time| the point in time when the last
584 // reported error occurred. Additionally, if the error log indicates that an
585 // update attempt has failed (abnormal), then |is_failure_occurred| will be
586 // set to true.
587 const int num_urls = update_state.download_urls.size();
588 int prev_url_idx = -1;
589 int url_num_errors = update_state.last_download_url_num_errors;
590 Time prev_err_time;
591 bool is_first = true;
592 for (const auto& err_tuple : update_state.download_errors) {
Tianjiee283ce42020-07-29 11:37:51 -0700593 // Do some validation checks.
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700594 int used_url_idx = get<0>(err_tuple);
595 if (is_first && url_idx >= 0 && used_url_idx != url_idx) {
596 LOG(WARNING) << "First URL in error log (" << used_url_idx
597 << ") not as expected (" << url_idx << ")";
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700598 }
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700599 is_first = false;
600 url_idx = used_url_idx;
601 if (url_idx < 0 || url_idx >= num_urls) {
602 LOG(ERROR) << "Download error log contains an invalid URL index ("
603 << url_idx << ")";
604 return EvalStatus::kFailed;
605 }
606 err_time = get<2>(err_tuple);
607 if (!(prev_err_time.is_null() || err_time >= prev_err_time)) {
608 // TODO(garnold) Monotonicity cannot really be assumed when dealing with
609 // wallclock-based timestamps. However, we're making a simplifying
610 // assumption so as to keep the policy implementation straightforward, for
611 // now. In general, we should convert all timestamp handling in the
612 // UpdateManager to use monotonic time (instead of wallclock), including
613 // the computation of various expiration times (backoff, scattering, etc).
614 // The client will do whatever conversions necessary when
615 // persisting/retrieving these values across reboots. See chromium:408794.
616 LOG(ERROR) << "Download error timestamps not monotonically increasing.";
617 return EvalStatus::kFailed;
618 }
619 prev_err_time = err_time;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700620
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700621 // Ignore errors that happened before the last known failed attempt.
622 if (!update_state.failures_last_updated.is_null() &&
623 err_time <= update_state.failures_last_updated)
624 continue;
625
626 if (prev_url_idx >= 0) {
627 if (url_idx < prev_url_idx) {
628 LOG(ERROR) << "The URLs in the download error log have wrapped around ("
629 << prev_url_idx << "->" << url_idx
630 << "). This should not have happened and means that there's "
631 "a bug. To be conservative, we record a failed attempt "
632 "(invalidating the rest of the error log) and resume "
633 "download from the first usable URL.";
634 url_idx = -1;
635 is_failure_occurred = true;
636 break;
637 }
638
639 if (url_idx > prev_url_idx) {
640 url_num_errors = 0;
641 do_advance_url = false;
642 }
643 }
644
645 if (HandleErrorCode(get<1>(err_tuple), &url_num_errors) ||
646 url_num_errors > update_state.download_errors_max)
647 do_advance_url = true;
648
649 prev_url_idx = url_idx;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700650 }
651
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700652 // If required, advance to the next usable URL. If the URLs wraparound, we
653 // mark an update attempt failure. Also be sure to set the download error
654 // count to zero.
655 if (url_idx < 0 || do_advance_url) {
656 url_num_errors = 0;
657 int start_url_idx = -1;
658 do {
659 if (++url_idx == num_urls) {
660 url_idx = 0;
661 // We only mark failure if an actual advancing of a URL was required.
662 if (do_advance_url)
663 is_failure_occurred = true;
664 }
665
666 if (start_url_idx < 0)
667 start_url_idx = url_idx;
668 else if (url_idx == start_url_idx)
669 url_idx = -1; // No usable URL.
670 } while (url_idx >= 0 &&
671 !IsUrlUsable(update_state.download_urls[url_idx], http_allowed));
672 }
673
674 // If we have a download URL but a failure was observed, compute a new backoff
675 // expiry (if allowed). The backoff period is generally 2 ^ (num_failures - 1)
676 // days, bounded by the size of int and kAttemptBackoffMaxIntervalInDays, and
677 // fuzzed by kAttemptBackoffFuzzInHours hours. Backoff expiry is computed from
678 // the latest recorded time of error.
679 Time backoff_expiry;
680 if (url_idx >= 0 && is_failure_occurred && may_backoff) {
681 CHECK(!err_time.is_null())
682 << "We must have an error timestamp if a failure occurred!";
683 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
684 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
685 PRNG prng(*seed);
Amin Hassani4b717432019-01-14 16:24:20 -0800686 int exp =
687 min(update_state.num_failures, static_cast<int>(sizeof(int)) * 8 - 2);
Amin Hassani186ff6a2018-02-27 11:06:03 -0800688 TimeDelta backoff_interval = TimeDelta::FromDays(min(
689 1 << exp,
690 kNextUpdateCheckPolicyConstants.attempt_backoff_max_interval_in_days));
691 TimeDelta backoff_fuzz = TimeDelta::FromHours(
692 kNextUpdateCheckPolicyConstants.attempt_backoff_fuzz_in_hours);
693 TimeDelta wait_period = NextUpdateCheckTimePolicyImpl::FuzzedInterval(
694 &prng, backoff_interval.InSeconds(), backoff_fuzz.InSeconds());
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700695 backoff_expiry = err_time + wait_period;
696
697 // If the newly computed backoff already expired, nullify it.
698 if (ec->IsWallclockTimeGreaterThan(backoff_expiry))
699 backoff_expiry = Time();
700 }
701
702 result->do_increment_failures = is_failure_occurred;
703 result->backoff_expiry = backoff_expiry;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700704 result->url_idx = url_idx;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700705 result->url_num_errors = url_num_errors;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700706 return EvalStatus::kSucceeded;
707}
708
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700709EvalStatus ChromeOSPolicy::UpdateScattering(
710 EvaluationContext* ec,
711 State* state,
712 string* error,
713 UpdateScatteringResult* result,
714 const UpdateState& update_state) const {
715 // Preconditions. These stem from the postconditions and usage contract.
716 DCHECK(update_state.scatter_wait_period >= kZeroInterval);
717 DCHECK_GE(update_state.scatter_check_threshold, 0);
718
719 // Set default result values.
720 result->is_scattering = false;
721 result->wait_period = kZeroInterval;
722 result->check_threshold = 0;
723
724 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
725
726 // Ensure that a device policy is loaded.
Amin Hassani4b717432019-01-14 16:24:20 -0800727 const bool* device_policy_is_loaded_p =
728 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700729 if (!(device_policy_is_loaded_p && *device_policy_is_loaded_p))
730 return EvalStatus::kSucceeded;
731
732 // Is scattering enabled by policy?
Amin Hassani4b717432019-01-14 16:24:20 -0800733 const TimeDelta* scatter_factor_p =
734 ec->GetValue(dp_provider->var_scatter_factor());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700735 if (!scatter_factor_p || *scatter_factor_p == kZeroInterval)
736 return EvalStatus::kSucceeded;
737
738 // Obtain a pseudo-random number generator.
739 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
740 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
741 PRNG prng(*seed);
742
743 // Step 1: Maintain the scattering wait period.
744 //
745 // If no wait period was previously determined, or it no longer fits in the
746 // scatter factor, then generate a new one. Otherwise, keep the one we have.
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700747 TimeDelta wait_period = update_state.scatter_wait_period;
748 if (wait_period == kZeroInterval || wait_period > *scatter_factor_p) {
749 wait_period = TimeDelta::FromSeconds(
750 prng.RandMinMax(1, scatter_factor_p->InSeconds()));
751 }
752
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700753 // If we surpassed the wait period or the max scatter period associated with
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700754 // the update, then no wait is needed.
755 Time wait_expires = (update_state.first_seen +
756 min(wait_period, update_state.scatter_wait_period_max));
Gilad Arnolda65fced2014-07-23 09:01:31 -0700757 if (ec->IsWallclockTimeGreaterThan(wait_expires))
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700758 wait_period = kZeroInterval;
759
760 // Step 2: Maintain the update check threshold count.
761 //
762 // If an update check threshold is not specified then generate a new
763 // one.
764 int check_threshold = update_state.scatter_check_threshold;
765 if (check_threshold == 0) {
Amin Hassani4b717432019-01-14 16:24:20 -0800766 check_threshold = prng.RandMinMax(update_state.scatter_check_threshold_min,
767 update_state.scatter_check_threshold_max);
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700768 }
769
770 // If the update check threshold is not within allowed range then nullify it.
771 // TODO(garnold) This is compliant with current logic found in
772 // OmahaRequestAction::IsUpdateCheckCountBasedWaitingSatisfied(). We may want
773 // to change it so that it behaves similarly to the wait period case, namely
774 // if the current value exceeds the maximum, we set a new one within range.
775 if (check_threshold > update_state.scatter_check_threshold_max)
776 check_threshold = 0;
777
778 // If the update check threshold is non-zero and satisfied, then nullify it.
779 if (check_threshold > 0 && update_state.num_checks >= check_threshold)
780 check_threshold = 0;
781
782 bool is_scattering = (wait_period != kZeroInterval || check_threshold);
783 EvalStatus ret = EvalStatus::kSucceeded;
784 if (is_scattering && wait_period == update_state.scatter_wait_period &&
785 check_threshold == update_state.scatter_check_threshold)
786 ret = EvalStatus::kAskMeAgainLater;
787 result->is_scattering = is_scattering;
788 result->wait_period = wait_period;
789 result->check_threshold = check_threshold;
790 return ret;
791}
792
Alex Deymo63784a52014-05-28 10:46:14 -0700793} // namespace chromeos_update_manager