blob: 24b8293e4db14bb64fec24b13d1764986cd11fe6 [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;
Saurabh Nijhara43d7adc2020-11-06 16:13:02 +0100226 result->quick_fix_build_token.clear();
Gilad Arnold42f253b2014-06-25 12:39:17 -0700227
Amin Hassani186ff6a2018-02-27 11:06:03 -0800228 EnoughSlotsAbUpdatesPolicyImpl enough_slots_ab_updates_policy;
229 EnterpriseDevicePolicyImpl enterprise_device_policy;
230 OnlyUpdateOfficialBuildsPolicyImpl only_update_official_builds_policy;
231 InteractiveUpdatePolicyImpl interactive_update_policy;
232 OobePolicyImpl oobe_policy;
233 NextUpdateCheckTimePolicyImpl next_update_check_time_policy(
234 kNextUpdateCheckPolicyConstants);
Gilad Arnolda1eabcd2014-07-09 15:42:40 -0700235
Amin Hassani186ff6a2018-02-27 11:06:03 -0800236 vector<Policy const*> policies_to_consult = {
237 // Do not perform any updates if there are not enough slots to do A/B
238 // updates.
239 &enough_slots_ab_updates_policy,
240
241 // Check to see if Enterprise-managed (has DevicePolicy) and/or
242 // Kiosk-mode. If so, then defer to those settings.
243 &enterprise_device_policy,
244
245 // Check to see if an interactive update was requested.
246 &interactive_update_policy,
247
248 // Unofficial builds should not perform periodic update checks.
249 &only_update_official_builds_policy,
250
251 // If OOBE is enabled, wait until it is completed.
252 &oobe_policy,
253
254 // Ensure that periodic update checks are timed properly.
255 &next_update_check_time_policy,
256 };
257
258 // Now that the list of policy implementations, and the order to consult them,
259 // has been setup, consult the policies. If none of the policies make a
260 // definitive decisions about whether or not to check for updates, then allow
261 // the update check to happen.
262 EvalStatus status = ConsultPolicies(policies_to_consult,
263 &Policy::UpdateCheckAllowed,
264 ec,
265 state,
266 error,
267 result);
268 if (EvalStatus::kContinue != status) {
269 return status;
270 } else {
271 // It is time to check for an update.
272 LOG(INFO) << "Allowing update check.";
Gilad Arnoldbfc44f72014-07-09 14:41:39 -0700273 return EvalStatus::kSucceeded;
274 }
Alex Deymoc705cc82014-02-19 11:15:00 -0800275}
276
Aaron Wood23bd3392017-10-06 14:48:25 -0700277EvalStatus ChromeOSPolicy::UpdateCanBeApplied(EvaluationContext* ec,
278 State* state,
279 std::string* error,
280 ErrorCode* result,
281 InstallPlan* install_plan) const {
Adolfo Victoria0dbf1f92018-07-19 14:18:11 -0700282 UpdateTimeRestrictionsPolicyImpl update_time_restrictions_policy;
283 InteractiveUpdatePolicyImpl interactive_update_policy;
284
285 vector<Policy const*> policies_to_consult = {
286 // Check to see if an interactive update has been requested.
287 &interactive_update_policy,
288
289 // Do not apply or download an update if we are inside one of the
290 // restricted times.
291 &update_time_restrictions_policy,
292 };
293
294 EvalStatus status = ConsultPolicies(policies_to_consult,
295 &Policy::UpdateCanBeApplied,
296 ec,
297 state,
298 error,
299 result,
300 install_plan);
301 if (EvalStatus::kContinue != status) {
302 return status;
303 } else {
304 // The update can proceed.
305 LOG(INFO) << "Allowing update to be applied.";
306 *result = ErrorCode::kSuccess;
307 return EvalStatus::kSucceeded;
308 }
Aaron Wood23bd3392017-10-06 14:48:25 -0700309}
310
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700311EvalStatus ChromeOSPolicy::UpdateCanStart(
312 EvaluationContext* ec,
313 State* state,
314 string* error,
Gilad Arnold42f253b2014-06-25 12:39:17 -0700315 UpdateDownloadParams* result,
Gilad Arnoldd78caf92014-09-24 09:28:14 -0700316 const UpdateState update_state) const {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700317 // Set the default return values. Note that we set persisted values (backoff,
318 // scattering) to the same values presented in the update state. The reason is
319 // that preemptive returns, such as the case where an update check is due,
320 // should not clear off the said values; rather, it is the deliberate
321 // inference of new values that should cause them to be reset.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700322 result->update_can_start = false;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700323 result->cannot_start_reason = UpdateCannotStartReason::kUndefined;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700324 result->download_url_idx = -1;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700325 result->download_url_allowed = true;
326 result->download_url_num_errors = 0;
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700327 result->p2p_downloading_allowed = false;
328 result->p2p_sharing_allowed = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700329 result->do_increment_failures = false;
330 result->backoff_expiry = update_state.backoff_expiry;
331 result->scatter_wait_period = update_state.scatter_wait_period;
332 result->scatter_check_threshold = update_state.scatter_check_threshold;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700333
334 // Make sure that we're not due for an update check.
335 UpdateCheckParams check_result;
336 EvalStatus check_status = UpdateCheckAllowed(ec, state, error, &check_result);
337 if (check_status == EvalStatus::kFailed)
338 return EvalStatus::kFailed;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700339 bool is_check_due = (check_status == EvalStatus::kSucceeded &&
340 check_result.updates_enabled == true);
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700341
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700342 // Check whether backoff applies, and if not then which URL can be used for
343 // downloading. These require scanning the download error log, and so they are
344 // done together.
345 UpdateBackoffAndDownloadUrlResult backoff_url_result;
346 EvalStatus backoff_url_status = UpdateBackoffAndDownloadUrl(
347 ec, state, error, &backoff_url_result, update_state);
Gilad Arnold14a9e702014-10-08 08:09:09 -0700348 if (backoff_url_status == EvalStatus::kFailed)
349 return EvalStatus::kFailed;
350 result->download_url_idx = backoff_url_result.url_idx;
351 result->download_url_num_errors = backoff_url_result.url_num_errors;
352 result->do_increment_failures = backoff_url_result.do_increment_failures;
353 result->backoff_expiry = backoff_url_result.backoff_expiry;
354 bool is_backoff_active =
355 (backoff_url_status == EvalStatus::kAskMeAgainLater) ||
356 !backoff_url_result.backoff_expiry.is_null();
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700357
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700358 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Gilad Arnold14a9e702014-10-08 08:09:09 -0700359 bool is_scattering_active = false;
360 EvalStatus scattering_status = EvalStatus::kSucceeded;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700361
Amin Hassani4b717432019-01-14 16:24:20 -0800362 const bool* device_policy_is_loaded_p =
363 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700364 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Gilad Arnold76a11f62014-05-20 09:02:12 -0700365 // Check whether scattering applies to this update attempt. We should not be
366 // scattering if this is an interactive update check, or if OOBE is enabled
367 // but not completed.
368 //
369 // Note: current code further suppresses scattering if a "deadline"
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700370 // attribute is found in the Omaha response. However, it appears that the
Gilad Arnold76a11f62014-05-20 09:02:12 -0700371 // presence of this attribute is merely indicative of an OOBE update, during
372 // which we suppress scattering anyway.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700373 bool is_scattering_applicable = false;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700374 result->scatter_wait_period = kZeroInterval;
375 result->scatter_check_threshold = 0;
Amin Hassanied37d682018-04-06 13:22:00 -0700376 if (!update_state.interactive) {
Amin Hassani4b717432019-01-14 16:24:20 -0800377 const bool* is_oobe_enabled_p =
378 ec->GetValue(state->config_provider()->var_is_oobe_enabled());
Gilad Arnold76a11f62014-05-20 09:02:12 -0700379 if (is_oobe_enabled_p && !(*is_oobe_enabled_p)) {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700380 is_scattering_applicable = true;
Gilad Arnold76a11f62014-05-20 09:02:12 -0700381 } else {
Amin Hassani4b717432019-01-14 16:24:20 -0800382 const bool* is_oobe_complete_p =
383 ec->GetValue(state->system_provider()->var_is_oobe_complete());
Gilad Arnold14a9e702014-10-08 08:09:09 -0700384 is_scattering_applicable = (is_oobe_complete_p && *is_oobe_complete_p);
Gilad Arnold76a11f62014-05-20 09:02:12 -0700385 }
386 }
387
388 // Compute scattering values.
Gilad Arnold14a9e702014-10-08 08:09:09 -0700389 if (is_scattering_applicable) {
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700390 UpdateScatteringResult scatter_result;
Amin Hassani4b717432019-01-14 16:24:20 -0800391 scattering_status =
392 UpdateScattering(ec, state, error, &scatter_result, update_state);
Gilad Arnold14a9e702014-10-08 08:09:09 -0700393 if (scattering_status == EvalStatus::kFailed) {
394 return EvalStatus::kFailed;
395 } else {
396 result->scatter_wait_period = scatter_result.wait_period;
397 result->scatter_check_threshold = scatter_result.check_threshold;
398 if (scattering_status == EvalStatus::kAskMeAgainLater ||
399 scatter_result.is_scattering)
400 is_scattering_active = true;
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700401 }
402 }
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700403 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700404
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700405 // Find out whether P2P is globally enabled.
406 bool p2p_enabled;
407 EvalStatus p2p_enabled_status = P2PEnabled(ec, state, error, &p2p_enabled);
408 if (p2p_enabled_status != EvalStatus::kSucceeded)
409 return EvalStatus::kFailed;
410
411 // Is P2P is enabled, consider allowing it for downloading and/or sharing.
412 if (p2p_enabled) {
413 // Sharing via P2P is allowed if not disabled by Omaha.
414 if (update_state.p2p_sharing_disabled) {
415 LOG(INFO) << "Blocked P2P sharing because it is disabled by Omaha.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700416 } else {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700417 result->p2p_sharing_allowed = true;
Gilad Arnoldef8d0872014-10-03 14:14:06 -0700418 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700419
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700420 // Downloading via P2P is allowed if not disabled by Omaha, an update is not
421 // interactive, and other limits haven't been reached.
422 if (update_state.p2p_downloading_disabled) {
423 LOG(INFO) << "Blocked P2P downloading because it is disabled by Omaha.";
Amin Hassanied37d682018-04-06 13:22:00 -0700424 } else if (update_state.interactive) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700425 LOG(INFO) << "Blocked P2P downloading because update is interactive.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700426 } else if (update_state.p2p_num_attempts >= kMaxP2PAttempts) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700427 LOG(INFO) << "Blocked P2P downloading as it was attempted too many "
428 "times.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700429 } else if (!update_state.p2p_first_attempted.is_null() &&
430 ec->IsWallclockTimeGreaterThan(
431 update_state.p2p_first_attempted +
432 TimeDelta::FromSeconds(kMaxP2PAttemptsPeriodInSeconds))) {
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700433 LOG(INFO) << "Blocked P2P downloading as its usage timespan exceeds "
434 "limit.";
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700435 } else {
Gilad Arnold14a9e702014-10-08 08:09:09 -0700436 // P2P download is allowed; if backoff or scattering are active, be sure
437 // to suppress them, yet prevent any download URL from being used.
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700438 result->p2p_downloading_allowed = true;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700439 if (is_backoff_active || is_scattering_active) {
440 is_backoff_active = is_scattering_active = false;
441 result->download_url_allowed = false;
442 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700443 }
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700444 }
445
Gilad Arnold14a9e702014-10-08 08:09:09 -0700446 // Check for various deterrents.
447 if (is_check_due) {
448 result->cannot_start_reason = UpdateCannotStartReason::kCheckDue;
449 return EvalStatus::kSucceeded;
450 }
451 if (is_backoff_active) {
452 result->cannot_start_reason = UpdateCannotStartReason::kBackoff;
453 return backoff_url_status;
454 }
455 if (is_scattering_active) {
456 result->cannot_start_reason = UpdateCannotStartReason::kScattering;
457 return scattering_status;
458 }
Gilad Arnoldb2f99192014-10-07 13:01:52 -0700459 if (result->download_url_idx < 0 && !result->p2p_downloading_allowed) {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700460 result->cannot_start_reason = UpdateCannotStartReason::kCannotDownload;
Gilad Arnold14a9e702014-10-08 08:09:09 -0700461 return EvalStatus::kSucceeded;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700462 }
463
Gilad Arnold14a9e702014-10-08 08:09:09 -0700464 // Update is good to go.
465 result->update_can_start = true;
Gilad Arnoldaf2f6ae2014-04-28 14:14:52 -0700466 return EvalStatus::kSucceeded;
467}
468
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700469EvalStatus ChromeOSPolicy::P2PEnabled(EvaluationContext* ec,
470 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800471 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700472 bool* result) const {
473 bool enabled = false;
474
475 // Determine whether use of P2P is allowed by policy. Even if P2P is not
476 // explicitly allowed, we allow it if the device is enterprise enrolled (that
477 // is, missing or empty owner string).
478 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Amin Hassani4b717432019-01-14 16:24:20 -0800479 const bool* device_policy_is_loaded_p =
480 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700481 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Amin Hassani4b717432019-01-14 16:24:20 -0800482 const bool* policy_au_p2p_enabled_p =
483 ec->GetValue(dp_provider->var_au_p2p_enabled());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700484 if (policy_au_p2p_enabled_p) {
485 enabled = *policy_au_p2p_enabled_p;
486 } else {
Jae Hoon Kim9d9492f2019-06-17 14:52:48 -0700487 const bool* policy_has_owner_p =
488 ec->GetValue(dp_provider->var_has_owner());
489 if (!policy_has_owner_p || !*policy_has_owner_p)
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700490 enabled = true;
491 }
492 }
493
494 // Enable P2P, if so mandated by the updater configuration. This is additive
495 // to whether or not P2P is enabled by device policy.
496 if (!enabled) {
Amin Hassani4b717432019-01-14 16:24:20 -0800497 const bool* updater_p2p_enabled_p =
498 ec->GetValue(state->updater_provider()->var_p2p_enabled());
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700499 enabled = updater_p2p_enabled_p && *updater_p2p_enabled_p;
500 }
501
502 *result = enabled;
503 return EvalStatus::kSucceeded;
504}
505
506EvalStatus ChromeOSPolicy::P2PEnabledChanged(EvaluationContext* ec,
507 State* state,
Alex Deymo39910dc2015-11-09 17:04:30 -0800508 string* error,
Gilad Arnold78ecbfc2014-10-22 14:38:25 -0700509 bool* result,
510 bool prev_result) const {
511 EvalStatus status = P2PEnabled(ec, state, error, result);
512 if (status == EvalStatus::kSucceeded && *result == prev_result)
513 return EvalStatus::kAskMeAgainLater;
514 return status;
515}
516
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700517EvalStatus ChromeOSPolicy::UpdateBackoffAndDownloadUrl(
Amin Hassani4b717432019-01-14 16:24:20 -0800518 EvaluationContext* ec,
519 State* state,
520 string* error,
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700521 UpdateBackoffAndDownloadUrlResult* result,
522 const UpdateState& update_state) const {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700523 DCHECK_GE(update_state.download_errors_max, 0);
524
525 // Set default result values.
526 result->do_increment_failures = false;
527 result->backoff_expiry = update_state.backoff_expiry;
528 result->url_idx = -1;
529 result->url_num_errors = 0;
530
Amin Hassani4b717432019-01-14 16:24:20 -0800531 const bool* is_official_build_p =
532 ec->GetValue(state->system_provider()->var_is_official_build());
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700533 bool is_official_build = (is_official_build_p ? *is_official_build_p : true);
534
535 // Check whether backoff is enabled.
536 bool may_backoff = false;
537 if (update_state.is_backoff_disabled) {
538 LOG(INFO) << "Backoff disabled by Omaha.";
Amin Hassanied37d682018-04-06 13:22:00 -0700539 } else if (update_state.interactive) {
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700540 LOG(INFO) << "No backoff for interactive updates.";
541 } else if (update_state.is_delta_payload) {
542 LOG(INFO) << "No backoff for delta payloads.";
543 } else if (!is_official_build) {
544 LOG(INFO) << "No backoff for unofficial builds.";
545 } else {
546 may_backoff = true;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700547 }
548
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700549 // If previous backoff still in effect, block.
550 if (may_backoff && !update_state.backoff_expiry.is_null() &&
551 !ec->IsWallclockTimeGreaterThan(update_state.backoff_expiry)) {
552 LOG(INFO) << "Previous backoff has not expired, waiting.";
553 return EvalStatus::kAskMeAgainLater;
554 }
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700555
556 // Determine whether HTTP downloads are forbidden by policy. This only
557 // applies to official system builds; otherwise, HTTP is always enabled.
558 bool http_allowed = true;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700559 if (is_official_build) {
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700560 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
Amin Hassani4b717432019-01-14 16:24:20 -0800561 const bool* device_policy_is_loaded_p =
562 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700563 if (device_policy_is_loaded_p && *device_policy_is_loaded_p) {
Amin Hassani4b717432019-01-14 16:24:20 -0800564 const bool* policy_http_downloads_enabled_p =
565 ec->GetValue(dp_provider->var_http_downloads_enabled());
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700566 http_allowed = (!policy_http_downloads_enabled_p ||
567 *policy_http_downloads_enabled_p);
568 }
569 }
570
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700571 int url_idx = update_state.last_download_url_idx;
572 if (url_idx < 0)
573 url_idx = -1;
574 bool do_advance_url = false;
575 bool is_failure_occurred = false;
576 Time err_time;
577
578 // Scan the relevant part of the download error log, tracking which URLs are
579 // being used, and accounting the number of errors for each URL. Note that
580 // this process may not traverse all errors provided, as it may decide to bail
581 // out midway depending on the particular errors exhibited, the number of
582 // failures allowed, etc. When this ends, |url_idx| will point to the last URL
583 // used (-1 if starting fresh), |do_advance_url| will determine whether the
584 // URL needs to be advanced, and |err_time| the point in time when the last
585 // reported error occurred. Additionally, if the error log indicates that an
586 // update attempt has failed (abnormal), then |is_failure_occurred| will be
587 // set to true.
588 const int num_urls = update_state.download_urls.size();
589 int prev_url_idx = -1;
590 int url_num_errors = update_state.last_download_url_num_errors;
591 Time prev_err_time;
592 bool is_first = true;
593 for (const auto& err_tuple : update_state.download_errors) {
Tianjiee283ce42020-07-29 11:37:51 -0700594 // Do some validation checks.
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700595 int used_url_idx = get<0>(err_tuple);
596 if (is_first && url_idx >= 0 && used_url_idx != url_idx) {
597 LOG(WARNING) << "First URL in error log (" << used_url_idx
598 << ") not as expected (" << url_idx << ")";
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700599 }
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700600 is_first = false;
601 url_idx = used_url_idx;
602 if (url_idx < 0 || url_idx >= num_urls) {
603 LOG(ERROR) << "Download error log contains an invalid URL index ("
604 << url_idx << ")";
605 return EvalStatus::kFailed;
606 }
607 err_time = get<2>(err_tuple);
608 if (!(prev_err_time.is_null() || err_time >= prev_err_time)) {
609 // TODO(garnold) Monotonicity cannot really be assumed when dealing with
610 // wallclock-based timestamps. However, we're making a simplifying
611 // assumption so as to keep the policy implementation straightforward, for
612 // now. In general, we should convert all timestamp handling in the
613 // UpdateManager to use monotonic time (instead of wallclock), including
614 // the computation of various expiration times (backoff, scattering, etc).
615 // The client will do whatever conversions necessary when
616 // persisting/retrieving these values across reboots. See chromium:408794.
617 LOG(ERROR) << "Download error timestamps not monotonically increasing.";
618 return EvalStatus::kFailed;
619 }
620 prev_err_time = err_time;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700621
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700622 // Ignore errors that happened before the last known failed attempt.
623 if (!update_state.failures_last_updated.is_null() &&
624 err_time <= update_state.failures_last_updated)
625 continue;
626
627 if (prev_url_idx >= 0) {
628 if (url_idx < prev_url_idx) {
629 LOG(ERROR) << "The URLs in the download error log have wrapped around ("
630 << prev_url_idx << "->" << url_idx
631 << "). This should not have happened and means that there's "
632 "a bug. To be conservative, we record a failed attempt "
633 "(invalidating the rest of the error log) and resume "
634 "download from the first usable URL.";
635 url_idx = -1;
636 is_failure_occurred = true;
637 break;
638 }
639
640 if (url_idx > prev_url_idx) {
641 url_num_errors = 0;
642 do_advance_url = false;
643 }
644 }
645
646 if (HandleErrorCode(get<1>(err_tuple), &url_num_errors) ||
647 url_num_errors > update_state.download_errors_max)
648 do_advance_url = true;
649
650 prev_url_idx = url_idx;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700651 }
652
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700653 // If required, advance to the next usable URL. If the URLs wraparound, we
654 // mark an update attempt failure. Also be sure to set the download error
655 // count to zero.
656 if (url_idx < 0 || do_advance_url) {
657 url_num_errors = 0;
658 int start_url_idx = -1;
659 do {
660 if (++url_idx == num_urls) {
661 url_idx = 0;
662 // We only mark failure if an actual advancing of a URL was required.
663 if (do_advance_url)
664 is_failure_occurred = true;
665 }
666
667 if (start_url_idx < 0)
668 start_url_idx = url_idx;
669 else if (url_idx == start_url_idx)
670 url_idx = -1; // No usable URL.
671 } while (url_idx >= 0 &&
672 !IsUrlUsable(update_state.download_urls[url_idx], http_allowed));
673 }
674
675 // If we have a download URL but a failure was observed, compute a new backoff
676 // expiry (if allowed). The backoff period is generally 2 ^ (num_failures - 1)
677 // days, bounded by the size of int and kAttemptBackoffMaxIntervalInDays, and
678 // fuzzed by kAttemptBackoffFuzzInHours hours. Backoff expiry is computed from
679 // the latest recorded time of error.
680 Time backoff_expiry;
681 if (url_idx >= 0 && is_failure_occurred && may_backoff) {
682 CHECK(!err_time.is_null())
683 << "We must have an error timestamp if a failure occurred!";
684 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
685 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
686 PRNG prng(*seed);
Amin Hassani4b717432019-01-14 16:24:20 -0800687 int exp =
688 min(update_state.num_failures, static_cast<int>(sizeof(int)) * 8 - 2);
Amin Hassani186ff6a2018-02-27 11:06:03 -0800689 TimeDelta backoff_interval = TimeDelta::FromDays(min(
690 1 << exp,
691 kNextUpdateCheckPolicyConstants.attempt_backoff_max_interval_in_days));
692 TimeDelta backoff_fuzz = TimeDelta::FromHours(
693 kNextUpdateCheckPolicyConstants.attempt_backoff_fuzz_in_hours);
694 TimeDelta wait_period = NextUpdateCheckTimePolicyImpl::FuzzedInterval(
695 &prng, backoff_interval.InSeconds(), backoff_fuzz.InSeconds());
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700696 backoff_expiry = err_time + wait_period;
697
698 // If the newly computed backoff already expired, nullify it.
699 if (ec->IsWallclockTimeGreaterThan(backoff_expiry))
700 backoff_expiry = Time();
701 }
702
703 result->do_increment_failures = is_failure_occurred;
704 result->backoff_expiry = backoff_expiry;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700705 result->url_idx = url_idx;
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700706 result->url_num_errors = url_num_errors;
Gilad Arnoldb3b05442014-05-30 14:25:05 -0700707 return EvalStatus::kSucceeded;
708}
709
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700710EvalStatus ChromeOSPolicy::UpdateScattering(
711 EvaluationContext* ec,
712 State* state,
713 string* error,
714 UpdateScatteringResult* result,
715 const UpdateState& update_state) const {
716 // Preconditions. These stem from the postconditions and usage contract.
717 DCHECK(update_state.scatter_wait_period >= kZeroInterval);
718 DCHECK_GE(update_state.scatter_check_threshold, 0);
719
720 // Set default result values.
721 result->is_scattering = false;
722 result->wait_period = kZeroInterval;
723 result->check_threshold = 0;
724
725 DevicePolicyProvider* const dp_provider = state->device_policy_provider();
726
727 // Ensure that a device policy is loaded.
Amin Hassani4b717432019-01-14 16:24:20 -0800728 const bool* device_policy_is_loaded_p =
729 ec->GetValue(dp_provider->var_device_policy_is_loaded());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700730 if (!(device_policy_is_loaded_p && *device_policy_is_loaded_p))
731 return EvalStatus::kSucceeded;
732
733 // Is scattering enabled by policy?
Amin Hassani4b717432019-01-14 16:24:20 -0800734 const TimeDelta* scatter_factor_p =
735 ec->GetValue(dp_provider->var_scatter_factor());
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700736 if (!scatter_factor_p || *scatter_factor_p == kZeroInterval)
737 return EvalStatus::kSucceeded;
738
739 // Obtain a pseudo-random number generator.
740 const uint64_t* seed = ec->GetValue(state->random_provider()->var_seed());
741 POLICY_CHECK_VALUE_AND_FAIL(seed, error);
742 PRNG prng(*seed);
743
744 // Step 1: Maintain the scattering wait period.
745 //
746 // If no wait period was previously determined, or it no longer fits in the
747 // scatter factor, then generate a new one. Otherwise, keep the one we have.
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700748 TimeDelta wait_period = update_state.scatter_wait_period;
749 if (wait_period == kZeroInterval || wait_period > *scatter_factor_p) {
750 wait_period = TimeDelta::FromSeconds(
751 prng.RandMinMax(1, scatter_factor_p->InSeconds()));
752 }
753
Gilad Arnolddc4bb262014-07-23 10:45:19 -0700754 // If we surpassed the wait period or the max scatter period associated with
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700755 // the update, then no wait is needed.
756 Time wait_expires = (update_state.first_seen +
757 min(wait_period, update_state.scatter_wait_period_max));
Gilad Arnolda65fced2014-07-23 09:01:31 -0700758 if (ec->IsWallclockTimeGreaterThan(wait_expires))
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700759 wait_period = kZeroInterval;
760
761 // Step 2: Maintain the update check threshold count.
762 //
763 // If an update check threshold is not specified then generate a new
764 // one.
765 int check_threshold = update_state.scatter_check_threshold;
766 if (check_threshold == 0) {
Amin Hassani4b717432019-01-14 16:24:20 -0800767 check_threshold = prng.RandMinMax(update_state.scatter_check_threshold_min,
768 update_state.scatter_check_threshold_max);
Gilad Arnoldf62a4b82014-05-01 07:41:07 -0700769 }
770
771 // If the update check threshold is not within allowed range then nullify it.
772 // TODO(garnold) This is compliant with current logic found in
773 // OmahaRequestAction::IsUpdateCheckCountBasedWaitingSatisfied(). We may want
774 // to change it so that it behaves similarly to the wait period case, namely
775 // if the current value exceeds the maximum, we set a new one within range.
776 if (check_threshold > update_state.scatter_check_threshold_max)
777 check_threshold = 0;
778
779 // If the update check threshold is non-zero and satisfied, then nullify it.
780 if (check_threshold > 0 && update_state.num_checks >= check_threshold)
781 check_threshold = 0;
782
783 bool is_scattering = (wait_period != kZeroInterval || check_threshold);
784 EvalStatus ret = EvalStatus::kSucceeded;
785 if (is_scattering && wait_period == update_state.scatter_wait_period &&
786 check_threshold == update_state.scatter_check_threshold)
787 ret = EvalStatus::kAskMeAgainLater;
788 result->is_scattering = is_scattering;
789 result->wait_period = wait_period;
790 result->check_threshold = check_threshold;
791 return ret;
792}
793
Alex Deymo63784a52014-05-28 10:46:14 -0700794} // namespace chromeos_update_manager