Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // |
| 2 | // Copyright (C) 2012 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 | // |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 16 | |
| 17 | #include "update_engine/update_attempter.h" |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 18 | |
Ben Chan | 9abb763 | 2014-08-07 00:10:53 -0700 | [diff] [blame] | 19 | #include <stdint.h> |
| 20 | |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 21 | #include <algorithm> |
Alex Deymo | bc91a27 | 2014-05-20 16:45:33 -0700 | [diff] [blame] | 22 | #include <memory> |
| 23 | #include <string> |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 24 | #include <utility> |
Alex Deymo | 44666f9 | 2014-07-22 20:29:24 -0700 | [diff] [blame] | 25 | #include <vector> |
Darin Petkov | 9d65b7b | 2010-07-20 09:13:01 -0700 | [diff] [blame] | 26 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 27 | #include <base/bind.h> |
Ben Chan | 06c76a4 | 2014-09-05 08:21:06 -0700 | [diff] [blame] | 28 | #include <base/files/file_util.h> |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 29 | #include <base/logging.h> |
Andrew de los Reyes | 4516810 | 2010-11-22 11:13:50 -0800 | [diff] [blame] | 30 | #include <base/rand_util.h> |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 31 | #include <base/strings/string_util.h> |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 32 | #include <base/strings/stringprintf.h> |
Sen Jiang | 2703ef4 | 2017-03-16 13:36:21 -0700 | [diff] [blame] | 33 | #include <brillo/data_encoding.h> |
Sen Jiang | e67bb5b | 2016-06-20 15:53:56 -0700 | [diff] [blame] | 34 | #include <brillo/errors/error_codes.h> |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 35 | #include <brillo/message_loops/message_loop.h> |
Patrick Dubroy | 7fbbe8a | 2011-08-01 17:28:22 +0200 | [diff] [blame] | 36 | #include <policy/device_policy.h> |
Alex Deymo | bc91a27 | 2014-05-20 16:45:33 -0700 | [diff] [blame] | 37 | #include <policy/libpolicy.h> |
Alex Deymo | d6deb1d | 2015-08-28 15:54:37 -0700 | [diff] [blame] | 38 | #include <update_engine/dbus-constants.h> |
Darin Petkov | 9d65b7b | 2010-07-20 09:13:01 -0700 | [diff] [blame] | 39 | |
Alex Deymo | 14c0da8 | 2016-07-20 16:45:45 -0700 | [diff] [blame] | 40 | #include "update_engine/certificate_checker.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 41 | #include "update_engine/common/boot_control_interface.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 42 | #include "update_engine/common/clock_interface.h" |
| 43 | #include "update_engine/common/constants.h" |
| 44 | #include "update_engine/common/hardware_interface.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 45 | #include "update_engine/common/platform_constants.h" |
| 46 | #include "update_engine/common/prefs_interface.h" |
| 47 | #include "update_engine/common/subprocess.h" |
| 48 | #include "update_engine/common/utils.h" |
Alex Deymo | 14c0da8 | 2016-07-20 16:45:45 -0700 | [diff] [blame] | 49 | #include "update_engine/libcurl_http_fetcher.h" |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 50 | #include "update_engine/metrics_reporter_interface.h" |
Darin Petkov | 6a5b322 | 2010-07-13 14:55:28 -0700 | [diff] [blame] | 51 | #include "update_engine/omaha_request_action.h" |
Darin Petkov | a4a8a8c | 2010-07-15 22:21:12 -0700 | [diff] [blame] | 52 | #include "update_engine/omaha_request_params.h" |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 53 | #include "update_engine/omaha_response_handler_action.h" |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 54 | #include "update_engine/p2p_manager.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 55 | #include "update_engine/payload_consumer/download_action.h" |
| 56 | #include "update_engine/payload_consumer/filesystem_verifier_action.h" |
| 57 | #include "update_engine/payload_consumer/postinstall_runner_action.h" |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 58 | #include "update_engine/payload_state_interface.h" |
Sen Jiang | b8c6a8f | 2016-06-07 17:33:17 -0700 | [diff] [blame] | 59 | #include "update_engine/power_manager_interface.h" |
Jay Srinivasan | 4348879 | 2012-06-19 00:25:31 -0700 | [diff] [blame] | 60 | #include "update_engine/system_state.h" |
Amin Hassani | 0882a51 | 2018-04-05 16:25:44 -0700 | [diff] [blame] | 61 | #include "update_engine/update_boot_flags_action.h" |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 62 | #include "update_engine/update_manager/policy.h" |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 63 | #include "update_engine/update_manager/policy_utils.h" |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 64 | #include "update_engine/update_manager/update_manager.h" |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 65 | #include "update_engine/update_status_utils.h" |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 66 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 67 | using base::Bind; |
| 68 | using base::Callback; |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 69 | using base::Time; |
Darin Petkov | af18305 | 2010-08-23 12:07:13 -0700 | [diff] [blame] | 70 | using base::TimeDelta; |
| 71 | using base::TimeTicks; |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 72 | using brillo::MessageLoop; |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 73 | using chromeos_update_manager::EvalStatus; |
| 74 | using chromeos_update_manager::Policy; |
| 75 | using chromeos_update_manager::UpdateCheckParams; |
Adolfo Victoria | 497044c | 2018-07-18 07:51:42 -0700 | [diff] [blame] | 76 | using chromeos_update_manager::CalculateStagingCase; |
| 77 | using chromeos_update_manager::StagingCase; |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 78 | using std::string; |
| 79 | using std::vector; |
Aaron Wood | bf5a252 | 2017-10-04 10:58:36 -0700 | [diff] [blame] | 80 | using update_engine::UpdateAttemptFlags; |
Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 81 | using update_engine::UpdateEngineStatus; |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 82 | |
| 83 | namespace chromeos_update_engine { |
| 84 | |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 85 | const int UpdateAttempter::kMaxDeltaUpdateFailures = 3; |
| 86 | |
Andrew de los Reyes | 4516810 | 2010-11-22 11:13:50 -0800 | [diff] [blame] | 87 | namespace { |
| 88 | const int kMaxConsecutiveObeyProxyRequests = 20; |
Gilad Arnold | 70e476e | 2013-07-30 16:01:13 -0700 | [diff] [blame] | 89 | |
Alex Deymo | 0d29854 | 2016-03-30 18:31:49 -0700 | [diff] [blame] | 90 | // Minimum threshold to broadcast an status update in progress and time. |
| 91 | const double kBroadcastThresholdProgress = 0.01; // 1% |
| 92 | const int kBroadcastThresholdSeconds = 10; |
| 93 | |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 94 | // By default autest bypasses scattering. If we want to test scattering, |
| 95 | // use kScheduledAUTestURLRequest. The URL used is same in both cases, but |
| 96 | // different params are passed to CheckForUpdate(). |
| 97 | const char kAUTestURLRequest[] = "autest"; |
| 98 | const char kScheduledAUTestURLRequest[] = "autest-scheduled"; |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 99 | } // namespace |
Andrew de los Reyes | 4516810 | 2010-11-22 11:13:50 -0800 | [diff] [blame] | 100 | |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 101 | // Turns a generic ErrorCode::kError to a generic error code specific |
Allie Wood | eb9e6d8 | 2015-04-17 13:55:30 -0700 | [diff] [blame] | 102 | // to |action| (e.g., ErrorCode::kFilesystemVerifierError). If |code| is |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 103 | // not ErrorCode::kError, or the action is not matched, returns |code| |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 104 | // unchanged. |
Sen Jiang | daeaa43 | 2018-10-09 18:18:45 -0700 | [diff] [blame] | 105 | ErrorCode GetErrorCodeForAction(AbstractAction* action, ErrorCode code) { |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 106 | if (code != ErrorCode::kError) |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 107 | return code; |
| 108 | |
| 109 | const string type = action->Type(); |
| 110 | if (type == OmahaRequestAction::StaticType()) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 111 | return ErrorCode::kOmahaRequestError; |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 112 | if (type == OmahaResponseHandlerAction::StaticType()) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 113 | return ErrorCode::kOmahaResponseHandlerError; |
Allie Wood | eb9e6d8 | 2015-04-17 13:55:30 -0700 | [diff] [blame] | 114 | if (type == FilesystemVerifierAction::StaticType()) |
| 115 | return ErrorCode::kFilesystemVerifierError; |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 116 | if (type == PostinstallRunnerAction::StaticType()) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 117 | return ErrorCode::kPostinstallRunnerError; |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 118 | |
| 119 | return code; |
| 120 | } |
| 121 | |
Sen Jiang | 1841408 | 2018-01-11 14:50:36 -0800 | [diff] [blame] | 122 | UpdateAttempter::UpdateAttempter(SystemState* system_state, |
| 123 | CertificateChecker* cert_checker) |
Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 124 | : processor_(new ActionProcessor()), |
| 125 | system_state_(system_state), |
Sen Jiang | 1841408 | 2018-01-11 14:50:36 -0800 | [diff] [blame] | 126 | cert_checker_(cert_checker) {} |
Darin Petkov | c6c135c | 2010-08-11 13:36:18 -0700 | [diff] [blame] | 127 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 128 | UpdateAttempter::~UpdateAttempter() { |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 129 | // CertificateChecker might not be initialized in unittests. |
| 130 | if (cert_checker_) |
| 131 | cert_checker_->SetObserver(nullptr); |
Alex Deymo | 2b4268c | 2015-12-04 13:56:25 -0800 | [diff] [blame] | 132 | // Release ourselves as the ActionProcessor's delegate to prevent |
| 133 | // re-scheduling the updates due to the processing stopped. |
| 134 | processor_->set_delegate(nullptr); |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 135 | } |
| 136 | |
Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 137 | void UpdateAttempter::Init() { |
| 138 | // Pulling from the SystemState can only be done after construction, since |
| 139 | // this is an aggregate of various objects (such as the UpdateAttempter), |
| 140 | // which requires them all to be constructed prior to it being used. |
| 141 | prefs_ = system_state_->prefs(); |
| 142 | omaha_request_params_ = system_state_->request_params(); |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 143 | |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 144 | if (cert_checker_) |
| 145 | cert_checker_->SetObserver(this); |
| 146 | |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 147 | // In case of update_engine restart without a reboot we need to restore the |
| 148 | // reboot needed state. |
| 149 | if (GetBootTimeAtUpdate(nullptr)) |
| 150 | status_ = UpdateStatus::UPDATED_NEED_REBOOT; |
| 151 | else |
| 152 | status_ = UpdateStatus::IDLE; |
Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 153 | } |
| 154 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 155 | void UpdateAttempter::ScheduleUpdates() { |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 156 | if (IsUpdateRunningOrScheduled()) |
| 157 | return; |
| 158 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 159 | chromeos_update_manager::UpdateManager* const update_manager = |
| 160 | system_state_->update_manager(); |
| 161 | CHECK(update_manager); |
| 162 | Callback<void(EvalStatus, const UpdateCheckParams&)> callback = Bind( |
| 163 | &UpdateAttempter::OnUpdateScheduled, base::Unretained(this)); |
| 164 | // We limit the async policy request to a reasonably short time, to avoid a |
| 165 | // starvation due to a transient bug. |
| 166 | update_manager->AsyncPolicyRequest(callback, &Policy::UpdateCheckAllowed); |
| 167 | waiting_for_scheduled_check_ = true; |
Darin Petkov | c6c135c | 2010-08-11 13:36:18 -0700 | [diff] [blame] | 168 | } |
| 169 | |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 170 | void UpdateAttempter::CertificateChecked(ServerToCheck server_to_check, |
| 171 | CertificateCheckResult result) { |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 172 | system_state_->metrics_reporter()->ReportCertificateCheckMetrics( |
| 173 | server_to_check, result); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 174 | } |
| 175 | |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 176 | bool UpdateAttempter::CheckAndReportDailyMetrics() { |
| 177 | int64_t stored_value; |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 178 | Time now = system_state_->clock()->GetWallclockTime(); |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 179 | if (system_state_->prefs()->Exists(kPrefsDailyMetricsLastReportedAt) && |
| 180 | system_state_->prefs()->GetInt64(kPrefsDailyMetricsLastReportedAt, |
| 181 | &stored_value)) { |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 182 | Time last_reported_at = Time::FromInternalValue(stored_value); |
| 183 | TimeDelta time_reported_since = now - last_reported_at; |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 184 | if (time_reported_since.InSeconds() < 0) { |
| 185 | LOG(WARNING) << "Last reported daily metrics " |
| 186 | << utils::FormatTimeDelta(time_reported_since) << " ago " |
| 187 | << "which is negative. Either the system clock is wrong or " |
| 188 | << "the kPrefsDailyMetricsLastReportedAt state variable " |
| 189 | << "is wrong."; |
| 190 | // In this case, report daily metrics to reset. |
| 191 | } else { |
| 192 | if (time_reported_since.InSeconds() < 24*60*60) { |
| 193 | LOG(INFO) << "Last reported daily metrics " |
| 194 | << utils::FormatTimeDelta(time_reported_since) << " ago."; |
| 195 | return false; |
| 196 | } |
| 197 | LOG(INFO) << "Last reported daily metrics " |
| 198 | << utils::FormatTimeDelta(time_reported_since) << " ago, " |
| 199 | << "which is more than 24 hours ago."; |
| 200 | } |
| 201 | } |
| 202 | |
| 203 | LOG(INFO) << "Reporting daily metrics."; |
| 204 | system_state_->prefs()->SetInt64(kPrefsDailyMetricsLastReportedAt, |
| 205 | now.ToInternalValue()); |
| 206 | |
| 207 | ReportOSAge(); |
| 208 | |
| 209 | return true; |
| 210 | } |
| 211 | |
| 212 | void UpdateAttempter::ReportOSAge() { |
| 213 | struct stat sb; |
| 214 | |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 215 | if (system_state_ == nullptr) |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 216 | return; |
| 217 | |
| 218 | if (stat("/etc/lsb-release", &sb) != 0) { |
David Zeuthen | d9aca3b | 2014-08-21 13:11:21 -0400 | [diff] [blame] | 219 | PLOG(ERROR) << "Error getting file status for /etc/lsb-release " |
| 220 | << "(Note: this may happen in some unit tests)"; |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 221 | return; |
| 222 | } |
| 223 | |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 224 | Time lsb_release_timestamp = utils::TimeFromStructTimespec(&sb.st_ctim); |
| 225 | Time now = system_state_->clock()->GetWallclockTime(); |
| 226 | TimeDelta age = now - lsb_release_timestamp; |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 227 | if (age.InSeconds() < 0) { |
| 228 | LOG(ERROR) << "The OS age (" << utils::FormatTimeDelta(age) |
David Zeuthen | d9aca3b | 2014-08-21 13:11:21 -0400 | [diff] [blame] | 229 | << ") is negative. Maybe the clock is wrong? " |
| 230 | << "(Note: this may happen in some unit tests.)"; |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 231 | return; |
| 232 | } |
| 233 | |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 234 | system_state_->metrics_reporter()->ReportDailyMetrics(age); |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 235 | } |
| 236 | |
Gilad Arnold | 28e2f39 | 2012-02-09 14:36:46 -0800 | [diff] [blame] | 237 | void UpdateAttempter::Update(const string& app_version, |
| 238 | const string& omaha_url, |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 239 | const string& target_channel, |
| 240 | const string& target_version_prefix, |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 241 | bool rollback_allowed, |
Andrew de los Reyes | fb2f461 | 2011-06-09 18:21:49 -0700 | [diff] [blame] | 242 | bool obey_proxies, |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 243 | bool interactive) { |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 244 | // This is normally called frequently enough so it's appropriate to use as a |
| 245 | // hook for reporting daily metrics. |
| 246 | // TODO(garnold) This should be hooked to a separate (reliable and consistent) |
| 247 | // timeout event. |
David Zeuthen | 985b112 | 2013-10-09 12:13:15 -0700 | [diff] [blame] | 248 | CheckAndReportDailyMetrics(); |
| 249 | |
Andrew de los Reyes | c1d5c93 | 2011-04-20 17:15:47 -0700 | [diff] [blame] | 250 | fake_update_success_ = false; |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 251 | if (status_ == UpdateStatus::UPDATED_NEED_REBOOT) { |
Thieu Le | 116fda3 | 2011-04-19 11:01:54 -0700 | [diff] [blame] | 252 | // Although we have applied an update, we still want to ping Omaha |
| 253 | // to ensure the number of active statistics is accurate. |
David Zeuthen | 33bae49 | 2014-02-25 16:16:18 -0800 | [diff] [blame] | 254 | // |
| 255 | // Also convey to the UpdateEngine.Check.Result metric that we're |
| 256 | // not performing an update check because of this. |
Andrew de los Reyes | 6b78e29 | 2010-05-10 15:54:39 -0700 | [diff] [blame] | 257 | LOG(INFO) << "Not updating b/c we already updated and we're waiting for " |
Thieu Le | 116fda3 | 2011-04-19 11:01:54 -0700 | [diff] [blame] | 258 | << "reboot, we'll ping Omaha instead"; |
Tianjie Xu | 282aa1f | 2017-09-05 13:42:45 -0700 | [diff] [blame] | 259 | system_state_->metrics_reporter()->ReportUpdateCheckMetrics( |
| 260 | system_state_, |
| 261 | metrics::CheckResult::kRebootPending, |
| 262 | metrics::CheckReaction::kUnset, |
| 263 | metrics::DownloadErrorCode::kUnset); |
Thieu Le | 116fda3 | 2011-04-19 11:01:54 -0700 | [diff] [blame] | 264 | PingOmaha(); |
Andrew de los Reyes | 6b78e29 | 2010-05-10 15:54:39 -0700 | [diff] [blame] | 265 | return; |
| 266 | } |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 267 | if (status_ != UpdateStatus::IDLE) { |
Andrew de los Reyes | 6b78e29 | 2010-05-10 15:54:39 -0700 | [diff] [blame] | 268 | // Update in progress. Do nothing |
| 269 | return; |
| 270 | } |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 271 | |
| 272 | if (!CalculateUpdateParams(app_version, |
| 273 | omaha_url, |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 274 | target_channel, |
| 275 | target_version_prefix, |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 276 | rollback_allowed, |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 277 | obey_proxies, |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 278 | interactive)) { |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 279 | return; |
| 280 | } |
| 281 | |
| 282 | BuildUpdateActions(interactive); |
| 283 | |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 284 | SetStatusAndNotify(UpdateStatus::CHECKING_FOR_UPDATE); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 285 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 286 | // Update the last check time here; it may be re-updated when an Omaha |
| 287 | // response is received, but this will prevent us from repeatedly scheduling |
| 288 | // checks in the case where a response is not received. |
| 289 | UpdateLastCheckedTime(); |
| 290 | |
Amin Hassani | 0882a51 | 2018-04-05 16:25:44 -0700 | [diff] [blame] | 291 | ScheduleProcessingStart(); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 292 | } |
| 293 | |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 294 | void UpdateAttempter::RefreshDevicePolicy() { |
| 295 | // Lazy initialize the policy provider, or reload the latest policy data. |
| 296 | if (!policy_provider_.get()) |
| 297 | policy_provider_.reset(new policy::PolicyProvider()); |
| 298 | policy_provider_->Reload(); |
| 299 | |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 300 | const policy::DevicePolicy* device_policy = nullptr; |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 301 | if (policy_provider_->device_policy_is_loaded()) |
| 302 | device_policy = &policy_provider_->GetDevicePolicy(); |
| 303 | |
| 304 | if (device_policy) |
| 305 | LOG(INFO) << "Device policies/settings present"; |
| 306 | else |
| 307 | LOG(INFO) << "No device policies/settings present."; |
| 308 | |
| 309 | system_state_->set_device_policy(device_policy); |
David Zeuthen | 92d9c8b | 2013-09-11 10:58:11 -0700 | [diff] [blame] | 310 | system_state_->p2p_manager()->SetDevicePolicy(device_policy); |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 311 | } |
| 312 | |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 313 | void UpdateAttempter::CalculateP2PParams(bool interactive) { |
| 314 | bool use_p2p_for_downloading = false; |
| 315 | bool use_p2p_for_sharing = false; |
| 316 | |
Thiemo Nagel | 27d9d53 | 2017-05-26 16:35:20 +0200 | [diff] [blame] | 317 | // Never use p2p for downloading in interactive checks unless the developer |
| 318 | // has opted in for it via a marker file. |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 319 | // |
Thiemo Nagel | 27d9d53 | 2017-05-26 16:35:20 +0200 | [diff] [blame] | 320 | // (Why would a developer want to opt in? If they are working on the |
| 321 | // update_engine or p2p codebases so they can actually test their code.) |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 322 | |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 323 | if (system_state_ != nullptr) { |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 324 | if (!system_state_->p2p_manager()->IsP2PEnabled()) { |
| 325 | LOG(INFO) << "p2p is not enabled - disallowing p2p for both" |
| 326 | << " downloading and sharing."; |
| 327 | } else { |
| 328 | // Allow p2p for sharing, even in interactive checks. |
| 329 | use_p2p_for_sharing = true; |
| 330 | if (!interactive) { |
| 331 | LOG(INFO) << "Non-interactive check - allowing p2p for downloading"; |
| 332 | use_p2p_for_downloading = true; |
| 333 | } else { |
David Zeuthen | 052d290 | 2013-09-06 11:41:30 -0700 | [diff] [blame] | 334 | LOG(INFO) << "Forcibly disabling use of p2p for downloading " |
| 335 | << "since this update attempt is interactive."; |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 336 | } |
| 337 | } |
| 338 | } |
| 339 | |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 340 | PayloadStateInterface* const payload_state = system_state_->payload_state(); |
| 341 | payload_state->SetUsingP2PForDownloading(use_p2p_for_downloading); |
| 342 | payload_state->SetUsingP2PForSharing(use_p2p_for_sharing); |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 343 | } |
| 344 | |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 345 | bool UpdateAttempter::CalculateUpdateParams(const string& app_version, |
| 346 | const string& omaha_url, |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 347 | const string& target_channel, |
| 348 | const string& target_version_prefix, |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 349 | bool rollback_allowed, |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 350 | bool obey_proxies, |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 351 | bool interactive) { |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 352 | http_response_code_ = 0; |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 353 | PayloadStateInterface* const payload_state = system_state_->payload_state(); |
Patrick Dubroy | 7fbbe8a | 2011-08-01 17:28:22 +0200 | [diff] [blame] | 354 | |
Alex Deymo | 749ecf1 | 2014-10-21 20:06:57 -0700 | [diff] [blame] | 355 | // Refresh the policy before computing all the update parameters. |
| 356 | RefreshDevicePolicy(); |
| 357 | |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 358 | // Check whether we need to clear the rollback-happened preference after |
| 359 | // policy is available again. |
| 360 | UpdateRollbackHappened(); |
| 361 | |
Xiyuan Xia | c0e8f9a | 2017-02-22 13:19:35 -0800 | [diff] [blame] | 362 | // Update the target version prefix. |
| 363 | omaha_request_params_->set_target_version_prefix(target_version_prefix); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 364 | |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 365 | // Set whether rollback is allowed. |
| 366 | omaha_request_params_->set_rollback_allowed(rollback_allowed); |
| 367 | |
Adolfo Victoria | 497044c | 2018-07-18 07:51:42 -0700 | [diff] [blame] | 368 | CalculateStagingParams(interactive); |
| 369 | // If staging_wait_time_ wasn't set, staging is off, use scattering instead. |
| 370 | if (staging_wait_time_.InSeconds() == 0) { |
| 371 | CalculateScatteringParams(interactive); |
| 372 | } |
Patrick Dubroy | 7fbbe8a | 2011-08-01 17:28:22 +0200 | [diff] [blame] | 373 | |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 374 | CalculateP2PParams(interactive); |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 375 | if (payload_state->GetUsingP2PForDownloading() || |
| 376 | payload_state->GetUsingP2PForSharing()) { |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 377 | // OK, p2p is to be used - start it and perform housekeeping. |
| 378 | if (!StartP2PAndPerformHousekeeping()) { |
| 379 | // If this fails, disable p2p for this attempt |
| 380 | LOG(INFO) << "Forcibly disabling use of p2p since starting p2p or " |
| 381 | << "performing housekeeping failed."; |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 382 | payload_state->SetUsingP2PForDownloading(false); |
| 383 | payload_state->SetUsingP2PForSharing(false); |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 384 | } |
| 385 | } |
| 386 | |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 387 | if (!omaha_request_params_->Init(app_version, |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 388 | omaha_url, |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 389 | interactive)) { |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 390 | LOG(ERROR) << "Unable to initialize Omaha request params."; |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 391 | return false; |
Darin Petkov | a4a8a8c | 2010-07-15 22:21:12 -0700 | [diff] [blame] | 392 | } |
Darin Petkov | 3aefa86 | 2010-12-07 14:45:00 -0800 | [diff] [blame] | 393 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 394 | // Set the target channel, if one was provided. |
| 395 | if (target_channel.empty()) { |
| 396 | LOG(INFO) << "No target channel mandated by policy."; |
| 397 | } else { |
| 398 | LOG(INFO) << "Setting target channel as mandated: " << target_channel; |
| 399 | // Pass in false for powerwash_allowed until we add it to the policy |
| 400 | // protobuf. |
Alex Deymo | d942f9d | 2015-11-06 16:11:50 -0800 | [diff] [blame] | 401 | string error_message; |
| 402 | if (!omaha_request_params_->SetTargetChannel(target_channel, false, |
| 403 | &error_message)) { |
| 404 | LOG(ERROR) << "Setting the channel failed: " << error_message; |
| 405 | } |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 406 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 407 | // Since this is the beginning of a new attempt, update the download |
| 408 | // channel. The download channel won't be updated until the next attempt, |
| 409 | // even if target channel changes meanwhile, so that how we'll know if we |
| 410 | // should cancel the current download attempt if there's such a change in |
| 411 | // target channel. |
| 412 | omaha_request_params_->UpdateDownloadChannel(); |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 413 | } |
| 414 | |
Gilad Arnold | afd70ed | 2014-09-30 16:27:06 -0700 | [diff] [blame] | 415 | LOG(INFO) << "target_version_prefix = " |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 416 | << omaha_request_params_->target_version_prefix() |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 417 | << ", rollback_allowed = " |
| 418 | << omaha_request_params_->rollback_allowed() |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 419 | << ", scatter_factor_in_seconds = " |
| 420 | << utils::FormatSecs(scatter_factor_.InSeconds()); |
| 421 | |
| 422 | LOG(INFO) << "Wall Clock Based Wait Enabled = " |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 423 | << omaha_request_params_->wall_clock_based_wait_enabled() |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 424 | << ", Update Check Count Wait Enabled = " |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 425 | << omaha_request_params_->update_check_count_wait_enabled() |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 426 | << ", Waiting Period = " << utils::FormatSecs( |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 427 | omaha_request_params_->waiting_period().InSeconds()); |
Jay Srinivasan | 0a70874 | 2012-03-20 11:26:12 -0700 | [diff] [blame] | 428 | |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 429 | LOG(INFO) << "Use p2p For Downloading = " |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 430 | << payload_state->GetUsingP2PForDownloading() |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 431 | << ", Use p2p For Sharing = " |
Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 432 | << payload_state->GetUsingP2PForSharing(); |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 433 | |
Andrew de los Reyes | 4516810 | 2010-11-22 11:13:50 -0800 | [diff] [blame] | 434 | obeying_proxies_ = true; |
| 435 | if (obey_proxies || proxy_manual_checks_ == 0) { |
| 436 | LOG(INFO) << "forced to obey proxies"; |
| 437 | // If forced to obey proxies, every 20th request will not use proxies |
| 438 | proxy_manual_checks_++; |
| 439 | LOG(INFO) << "proxy manual checks: " << proxy_manual_checks_; |
| 440 | if (proxy_manual_checks_ >= kMaxConsecutiveObeyProxyRequests) { |
| 441 | proxy_manual_checks_ = 0; |
| 442 | obeying_proxies_ = false; |
| 443 | } |
| 444 | } else if (base::RandInt(0, 4) == 0) { |
| 445 | obeying_proxies_ = false; |
| 446 | } |
| 447 | LOG_IF(INFO, !obeying_proxies_) << "To help ensure updates work, this update " |
| 448 | "check we are ignoring the proxy settings and using " |
| 449 | "direct connections."; |
| 450 | |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 451 | DisableDeltaUpdateIfNeeded(); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 452 | return true; |
| 453 | } |
| 454 | |
Gilad Arnold | b92f0df | 2013-01-10 16:32:45 -0800 | [diff] [blame] | 455 | void UpdateAttempter::CalculateScatteringParams(bool interactive) { |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 456 | // Take a copy of the old scatter value before we update it, as |
| 457 | // we need to update the waiting period if this value changes. |
| 458 | TimeDelta old_scatter_factor = scatter_factor_; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 459 | const policy::DevicePolicy* device_policy = system_state_->device_policy(); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 460 | if (device_policy) { |
Ben Chan | 9abb763 | 2014-08-07 00:10:53 -0700 | [diff] [blame] | 461 | int64_t new_scatter_factor_in_secs = 0; |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 462 | device_policy->GetScatterFactorInSeconds(&new_scatter_factor_in_secs); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 463 | if (new_scatter_factor_in_secs < 0) // sanitize input, just in case. |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 464 | new_scatter_factor_in_secs = 0; |
| 465 | scatter_factor_ = TimeDelta::FromSeconds(new_scatter_factor_in_secs); |
| 466 | } |
| 467 | |
| 468 | bool is_scatter_enabled = false; |
| 469 | if (scatter_factor_.InSeconds() == 0) { |
| 470 | LOG(INFO) << "Scattering disabled since scatter factor is set to 0"; |
Gilad Arnold | b92f0df | 2013-01-10 16:32:45 -0800 | [diff] [blame] | 471 | } else if (interactive) { |
| 472 | LOG(INFO) << "Scattering disabled as this is an interactive update check"; |
Alex Deymo | 46a9aae | 2016-05-04 20:20:11 -0700 | [diff] [blame] | 473 | } else if (system_state_->hardware()->IsOOBEEnabled() && |
| 474 | !system_state_->hardware()->IsOOBEComplete(nullptr)) { |
| 475 | LOG(INFO) << "Scattering disabled since OOBE is enabled but not complete " |
| 476 | "yet"; |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 477 | } else { |
| 478 | is_scatter_enabled = true; |
| 479 | LOG(INFO) << "Scattering is enabled"; |
| 480 | } |
| 481 | |
| 482 | if (is_scatter_enabled) { |
| 483 | // This means the scattering policy is turned on. |
| 484 | // Now check if we need to update the waiting period. The two cases |
| 485 | // in which we'd need to update the waiting period are: |
| 486 | // 1. First time in process or a scheduled check after a user-initiated one. |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 487 | // (omaha_request_params_->waiting_period will be zero in this case). |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 488 | // 2. Admin has changed the scattering policy value. |
| 489 | // (new scattering value will be different from old one in this case). |
Ben Chan | 9abb763 | 2014-08-07 00:10:53 -0700 | [diff] [blame] | 490 | int64_t wait_period_in_secs = 0; |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 491 | if (omaha_request_params_->waiting_period().InSeconds() == 0) { |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 492 | // First case. Check if we have a suitable value to set for |
| 493 | // the waiting period. |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 494 | if (prefs_->GetInt64(kPrefsWallClockScatteringWaitPeriod, |
| 495 | &wait_period_in_secs) && |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 496 | wait_period_in_secs > 0 && |
| 497 | wait_period_in_secs <= scatter_factor_.InSeconds()) { |
| 498 | // This means: |
| 499 | // 1. There's a persisted value for the waiting period available. |
| 500 | // 2. And that persisted value is still valid. |
| 501 | // So, in this case, we should reuse the persisted value instead of |
| 502 | // generating a new random value to improve the chances of a good |
| 503 | // distribution for scattering. |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 504 | omaha_request_params_->set_waiting_period( |
| 505 | TimeDelta::FromSeconds(wait_period_in_secs)); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 506 | LOG(INFO) << "Using persisted wall-clock waiting period: " << |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 507 | utils::FormatSecs( |
| 508 | omaha_request_params_->waiting_period().InSeconds()); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 509 | } else { |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 510 | // This means there's no persisted value for the waiting period |
| 511 | // available or its value is invalid given the new scatter_factor value. |
| 512 | // So, we should go ahead and regenerate a new value for the |
| 513 | // waiting period. |
| 514 | LOG(INFO) << "Persisted value not present or not valid (" |
| 515 | << utils::FormatSecs(wait_period_in_secs) |
| 516 | << ") for wall-clock waiting period."; |
| 517 | GenerateNewWaitingPeriod(); |
| 518 | } |
| 519 | } else if (scatter_factor_ != old_scatter_factor) { |
| 520 | // This means there's already a waiting period value, but we detected |
| 521 | // a change in the scattering policy value. So, we should regenerate the |
| 522 | // waiting period to make sure it's within the bounds of the new scatter |
| 523 | // factor value. |
| 524 | GenerateNewWaitingPeriod(); |
| 525 | } else { |
| 526 | // Neither the first time scattering is enabled nor the scattering value |
| 527 | // changed. Nothing to do. |
| 528 | LOG(INFO) << "Keeping current wall-clock waiting period: " << |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 529 | utils::FormatSecs( |
| 530 | omaha_request_params_->waiting_period().InSeconds()); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 531 | } |
| 532 | |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 533 | // The invariant at this point is that omaha_request_params_->waiting_period |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 534 | // is non-zero no matter which path we took above. |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 535 | LOG_IF(ERROR, omaha_request_params_->waiting_period().InSeconds() == 0) |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 536 | << "Waiting Period should NOT be zero at this point!!!"; |
| 537 | |
| 538 | // Since scattering is enabled, wall clock based wait will always be |
| 539 | // enabled. |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 540 | omaha_request_params_->set_wall_clock_based_wait_enabled(true); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 541 | |
| 542 | // If we don't have any issues in accessing the file system to update |
| 543 | // the update check count value, we'll turn that on as well. |
| 544 | bool decrement_succeeded = DecrementUpdateCheckCount(); |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 545 | omaha_request_params_->set_update_check_count_wait_enabled( |
| 546 | decrement_succeeded); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 547 | } else { |
| 548 | // This means the scattering feature is turned off or disabled for |
| 549 | // this particular update check. Make sure to disable |
| 550 | // all the knobs and artifacts so that we don't invoke any scattering |
| 551 | // related code. |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 552 | omaha_request_params_->set_wall_clock_based_wait_enabled(false); |
| 553 | omaha_request_params_->set_update_check_count_wait_enabled(false); |
| 554 | omaha_request_params_->set_waiting_period(TimeDelta::FromSeconds(0)); |
Adolfo Victoria | d3a1e35 | 2018-07-16 11:40:47 -0700 | [diff] [blame] | 555 | prefs_->Delete(kPrefsWallClockScatteringWaitPeriod); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 556 | prefs_->Delete(kPrefsUpdateCheckCount); |
| 557 | // Don't delete the UpdateFirstSeenAt file as we don't want manual checks |
| 558 | // that result in no-updates (e.g. due to server side throttling) to |
| 559 | // cause update starvation by having the client generate a new |
| 560 | // UpdateFirstSeenAt for each scheduled check that follows a manual check. |
| 561 | } |
| 562 | } |
| 563 | |
| 564 | void UpdateAttempter::GenerateNewWaitingPeriod() { |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 565 | omaha_request_params_->set_waiting_period(TimeDelta::FromSeconds( |
| 566 | base::RandInt(1, scatter_factor_.InSeconds()))); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 567 | |
| 568 | LOG(INFO) << "Generated new wall-clock waiting period: " << utils::FormatSecs( |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 569 | omaha_request_params_->waiting_period().InSeconds()); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 570 | |
| 571 | // Do a best-effort to persist this in all cases. Even if the persistence |
| 572 | // fails, we'll still be able to scatter based on our in-memory value. |
| 573 | // The persistence only helps in ensuring a good overall distribution |
| 574 | // across multiple devices if they tend to reboot too often. |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 575 | system_state_->payload_state()->SetScatteringWaitPeriod( |
| 576 | omaha_request_params_->waiting_period()); |
Jay Srinivasan | 21be075 | 2012-07-25 15:44:56 -0700 | [diff] [blame] | 577 | } |
| 578 | |
Adolfo Victoria | 497044c | 2018-07-18 07:51:42 -0700 | [diff] [blame] | 579 | void UpdateAttempter::CalculateStagingParams(bool interactive) { |
| 580 | bool oobe_complete = system_state_->hardware()->IsOOBEEnabled() && |
| 581 | system_state_->hardware()->IsOOBEComplete(nullptr); |
| 582 | auto device_policy = system_state_->device_policy(); |
| 583 | StagingCase staging_case = StagingCase::kOff; |
| 584 | if (device_policy && !interactive && oobe_complete) { |
| 585 | staging_wait_time_ = omaha_request_params_->waiting_period(); |
| 586 | staging_case = CalculateStagingCase( |
| 587 | device_policy, prefs_, &staging_wait_time_, &staging_schedule_); |
| 588 | } |
| 589 | switch (staging_case) { |
| 590 | case StagingCase::kOff: |
| 591 | // Staging is off, get rid of persisted value. |
| 592 | prefs_->Delete(kPrefsWallClockStagingWaitPeriod); |
| 593 | // Set |staging_wait_time_| to its default value so scattering can still |
| 594 | // be turned on |
| 595 | staging_wait_time_ = TimeDelta(); |
| 596 | break; |
| 597 | // Let the cases fall through since they just add, and never remove, steps |
| 598 | // to turning staging on. |
| 599 | case StagingCase::kNoSavedValue: |
| 600 | prefs_->SetInt64(kPrefsWallClockStagingWaitPeriod, |
| 601 | staging_wait_time_.InDays()); |
| 602 | case StagingCase::kSetStagingFromPref: |
| 603 | omaha_request_params_->set_waiting_period(staging_wait_time_); |
| 604 | case StagingCase::kNoAction: |
| 605 | // Staging is on, enable wallclock based wait so that its values get used. |
| 606 | omaha_request_params_->set_wall_clock_based_wait_enabled(true); |
| 607 | // Use UpdateCheckCount if possible to prevent devices updating all at |
| 608 | // once. |
| 609 | omaha_request_params_->set_update_check_count_wait_enabled( |
| 610 | DecrementUpdateCheckCount()); |
| 611 | // Scattering should not be turned on if staging is on, delete the |
| 612 | // existing scattering configuration. |
| 613 | prefs_->Delete(kPrefsWallClockScatteringWaitPeriod); |
| 614 | scatter_factor_ = TimeDelta(); |
| 615 | } |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 616 | } |
| 617 | |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 618 | void UpdateAttempter::BuildUpdateActions(bool interactive) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 619 | CHECK(!processor_->IsRunning()); |
| 620 | processor_->set_delegate(this); |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 621 | |
| 622 | // Actions: |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 623 | auto update_check_fetcher = std::make_unique<LibcurlHttpFetcher>( |
| 624 | GetProxyResolver(), system_state_->hardware()); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 625 | update_check_fetcher->set_server_to_check(ServerToCheck::kUpdate); |
Andrew de los Reyes | fb2f461 | 2011-06-09 18:21:49 -0700 | [diff] [blame] | 626 | // Try harder to connect to the network, esp when not interactive. |
| 627 | // See comment in libcurl_http_fetcher.cc. |
| 628 | update_check_fetcher->set_no_network_max_retries(interactive ? 1 : 3); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 629 | auto update_check_action = std::make_unique<OmahaRequestAction>( |
| 630 | system_state_, nullptr, std::move(update_check_fetcher), false); |
| 631 | auto response_handler_action = |
| 632 | std::make_unique<OmahaResponseHandlerAction>(system_state_); |
Amin Hassani | 0882a51 | 2018-04-05 16:25:44 -0700 | [diff] [blame] | 633 | auto update_boot_flags_action = |
| 634 | std::make_unique<UpdateBootFlagsAction>(system_state_->boot_control()); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 635 | auto download_started_action = std::make_unique<OmahaRequestAction>( |
Sen Jiang | 1841408 | 2018-01-11 14:50:36 -0800 | [diff] [blame] | 636 | system_state_, |
| 637 | new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted), |
| 638 | std::make_unique<LibcurlHttpFetcher>(GetProxyResolver(), |
| 639 | system_state_->hardware()), |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 640 | false); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 641 | |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 642 | LibcurlHttpFetcher* download_fetcher = |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 643 | new LibcurlHttpFetcher(GetProxyResolver(), system_state_->hardware()); |
| 644 | download_fetcher->set_server_to_check(ServerToCheck::kDownload); |
Sen Jiang | ee174a1 | 2017-12-21 17:38:08 -0800 | [diff] [blame] | 645 | if (interactive) |
| 646 | download_fetcher->set_max_retry_count(kDownloadMaxRetryCountInteractive); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 647 | auto download_action = |
| 648 | std::make_unique<DownloadAction>(prefs_, |
| 649 | system_state_->boot_control(), |
| 650 | system_state_->hardware(), |
| 651 | system_state_, |
| 652 | download_fetcher, // passes ownership |
| 653 | interactive); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 654 | download_action->set_delegate(this); |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 655 | |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 656 | auto download_finished_action = std::make_unique<OmahaRequestAction>( |
| 657 | system_state_, |
| 658 | new OmahaEvent(OmahaEvent::kTypeUpdateDownloadFinished), |
| 659 | std::make_unique<LibcurlHttpFetcher>(GetProxyResolver(), |
| 660 | system_state_->hardware()), |
| 661 | false); |
| 662 | auto filesystem_verifier_action = |
| 663 | std::make_unique<FilesystemVerifierAction>(); |
| 664 | auto update_complete_action = std::make_unique<OmahaRequestAction>( |
| 665 | system_state_, |
| 666 | new OmahaEvent(OmahaEvent::kTypeUpdateComplete), |
| 667 | std::make_unique<LibcurlHttpFetcher>(GetProxyResolver(), |
| 668 | system_state_->hardware()), |
| 669 | false); |
| 670 | |
| 671 | auto postinstall_runner_action = std::make_unique<PostinstallRunnerAction>( |
| 672 | system_state_->boot_control(), system_state_->hardware()); |
| 673 | postinstall_runner_action->set_delegate(this); |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 674 | |
| 675 | // Bond them together. We have to use the leaf-types when calling |
| 676 | // BondActions(). |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 677 | BondActions(update_check_action.get(), response_handler_action.get()); |
| 678 | BondActions(response_handler_action.get(), download_action.get()); |
| 679 | BondActions(download_action.get(), filesystem_verifier_action.get()); |
| 680 | BondActions(filesystem_verifier_action.get(), |
| 681 | postinstall_runner_action.get()); |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 682 | |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 683 | processor_->EnqueueAction(std::move(update_check_action)); |
| 684 | processor_->EnqueueAction(std::move(response_handler_action)); |
Amin Hassani | 0882a51 | 2018-04-05 16:25:44 -0700 | [diff] [blame] | 685 | processor_->EnqueueAction(std::move(update_boot_flags_action)); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 686 | processor_->EnqueueAction(std::move(download_started_action)); |
| 687 | processor_->EnqueueAction(std::move(download_action)); |
| 688 | processor_->EnqueueAction(std::move(download_finished_action)); |
| 689 | processor_->EnqueueAction(std::move(filesystem_verifier_action)); |
| 690 | processor_->EnqueueAction(std::move(postinstall_runner_action)); |
| 691 | processor_->EnqueueAction(std::move(update_complete_action)); |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 692 | } |
| 693 | |
Chris Sosa | 44b9b7e | 2014-04-02 13:53:46 -0700 | [diff] [blame] | 694 | bool UpdateAttempter::Rollback(bool powerwash) { |
| 695 | if (!CanRollback()) { |
| 696 | return false; |
| 697 | } |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 698 | |
Chris Sosa | 44b9b7e | 2014-04-02 13:53:46 -0700 | [diff] [blame] | 699 | // Extra check for enterprise-enrolled devices since they don't support |
| 700 | // powerwash. |
| 701 | if (powerwash) { |
| 702 | // Enterprise-enrolled devices have an empty owner in their device policy. |
| 703 | string owner; |
| 704 | RefreshDevicePolicy(); |
| 705 | const policy::DevicePolicy* device_policy = system_state_->device_policy(); |
| 706 | if (device_policy && (!device_policy->GetOwner(&owner) || owner.empty())) { |
| 707 | LOG(ERROR) << "Enterprise device detected. " |
| 708 | << "Cannot perform a powerwash for enterprise devices."; |
| 709 | return false; |
| 710 | } |
| 711 | } |
| 712 | |
| 713 | processor_->set_delegate(this); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 714 | |
Chris Sosa | 28e479c | 2013-07-12 11:39:53 -0700 | [diff] [blame] | 715 | // Initialize the default request params. |
| 716 | if (!omaha_request_params_->Init("", "", true)) { |
| 717 | LOG(ERROR) << "Unable to initialize Omaha request params."; |
| 718 | return false; |
| 719 | } |
| 720 | |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 721 | LOG(INFO) << "Setting rollback options."; |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 722 | install_plan_.reset(new InstallPlan()); |
| 723 | install_plan_->target_slot = GetRollbackSlot(); |
| 724 | install_plan_->source_slot = system_state_->boot_control()->GetCurrentSlot(); |
Chris Sosa | 76a29ae | 2013-07-11 17:59:24 -0700 | [diff] [blame] | 725 | |
Alex Deymo | 706a5ab | 2015-11-23 17:48:30 -0300 | [diff] [blame] | 726 | TEST_AND_RETURN_FALSE( |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 727 | install_plan_->LoadPartitionsFromSlots(system_state_->boot_control())); |
| 728 | install_plan_->powerwash_required = powerwash; |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 729 | |
| 730 | LOG(INFO) << "Using this install plan:"; |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 731 | install_plan_->Dump(); |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 732 | |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 733 | auto install_plan_action = |
| 734 | std::make_unique<InstallPlanAction>(*install_plan_); |
| 735 | auto postinstall_runner_action = std::make_unique<PostinstallRunnerAction>( |
| 736 | system_state_->boot_control(), system_state_->hardware()); |
| 737 | postinstall_runner_action->set_delegate(this); |
| 738 | BondActions(install_plan_action.get(), postinstall_runner_action.get()); |
| 739 | processor_->EnqueueAction(std::move(install_plan_action)); |
| 740 | processor_->EnqueueAction(std::move(postinstall_runner_action)); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 741 | |
| 742 | // Update the payload state for Rollback. |
| 743 | system_state_->payload_state()->Rollback(); |
| 744 | |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 745 | SetStatusAndNotify(UpdateStatus::ATTEMPTING_ROLLBACK); |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 746 | |
Amin Hassani | 0882a51 | 2018-04-05 16:25:44 -0700 | [diff] [blame] | 747 | ScheduleProcessingStart(); |
Chris Sosa | aa18e16 | 2013-06-20 13:20:30 -0700 | [diff] [blame] | 748 | return true; |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 749 | } |
| 750 | |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 751 | bool UpdateAttempter::CanRollback() const { |
Chris Sosa | 44b9b7e | 2014-04-02 13:53:46 -0700 | [diff] [blame] | 752 | // We can only rollback if the update_engine isn't busy and we have a valid |
| 753 | // rollback partition. |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 754 | return (status_ == UpdateStatus::IDLE && |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 755 | GetRollbackSlot() != BootControlInterface::kInvalidSlot); |
Alex Vakulenko | 2bddadd | 2014-03-27 13:23:46 -0700 | [diff] [blame] | 756 | } |
| 757 | |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 758 | BootControlInterface::Slot UpdateAttempter::GetRollbackSlot() const { |
| 759 | LOG(INFO) << "UpdateAttempter::GetRollbackSlot"; |
| 760 | const unsigned int num_slots = system_state_->boot_control()->GetNumSlots(); |
| 761 | const BootControlInterface::Slot current_slot = |
| 762 | system_state_->boot_control()->GetCurrentSlot(); |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 763 | |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 764 | LOG(INFO) << " Installed slots: " << num_slots; |
| 765 | LOG(INFO) << " Booted from slot: " |
| 766 | << BootControlInterface::SlotName(current_slot); |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 767 | |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 768 | if (current_slot == BootControlInterface::kInvalidSlot || num_slots < 2) { |
| 769 | LOG(INFO) << "Device is not updateable."; |
| 770 | return BootControlInterface::kInvalidSlot; |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 771 | } |
| 772 | |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 773 | vector<BootControlInterface::Slot> bootable_slots; |
Alex Deymo | f7ead81 | 2015-10-23 17:37:27 -0700 | [diff] [blame] | 774 | for (BootControlInterface::Slot slot = 0; slot < num_slots; slot++) { |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 775 | if (slot != current_slot && |
| 776 | system_state_->boot_control()->IsSlotBootable(slot)) { |
| 777 | LOG(INFO) << "Found bootable slot " |
| 778 | << BootControlInterface::SlotName(slot); |
| 779 | return slot; |
Alex Vakulenko | 59e253e | 2014-02-24 10:40:21 -0800 | [diff] [blame] | 780 | } |
| 781 | } |
Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 782 | LOG(INFO) << "No other bootable slot found."; |
| 783 | return BootControlInterface::kInvalidSlot; |
Alex Vakulenko | 2bddadd | 2014-03-27 13:23:46 -0700 | [diff] [blame] | 784 | } |
| 785 | |
Aaron Wood | 081c023 | 2017-10-19 17:14:58 -0700 | [diff] [blame] | 786 | bool UpdateAttempter::CheckForUpdate(const string& app_version, |
Jay Srinivasan | e73acab | 2012-07-10 14:34:03 -0700 | [diff] [blame] | 787 | const string& omaha_url, |
Aaron Wood | 081c023 | 2017-10-19 17:14:58 -0700 | [diff] [blame] | 788 | UpdateAttemptFlags flags) { |
| 789 | bool interactive = !(flags & UpdateAttemptFlags::kFlagNonInteractive); |
| 790 | |
| 791 | if (interactive && status_ != UpdateStatus::IDLE) { |
| 792 | // An update check is either in-progress, or an update has completed and the |
| 793 | // system is in UPDATED_NEED_REBOOT. Either way, don't do an interactive |
| 794 | // update at this time |
| 795 | LOG(INFO) << "Refusing to do an interactive update with an update already " |
| 796 | "in progress"; |
| 797 | return false; |
| 798 | } |
| 799 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 800 | LOG(INFO) << "Forced update check requested."; |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 801 | forced_app_version_.clear(); |
| 802 | forced_omaha_url_.clear(); |
| 803 | |
| 804 | // Certain conditions must be met to allow setting custom version and update |
| 805 | // server URLs. However, kScheduledAUTestURLRequest and kAUTestURLRequest are |
| 806 | // always allowed regardless of device state. |
| 807 | if (IsAnyUpdateSourceAllowed()) { |
| 808 | forced_app_version_ = app_version; |
| 809 | forced_omaha_url_ = omaha_url; |
| 810 | } |
| 811 | if (omaha_url == kScheduledAUTestURLRequest) { |
Alex Deymo | ac41a82 | 2015-09-15 20:52:53 -0700 | [diff] [blame] | 812 | forced_omaha_url_ = constants::kOmahaDefaultAUTestURL; |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 813 | // Pretend that it's not user-initiated even though it is, |
| 814 | // so as to test scattering logic, etc. which get kicked off |
| 815 | // only in scheduled update checks. |
| 816 | interactive = false; |
| 817 | } else if (omaha_url == kAUTestURLRequest) { |
Alex Deymo | ac41a82 | 2015-09-15 20:52:53 -0700 | [diff] [blame] | 818 | forced_omaha_url_ = constants::kOmahaDefaultAUTestURL; |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 819 | } |
| 820 | |
Aaron Wood | 081c023 | 2017-10-19 17:14:58 -0700 | [diff] [blame] | 821 | if (interactive) { |
| 822 | // Use the passed-in update attempt flags for this update attempt instead |
| 823 | // of the previously set ones. |
| 824 | current_update_attempt_flags_ = flags; |
| 825 | // Note: The caching for non-interactive update checks happens in |
| 826 | // OnUpdateScheduled(). |
| 827 | } |
| 828 | |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 829 | if (forced_update_pending_callback_.get()) { |
| 830 | // Make sure that a scheduling request is made prior to calling the forced |
| 831 | // update pending callback. |
| 832 | ScheduleUpdates(); |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 833 | forced_update_pending_callback_->Run(true, interactive); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 834 | } |
Aaron Wood | 081c023 | 2017-10-19 17:14:58 -0700 | [diff] [blame] | 835 | |
| 836 | return true; |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 837 | } |
| 838 | |
Darin Petkov | 296889c | 2010-07-23 16:20:54 -0700 | [diff] [blame] | 839 | bool UpdateAttempter::RebootIfNeeded() { |
Amin Hassani | e79eab6 | 2018-03-06 11:55:57 -0800 | [diff] [blame] | 840 | #ifdef __ANDROID__ |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 841 | if (status_ != UpdateStatus::UPDATED_NEED_REBOOT) { |
Darin Petkov | 296889c | 2010-07-23 16:20:54 -0700 | [diff] [blame] | 842 | LOG(INFO) << "Reboot requested, but status is " |
| 843 | << UpdateStatusToString(status_) << ", so not rebooting."; |
| 844 | return false; |
| 845 | } |
Amin Hassani | e79eab6 | 2018-03-06 11:55:57 -0800 | [diff] [blame] | 846 | #endif // __ANDROID__ |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 847 | |
Sen Jiang | b8c6a8f | 2016-06-07 17:33:17 -0700 | [diff] [blame] | 848 | if (system_state_->power_manager()->RequestReboot()) |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 849 | return true; |
| 850 | |
| 851 | return RebootDirectly(); |
Darin Petkov | 296889c | 2010-07-23 16:20:54 -0700 | [diff] [blame] | 852 | } |
| 853 | |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 854 | void UpdateAttempter::WriteUpdateCompletedMarker() { |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 855 | string boot_id; |
| 856 | if (!utils::GetBootId(&boot_id)) |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 857 | return; |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 858 | prefs_->SetString(kPrefsUpdateCompletedOnBootId, boot_id); |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 859 | |
| 860 | int64_t value = system_state_->clock()->GetBootTime().ToInternalValue(); |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 861 | prefs_->SetInt64(kPrefsUpdateCompletedBootTime, value); |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 862 | } |
| 863 | |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 864 | bool UpdateAttempter::RebootDirectly() { |
| 865 | vector<string> command; |
| 866 | command.push_back("/sbin/shutdown"); |
| 867 | command.push_back("-r"); |
| 868 | command.push_back("now"); |
Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 869 | LOG(INFO) << "Running \"" << base::JoinString(command, " ") << "\""; |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 870 | int rc = 0; |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 871 | Subprocess::SynchronousExec(command, &rc, nullptr); |
Daniel Erat | 65f1da0 | 2014-06-27 22:05:38 -0700 | [diff] [blame] | 872 | return rc == 0; |
| 873 | } |
| 874 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 875 | void UpdateAttempter::OnUpdateScheduled(EvalStatus status, |
| 876 | const UpdateCheckParams& params) { |
| 877 | waiting_for_scheduled_check_ = false; |
| 878 | |
| 879 | if (status == EvalStatus::kSucceeded) { |
| 880 | if (!params.updates_enabled) { |
| 881 | LOG(WARNING) << "Updates permanently disabled."; |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 882 | // Signal disabled status, then switch right back to idle. This is |
| 883 | // necessary for ensuring that observers waiting for a signal change will |
| 884 | // actually notice one on subsequent calls. Note that we don't need to |
| 885 | // re-schedule a check in this case as updates are permanently disabled; |
| 886 | // further (forced) checks may still initiate a scheduling call. |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 887 | SetStatusAndNotify(UpdateStatus::DISABLED); |
| 888 | SetStatusAndNotify(UpdateStatus::IDLE); |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 889 | return; |
| 890 | } |
| 891 | |
Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 892 | LOG(INFO) << "Running " << (params.interactive ? "interactive" : "periodic") |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 893 | << " update."; |
| 894 | |
Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 895 | if (!params.interactive) { |
Aaron Wood | 081c023 | 2017-10-19 17:14:58 -0700 | [diff] [blame] | 896 | // Cache the update attempt flags that will be used by this update attempt |
| 897 | // so that they can't be changed mid-way through. |
| 898 | current_update_attempt_flags_ = update_attempt_flags_; |
| 899 | } |
| 900 | |
Aaron Wood | bf5a252 | 2017-10-04 10:58:36 -0700 | [diff] [blame] | 901 | LOG(INFO) << "Update attempt flags in use = 0x" << std::hex |
| 902 | << current_update_attempt_flags_; |
| 903 | |
Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 904 | Update(forced_app_version_, |
| 905 | forced_omaha_url_, |
| 906 | params.target_channel, |
| 907 | params.target_version_prefix, |
Marton Hunyady | ba51c3f | 2018-04-25 15:18:10 +0200 | [diff] [blame] | 908 | params.rollback_allowed, |
| 909 | /*obey_proxies=*/false, |
Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 910 | params.interactive); |
Alex Deymo | 7147908 | 2016-03-25 17:54:28 -0700 | [diff] [blame] | 911 | // Always clear the forced app_version and omaha_url after an update attempt |
| 912 | // so the next update uses the defaults. |
| 913 | forced_app_version_.clear(); |
| 914 | forced_omaha_url_.clear(); |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 915 | } else { |
| 916 | LOG(WARNING) |
| 917 | << "Update check scheduling failed (possibly timed out); retrying."; |
| 918 | ScheduleUpdates(); |
| 919 | } |
| 920 | |
| 921 | // This check ensures that future update checks will be or are already |
| 922 | // scheduled. The check should never fail. A check failure means that there's |
| 923 | // a bug that will most likely prevent further automatic update checks. It |
| 924 | // seems better to crash in such cases and restart the update_engine daemon |
| 925 | // into, hopefully, a known good state. |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 926 | CHECK(IsUpdateRunningOrScheduled()); |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 927 | } |
| 928 | |
| 929 | void UpdateAttempter::UpdateLastCheckedTime() { |
| 930 | last_checked_time_ = system_state_->clock()->GetWallclockTime().ToTimeT(); |
| 931 | } |
| 932 | |
Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 933 | void UpdateAttempter::UpdateRollbackHappened() { |
| 934 | DCHECK(system_state_); |
| 935 | DCHECK(system_state_->payload_state()); |
| 936 | DCHECK(policy_provider_); |
| 937 | if (system_state_->payload_state()->GetRollbackHappened() && |
| 938 | (policy_provider_->device_policy_is_loaded() || |
| 939 | policy_provider_->IsConsumerDevice())) { |
| 940 | // Rollback happened, but we already went through OOBE and policy is |
| 941 | // present or it's a consumer device. |
| 942 | system_state_->payload_state()->SetRollbackHappened(false); |
| 943 | } |
| 944 | } |
| 945 | |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 946 | // Delegate methods: |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 947 | void UpdateAttempter::ProcessingDone(const ActionProcessor* processor, |
David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 948 | ErrorCode code) { |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 949 | LOG(INFO) << "Processing Done."; |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 950 | |
Chris Sosa | 4f8ee27 | 2012-11-30 13:01:54 -0800 | [diff] [blame] | 951 | // Reset cpu shares back to normal. |
Alex Deymo | ab0d976 | 2016-02-02 10:52:56 -0800 | [diff] [blame] | 952 | cpu_limiter_.StopLimiter(); |
Darin Petkov | c6c135c | 2010-08-11 13:36:18 -0700 | [diff] [blame] | 953 | |
Aaron Wood | bf5a252 | 2017-10-04 10:58:36 -0700 | [diff] [blame] | 954 | // reset the state that's only valid for a single update pass |
| 955 | current_update_attempt_flags_ = UpdateAttemptFlags::kNone; |
| 956 | |
Adolfo Victoria | 20262ad | 2018-08-06 10:40:11 -0700 | [diff] [blame] | 957 | if (forced_update_pending_callback_.get()) |
| 958 | // Clear prior interactive requests once the processor is done. |
| 959 | forced_update_pending_callback_->Run(false, false); |
| 960 | |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 961 | if (status_ == UpdateStatus::REPORTING_ERROR_EVENT) { |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 962 | LOG(INFO) << "Error event sent."; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 963 | |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 964 | // Inform scheduler of new status; |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 965 | SetStatusAndNotify(UpdateStatus::IDLE); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 966 | ScheduleUpdates(); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 967 | |
Andrew de los Reyes | c1d5c93 | 2011-04-20 17:15:47 -0700 | [diff] [blame] | 968 | if (!fake_update_success_) { |
| 969 | return; |
| 970 | } |
| 971 | LOG(INFO) << "Booted from FW B and tried to install new firmware, " |
| 972 | "so requesting reboot from user."; |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 973 | } |
| 974 | |
Sen Jiang | 89e24c1 | 2018-03-22 18:05:44 -0700 | [diff] [blame] | 975 | attempt_error_code_ = utils::GetBaseErrorCode(code); |
| 976 | |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 977 | if (code == ErrorCode::kSuccess) { |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 978 | WriteUpdateCompletedMarker(); |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 979 | prefs_->SetInt64(kPrefsDeltaUpdateFailures, 0); |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 980 | prefs_->SetString(kPrefsPreviousVersion, |
| 981 | omaha_request_params_->app_version()); |
Darin Petkov | 9b23057 | 2010-10-08 10:20:09 -0700 | [diff] [blame] | 982 | DeltaPerformer::ResetUpdateProgress(prefs_, false); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 983 | |
David Zeuthen | 9a017f2 | 2013-04-11 16:10:26 -0700 | [diff] [blame] | 984 | system_state_->payload_state()->UpdateSucceeded(); |
| 985 | |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 986 | // Since we're done with scattering fully at this point, this is the |
| 987 | // safest point delete the state files, as we're sure that the status is |
| 988 | // set to reboot (which means no more updates will be applied until reboot) |
| 989 | // This deletion is required for correctness as we want the next update |
| 990 | // check to re-create a new random number for the update check count. |
| 991 | // Similarly, we also delete the wall-clock-wait period that was persisted |
| 992 | // so that we start with a new random value for the next update check |
| 993 | // after reboot so that the same device is not favored or punished in any |
| 994 | // way. |
| 995 | prefs_->Delete(kPrefsUpdateCheckCount); |
Gilad Arnold | 519cfc7 | 2014-10-02 10:34:54 -0700 | [diff] [blame] | 996 | system_state_->payload_state()->SetScatteringWaitPeriod(TimeDelta()); |
Adolfo Victoria | 497044c | 2018-07-18 07:51:42 -0700 | [diff] [blame] | 997 | system_state_->payload_state()->SetStagingWaitPeriod(TimeDelta()); |
Jay Srinivasan | 34b5d86 | 2012-07-23 11:43:22 -0700 | [diff] [blame] | 998 | prefs_->Delete(kPrefsUpdateFirstSeenAt); |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 999 | |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1000 | SetStatusAndNotify(UpdateStatus::UPDATED_NEED_REBOOT); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1001 | ScheduleUpdates(); |
Jay Srinivasan | 19409b7 | 2013-04-12 19:23:36 -0700 | [diff] [blame] | 1002 | LOG(INFO) << "Update successfully applied, waiting to reboot."; |
Darin Petkov | 9d65b7b | 2010-07-20 09:13:01 -0700 | [diff] [blame] | 1003 | |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1004 | // |install_plan_| is null during rollback operations, and the stats don't |
| 1005 | // make much sense then anyway. |
| 1006 | if (install_plan_) { |
Don Garrett | af9085e | 2013-11-06 18:14:29 -0800 | [diff] [blame] | 1007 | // Generate an unique payload identifier. |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 1008 | string target_version_uid; |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1009 | for (const auto& payload : install_plan_->payloads) { |
Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 1010 | target_version_uid += |
| 1011 | brillo::data_encoding::Base64Encode(payload.hash) + ":" + |
| 1012 | payload.metadata_signature + ":"; |
| 1013 | } |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1014 | |
Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 1015 | // If we just downloaded a rollback image, we should preserve this fact |
| 1016 | // over the following powerwash. |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1017 | if (install_plan_->is_rollback) { |
Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 1018 | system_state_->payload_state()->SetRollbackHappened(true); |
Marton Hunyady | a030268 | 2018-05-16 18:52:13 +0200 | [diff] [blame] | 1019 | system_state_->metrics_reporter()->ReportEnterpriseRollbackMetrics( |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1020 | /*success=*/true, install_plan_->version); |
Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 1021 | } |
| 1022 | |
Don Garrett | af9085e | 2013-11-06 18:14:29 -0800 | [diff] [blame] | 1023 | // Expect to reboot into the new version to send the proper metric during |
| 1024 | // next boot. |
| 1025 | system_state_->payload_state()->ExpectRebootInNewVersion( |
| 1026 | target_version_uid); |
Don Garrett | af9085e | 2013-11-06 18:14:29 -0800 | [diff] [blame] | 1027 | } else { |
| 1028 | // If we just finished a rollback, then we expect to have no Omaha |
| 1029 | // response. Otherwise, it's an error. |
| 1030 | if (system_state_->payload_state()->GetRollbackVersion().empty()) { |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1031 | LOG(ERROR) << "Can't send metrics because there was no Omaha response"; |
Don Garrett | af9085e | 2013-11-06 18:14:29 -0800 | [diff] [blame] | 1032 | } |
| 1033 | } |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1034 | return; |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 1035 | } |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1036 | |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1037 | if (ScheduleErrorEventAction()) { |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1038 | return; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1039 | } |
| 1040 | LOG(INFO) << "No update."; |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1041 | SetStatusAndNotify(UpdateStatus::IDLE); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1042 | ScheduleUpdates(); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1043 | } |
| 1044 | |
| 1045 | void UpdateAttempter::ProcessingStopped(const ActionProcessor* processor) { |
Chris Sosa | 4f8ee27 | 2012-11-30 13:01:54 -0800 | [diff] [blame] | 1046 | // Reset cpu shares back to normal. |
Alex Deymo | ab0d976 | 2016-02-02 10:52:56 -0800 | [diff] [blame] | 1047 | cpu_limiter_.StopLimiter(); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1048 | download_progress_ = 0.0; |
Adolfo Victoria | 20262ad | 2018-08-06 10:40:11 -0700 | [diff] [blame] | 1049 | if (forced_update_pending_callback_.get()) |
| 1050 | // Clear prior interactive requests once the processor is done. |
| 1051 | forced_update_pending_callback_->Run(false, false); |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1052 | SetStatusAndNotify(UpdateStatus::IDLE); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1053 | ScheduleUpdates(); |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1054 | error_event_.reset(nullptr); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1055 | } |
| 1056 | |
| 1057 | // Called whenever an action has finished processing, either successfully |
| 1058 | // or otherwise. |
| 1059 | void UpdateAttempter::ActionCompleted(ActionProcessor* processor, |
| 1060 | AbstractAction* action, |
David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 1061 | ErrorCode code) { |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1062 | // Reset download progress regardless of whether or not the download |
| 1063 | // action succeeded. Also, get the response code from HTTP request |
| 1064 | // actions (update download as well as the initial update check |
| 1065 | // actions). |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1066 | const string type = action->Type(); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1067 | if (type == DownloadAction::StaticType()) { |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1068 | download_progress_ = 0.0; |
Gilad Arnold | cf175a0 | 2014-07-10 16:48:47 -0700 | [diff] [blame] | 1069 | DownloadAction* download_action = static_cast<DownloadAction*>(action); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1070 | http_response_code_ = download_action->GetHTTPResponseCode(); |
| 1071 | } else if (type == OmahaRequestAction::StaticType()) { |
| 1072 | OmahaRequestAction* omaha_request_action = |
Gilad Arnold | cf175a0 | 2014-07-10 16:48:47 -0700 | [diff] [blame] | 1073 | static_cast<OmahaRequestAction*>(action); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1074 | // If the request is not an event, then it's the update-check. |
| 1075 | if (!omaha_request_action->IsEvent()) { |
| 1076 | http_response_code_ = omaha_request_action->GetHTTPResponseCode(); |
Gilad Arnold | a6dab94 | 2014-04-25 11:46:03 -0700 | [diff] [blame] | 1077 | |
| 1078 | // Record the number of consecutive failed update checks. |
| 1079 | if (http_response_code_ == kHttpResponseInternalServerError || |
| 1080 | http_response_code_ == kHttpResponseServiceUnavailable) { |
| 1081 | consecutive_failed_update_checks_++; |
| 1082 | } else { |
| 1083 | consecutive_failed_update_checks_ = 0; |
| 1084 | } |
| 1085 | |
Weidong Guo | 421ff33 | 2017-04-17 10:08:38 -0700 | [diff] [blame] | 1086 | const OmahaResponse& omaha_response = |
| 1087 | omaha_request_action->GetOutputObject(); |
Gilad Arnold | a0258a5 | 2014-07-10 16:21:19 -0700 | [diff] [blame] | 1088 | // Store the server-dictated poll interval, if any. |
| 1089 | server_dictated_poll_interval_ = |
Weidong Guo | 421ff33 | 2017-04-17 10:08:38 -0700 | [diff] [blame] | 1090 | std::max(0, omaha_response.poll_interval); |
| 1091 | |
| 1092 | // This update is ignored by omaha request action because update over |
| 1093 | // cellular connection is not allowed. Needs to ask for user's permissions |
| 1094 | // to update. |
| 1095 | if (code == ErrorCode::kOmahaUpdateIgnoredOverCellular) { |
| 1096 | new_version_ = omaha_response.version; |
| 1097 | new_payload_size_ = 0; |
| 1098 | for (const auto& package : omaha_response.packages) { |
| 1099 | new_payload_size_ += package.size; |
| 1100 | } |
| 1101 | SetStatusAndNotify(UpdateStatus::NEED_PERMISSION_TO_UPDATE); |
| 1102 | } |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1103 | } |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1104 | } else if (type == OmahaResponseHandlerAction::StaticType()) { |
| 1105 | // Depending on the returned error code, note that an update is available. |
| 1106 | if (code == ErrorCode::kOmahaUpdateDeferredPerPolicy || |
| 1107 | code == ErrorCode::kSuccess) { |
| 1108 | // Note that the status will be updated to DOWNLOADING when some bytes |
| 1109 | // get actually downloaded from the server and the BytesReceived |
| 1110 | // callback is invoked. This avoids notifying the user that a download |
| 1111 | // has started in cases when the server and the client are unable to |
| 1112 | // initiate the download. |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1113 | auto omaha_response_handler_action = |
| 1114 | static_cast<OmahaResponseHandlerAction*>(action); |
| 1115 | install_plan_.reset( |
| 1116 | new InstallPlan(omaha_response_handler_action->install_plan())); |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1117 | UpdateLastCheckedTime(); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1118 | new_version_ = install_plan_->version; |
| 1119 | new_system_version_ = install_plan_->system_version; |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1120 | new_payload_size_ = 0; |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1121 | for (const auto& payload : install_plan_->payloads) |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1122 | new_payload_size_ += payload.size; |
| 1123 | cpu_limiter_.StartLimiter(); |
| 1124 | SetStatusAndNotify(UpdateStatus::UPDATE_AVAILABLE); |
| 1125 | } |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1126 | } |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1127 | // General failure cases. |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1128 | if (code != ErrorCode::kSuccess) { |
Darin Petkov | 7ed561b | 2011-10-04 02:59:03 -0700 | [diff] [blame] | 1129 | // If the current state is at or past the download phase, count the failure |
| 1130 | // in case a switch to full update becomes necessary. Ignore network |
| 1131 | // transfer timeouts and failures. |
Amin Hassani | 89a3712 | 2018-03-23 12:59:24 -0700 | [diff] [blame] | 1132 | if (code != ErrorCode::kDownloadTransferError) { |
| 1133 | switch (status_) { |
| 1134 | case UpdateStatus::IDLE: |
| 1135 | case UpdateStatus::CHECKING_FOR_UPDATE: |
| 1136 | case UpdateStatus::UPDATE_AVAILABLE: |
| 1137 | case UpdateStatus::NEED_PERMISSION_TO_UPDATE: |
| 1138 | break; |
| 1139 | case UpdateStatus::DOWNLOADING: |
| 1140 | case UpdateStatus::VERIFYING: |
| 1141 | case UpdateStatus::FINALIZING: |
| 1142 | case UpdateStatus::UPDATED_NEED_REBOOT: |
| 1143 | case UpdateStatus::REPORTING_ERROR_EVENT: |
| 1144 | case UpdateStatus::ATTEMPTING_ROLLBACK: |
| 1145 | case UpdateStatus::DISABLED: |
| 1146 | MarkDeltaUpdateFailure(); |
| 1147 | break; |
| 1148 | } |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 1149 | } |
Sen Jiang | 89e24c1 | 2018-03-22 18:05:44 -0700 | [diff] [blame] | 1150 | if (code != ErrorCode::kNoUpdate) { |
| 1151 | // On failure, schedule an error event to be sent to Omaha. |
| 1152 | CreatePendingErrorEvent(action, code); |
| 1153 | } |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1154 | return; |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1155 | } |
Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 1156 | // Find out which action completed (successfully). |
| 1157 | if (type == DownloadAction::StaticType()) { |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1158 | SetStatusAndNotify(UpdateStatus::FINALIZING); |
Amin Hassani | afd8cea | 2017-12-04 14:20:00 -0800 | [diff] [blame] | 1159 | } else if (type == FilesystemVerifierAction::StaticType()) { |
| 1160 | // Log the system properties before the postinst and after the file system |
| 1161 | // is verified. It used to be done in the postinst itself. But postinst |
| 1162 | // cannot do this anymore. On the other hand, these logs are frequently |
| 1163 | // looked at and it is preferable not to scatter them in random location in |
| 1164 | // the log and rather log it right before the postinst. The reason not do |
| 1165 | // this in the |PostinstallRunnerAction| is to prevent dependency from |
| 1166 | // libpayload_consumer to libupdate_engine. |
| 1167 | LogImageProperties(); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1168 | } |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 1169 | } |
| 1170 | |
Alex Deymo | 542c19b | 2015-12-03 07:43:31 -0300 | [diff] [blame] | 1171 | void UpdateAttempter::BytesReceived(uint64_t bytes_progressed, |
| 1172 | uint64_t bytes_received, |
| 1173 | uint64_t total) { |
| 1174 | // The PayloadState keeps track of how many bytes were actually downloaded |
| 1175 | // from a given URL for the URL skipping logic. |
| 1176 | system_state_->payload_state()->DownloadProgress(bytes_progressed); |
| 1177 | |
Alex Deymo | 0d29854 | 2016-03-30 18:31:49 -0700 | [diff] [blame] | 1178 | double progress = 0; |
| 1179 | if (total) |
| 1180 | progress = static_cast<double>(bytes_received) / static_cast<double>(total); |
| 1181 | if (status_ != UpdateStatus::DOWNLOADING || bytes_received == total) { |
Darin Petkov | af18305 | 2010-08-23 12:07:13 -0700 | [diff] [blame] | 1182 | download_progress_ = progress; |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1183 | SetStatusAndNotify(UpdateStatus::DOWNLOADING); |
Alex Deymo | 0d29854 | 2016-03-30 18:31:49 -0700 | [diff] [blame] | 1184 | } else { |
| 1185 | ProgressUpdate(progress); |
Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 1186 | } |
| 1187 | } |
| 1188 | |
Alex Deymo | 542c19b | 2015-12-03 07:43:31 -0300 | [diff] [blame] | 1189 | void UpdateAttempter::DownloadComplete() { |
| 1190 | system_state_->payload_state()->DownloadComplete(); |
| 1191 | } |
| 1192 | |
Alex Deymo | 0d29854 | 2016-03-30 18:31:49 -0700 | [diff] [blame] | 1193 | void UpdateAttempter::ProgressUpdate(double progress) { |
| 1194 | // Self throttle based on progress. Also send notifications if progress is |
| 1195 | // too slow. |
| 1196 | if (progress == 1.0 || |
| 1197 | progress - download_progress_ >= kBroadcastThresholdProgress || |
| 1198 | TimeTicks::Now() - last_notify_time_ >= |
| 1199 | TimeDelta::FromSeconds(kBroadcastThresholdSeconds)) { |
| 1200 | download_progress_ = progress; |
| 1201 | BroadcastStatus(); |
| 1202 | } |
| 1203 | } |
| 1204 | |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1205 | bool UpdateAttempter::ResetStatus() { |
| 1206 | LOG(INFO) << "Attempting to reset state from " |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1207 | << UpdateStatusToString(status_) << " to UpdateStatus::IDLE"; |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1208 | |
| 1209 | switch (status_) { |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1210 | case UpdateStatus::IDLE: |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1211 | // no-op. |
| 1212 | return true; |
| 1213 | |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1214 | case UpdateStatus::UPDATED_NEED_REBOOT: { |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1215 | bool ret_value = true; |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1216 | status_ = UpdateStatus::IDLE; |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1217 | |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1218 | // Remove the reboot marker so that if the machine is rebooted |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1219 | // after resetting to idle state, it doesn't go back to |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1220 | // UpdateStatus::UPDATED_NEED_REBOOT state. |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 1221 | ret_value = prefs_->Delete(kPrefsUpdateCompletedOnBootId) && ret_value; |
| 1222 | ret_value = prefs_->Delete(kPrefsUpdateCompletedBootTime) && ret_value; |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1223 | |
Alex Deymo | 9870c0e | 2015-09-23 13:58:31 -0700 | [diff] [blame] | 1224 | // Update the boot flags so the current slot has higher priority. |
| 1225 | BootControlInterface* boot_control = system_state_->boot_control(); |
| 1226 | if (!boot_control->SetActiveBootSlot(boot_control->GetCurrentSlot())) |
| 1227 | ret_value = false; |
| 1228 | |
Alex Deymo | 5259033 | 2016-11-29 18:29:13 -0800 | [diff] [blame] | 1229 | // Mark the current slot as successful again, since marking it as active |
| 1230 | // may reset the successful bit. We ignore the result of whether marking |
| 1231 | // the current slot as successful worked. |
Alex Deymo | dbc2a80 | 2016-12-02 15:10:32 -0800 | [diff] [blame] | 1232 | if (!boot_control->MarkBootSuccessfulAsync(Bind([](bool successful){}))) |
Alex Deymo | 5259033 | 2016-11-29 18:29:13 -0800 | [diff] [blame] | 1233 | ret_value = false; |
| 1234 | |
Alex Deymo | 4243291 | 2013-07-12 20:21:15 -0700 | [diff] [blame] | 1235 | // Notify the PayloadState that the successful payload was canceled. |
| 1236 | system_state_->payload_state()->ResetUpdateStatus(); |
| 1237 | |
Alex Deymo | 87c0886 | 2015-10-30 21:56:55 -0700 | [diff] [blame] | 1238 | // The previous version is used to report back to omaha after reboot that |
| 1239 | // we actually rebooted into the new version from this "prev-version". We |
| 1240 | // need to clear out this value now to prevent it being sent on the next |
| 1241 | // updatecheck request. |
| 1242 | ret_value = prefs_->SetString(kPrefsPreviousVersion, "") && ret_value; |
| 1243 | |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 1244 | LOG(INFO) << "Reset status " << (ret_value ? "successful" : "failed"); |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1245 | return ret_value; |
Jay Srinivasan | c1ba09a | 2012-08-14 14:15:57 -0700 | [diff] [blame] | 1246 | } |
| 1247 | |
| 1248 | default: |
| 1249 | LOG(ERROR) << "Reset not allowed in this state."; |
| 1250 | return false; |
| 1251 | } |
| 1252 | } |
| 1253 | |
Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 1254 | bool UpdateAttempter::GetStatus(UpdateEngineStatus* out_status) { |
Aaron Wood | 795c5b4 | 2017-12-05 16:06:13 -0800 | [diff] [blame] | 1255 | out_status->last_checked_time = last_checked_time_; |
Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 1256 | out_status->status = status_; |
| 1257 | out_status->current_version = omaha_request_params_->app_version(); |
| 1258 | out_status->current_system_version = omaha_request_params_->system_version(); |
| 1259 | out_status->progress = download_progress_; |
| 1260 | out_status->new_size_bytes = new_payload_size_; |
| 1261 | out_status->new_version = new_version_; |
| 1262 | out_status->new_system_version = new_system_version_; |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 1263 | return true; |
| 1264 | } |
| 1265 | |
Darin Petkov | 61635a9 | 2011-05-18 16:20:36 -0700 | [diff] [blame] | 1266 | void UpdateAttempter::BroadcastStatus() { |
Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 1267 | UpdateEngineStatus broadcast_status; |
| 1268 | // Use common method for generating the current status. |
| 1269 | GetStatus(&broadcast_status); |
| 1270 | |
Alex Deymo | fa78f14 | 2016-01-26 21:36:16 -0800 | [diff] [blame] | 1271 | for (const auto& observer : service_observers_) { |
Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 1272 | observer->SendStatusUpdate(broadcast_status); |
Alex Deymo | fa78f14 | 2016-01-26 21:36:16 -0800 | [diff] [blame] | 1273 | } |
Darin Petkov | af18305 | 2010-08-23 12:07:13 -0700 | [diff] [blame] | 1274 | last_notify_time_ = TimeTicks::Now(); |
Alex Deymo | fa78f14 | 2016-01-26 21:36:16 -0800 | [diff] [blame] | 1275 | } |
| 1276 | |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1277 | uint32_t UpdateAttempter::GetErrorCodeFlags() { |
| 1278 | uint32_t flags = 0; |
| 1279 | |
J. Richard Barnette | 056b0ab | 2013-10-29 15:24:56 -0700 | [diff] [blame] | 1280 | if (!system_state_->hardware()->IsNormalBootMode()) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1281 | flags |= static_cast<uint32_t>(ErrorCode::kDevModeFlag); |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1282 | |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1283 | if (install_plan_ && install_plan_->is_resume) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1284 | flags |= static_cast<uint32_t>(ErrorCode::kResumedFlag); |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1285 | |
J. Richard Barnette | 056b0ab | 2013-10-29 15:24:56 -0700 | [diff] [blame] | 1286 | if (!system_state_->hardware()->IsOfficialBuild()) |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1287 | flags |= static_cast<uint32_t>(ErrorCode::kTestImageFlag); |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1288 | |
Sen Jiang | daeaa43 | 2018-10-09 18:18:45 -0700 | [diff] [blame] | 1289 | if (!omaha_request_params_->IsUpdateUrlOfficial()) { |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1290 | flags |= static_cast<uint32_t>(ErrorCode::kTestOmahaUrlFlag); |
Alex Deymo | ac41a82 | 2015-09-15 20:52:53 -0700 | [diff] [blame] | 1291 | } |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1292 | |
| 1293 | return flags; |
| 1294 | } |
| 1295 | |
David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 1296 | bool UpdateAttempter::ShouldCancel(ErrorCode* cancel_reason) { |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1297 | // Check if the channel we're attempting to update to is the same as the |
| 1298 | // target channel currently chosen by the user. |
| 1299 | OmahaRequestParams* params = system_state_->request_params(); |
| 1300 | if (params->download_channel() != params->target_channel()) { |
| 1301 | LOG(ERROR) << "Aborting download as target channel: " |
| 1302 | << params->target_channel() |
| 1303 | << " is different from the download channel: " |
| 1304 | << params->download_channel(); |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1305 | *cancel_reason = ErrorCode::kUpdateCanceledByChannelChange; |
Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 1306 | return true; |
| 1307 | } |
| 1308 | |
| 1309 | return false; |
| 1310 | } |
| 1311 | |
Nam T. Nguyen | 7d623eb | 2014-05-13 16:06:28 -0700 | [diff] [blame] | 1312 | void UpdateAttempter::SetStatusAndNotify(UpdateStatus status) { |
Darin Petkov | 61635a9 | 2011-05-18 16:20:36 -0700 | [diff] [blame] | 1313 | status_ = status; |
Darin Petkov | 61635a9 | 2011-05-18 16:20:36 -0700 | [diff] [blame] | 1314 | BroadcastStatus(); |
| 1315 | } |
| 1316 | |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 1317 | void UpdateAttempter::CreatePendingErrorEvent(AbstractAction* action, |
David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 1318 | ErrorCode code) { |
Sen Jiang | 89e24c1 | 2018-03-22 18:05:44 -0700 | [diff] [blame] | 1319 | if (error_event_.get() || status_ == UpdateStatus::REPORTING_ERROR_EVENT) { |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1320 | // This shouldn't really happen. |
| 1321 | LOG(WARNING) << "There's already an existing pending error event."; |
| 1322 | return; |
| 1323 | } |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 1324 | |
Jay Srinivasan | 56d5aa4 | 2012-03-26 14:27:59 -0700 | [diff] [blame] | 1325 | // Classify the code to generate the appropriate result so that |
| 1326 | // the Borgmon charts show up the results correctly. |
| 1327 | // Do this before calling GetErrorCodeForAction which could potentially |
| 1328 | // augment the bit representation of code and thus cause no matches for |
| 1329 | // the switch cases below. |
| 1330 | OmahaEvent::Result event_result; |
| 1331 | switch (code) { |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1332 | case ErrorCode::kOmahaUpdateIgnoredPerPolicy: |
| 1333 | case ErrorCode::kOmahaUpdateDeferredPerPolicy: |
| 1334 | case ErrorCode::kOmahaUpdateDeferredForBackoff: |
Jay Srinivasan | 56d5aa4 | 2012-03-26 14:27:59 -0700 | [diff] [blame] | 1335 | event_result = OmahaEvent::kResultUpdateDeferred; |
| 1336 | break; |
| 1337 | default: |
| 1338 | event_result = OmahaEvent::kResultError; |
| 1339 | break; |
| 1340 | } |
| 1341 | |
Darin Petkov | 777dbfa | 2010-07-20 15:03:37 -0700 | [diff] [blame] | 1342 | code = GetErrorCodeForAction(action, code); |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1343 | fake_update_success_ = code == ErrorCode::kPostinstallBootedFromFirmwareB; |
Darin Petkov | 18c7bce | 2011-06-16 14:07:00 -0700 | [diff] [blame] | 1344 | |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1345 | // Compute the final error code with all the bit flags to be sent to Omaha. |
Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 1346 | code = static_cast<ErrorCode>( |
| 1347 | static_cast<uint32_t>(code) | GetErrorCodeFlags()); |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1348 | error_event_.reset(new OmahaEvent(OmahaEvent::kTypeUpdateComplete, |
Jay Srinivasan | 56d5aa4 | 2012-03-26 14:27:59 -0700 | [diff] [blame] | 1349 | event_result, |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1350 | code)); |
| 1351 | } |
| 1352 | |
| 1353 | bool UpdateAttempter::ScheduleErrorEventAction() { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1354 | if (error_event_.get() == nullptr) |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1355 | return false; |
| 1356 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 1357 | LOG(ERROR) << "Update failed."; |
| 1358 | system_state_->payload_state()->UpdateFailed(error_event_->error_code); |
| 1359 | |
Marton Hunyady | a030268 | 2018-05-16 18:52:13 +0200 | [diff] [blame] | 1360 | // Send metrics if it was a rollback. |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1361 | if (install_plan_ && install_plan_->is_rollback) { |
| 1362 | system_state_->metrics_reporter()->ReportEnterpriseRollbackMetrics( |
| 1363 | /*success=*/false, install_plan_->version); |
Marton Hunyady | a030268 | 2018-05-16 18:52:13 +0200 | [diff] [blame] | 1364 | } |
| 1365 | |
Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 1366 | // Send it to Omaha. |
Alex Deymo | af9a863 | 2015-09-23 18:51:48 -0700 | [diff] [blame] | 1367 | LOG(INFO) << "Reporting the error event"; |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1368 | auto error_event_action = std::make_unique<OmahaRequestAction>( |
| 1369 | system_state_, |
| 1370 | error_event_.release(), // Pass ownership. |
| 1371 | std::make_unique<LibcurlHttpFetcher>(GetProxyResolver(), |
| 1372 | system_state_->hardware()), |
| 1373 | false); |
| 1374 | processor_->EnqueueAction(std::move(error_event_action)); |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1375 | SetStatusAndNotify(UpdateStatus::REPORTING_ERROR_EVENT); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 1376 | processor_->StartProcessing(); |
Darin Petkov | 09f96c3 | 2010-07-20 09:24:57 -0700 | [diff] [blame] | 1377 | return true; |
| 1378 | } |
| 1379 | |
Darin Petkov | 58dd134 | 2011-05-06 12:05:13 -0700 | [diff] [blame] | 1380 | void UpdateAttempter::ScheduleProcessingStart() { |
| 1381 | LOG(INFO) << "Scheduling an action processor start."; |
Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 1382 | MessageLoop::current()->PostTask( |
| 1383 | FROM_HERE, |
Luis Hector Chavez | f1cf348 | 2016-07-19 14:29:19 -0700 | [diff] [blame] | 1384 | Bind([](ActionProcessor* processor) { processor->StartProcessing(); }, |
| 1385 | base::Unretained(processor_.get()))); |
Darin Petkov | 58dd134 | 2011-05-06 12:05:13 -0700 | [diff] [blame] | 1386 | } |
| 1387 | |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 1388 | void UpdateAttempter::DisableDeltaUpdateIfNeeded() { |
| 1389 | int64_t delta_failures; |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 1390 | if (omaha_request_params_->delta_okay() && |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 1391 | prefs_->GetInt64(kPrefsDeltaUpdateFailures, &delta_failures) && |
| 1392 | delta_failures >= kMaxDeltaUpdateFailures) { |
| 1393 | LOG(WARNING) << "Too many delta update failures, forcing full update."; |
Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 1394 | omaha_request_params_->set_delta_okay(false); |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 1395 | } |
| 1396 | } |
| 1397 | |
| 1398 | void UpdateAttempter::MarkDeltaUpdateFailure() { |
Darin Petkov | 2dd0109 | 2010-10-08 15:43:05 -0700 | [diff] [blame] | 1399 | // Don't try to resume a failed delta update. |
| 1400 | DeltaPerformer::ResetUpdateProgress(prefs_, false); |
Darin Petkov | 3627577 | 2010-10-01 11:40:57 -0700 | [diff] [blame] | 1401 | int64_t delta_failures; |
| 1402 | if (!prefs_->GetInt64(kPrefsDeltaUpdateFailures, &delta_failures) || |
| 1403 | delta_failures < 0) { |
| 1404 | delta_failures = 0; |
| 1405 | } |
| 1406 | prefs_->SetInt64(kPrefsDeltaUpdateFailures, ++delta_failures); |
| 1407 | } |
| 1408 | |
Thieu Le | 116fda3 | 2011-04-19 11:01:54 -0700 | [diff] [blame] | 1409 | void UpdateAttempter::PingOmaha() { |
Thieu Le | d88a857 | 2011-05-26 09:09:19 -0700 | [diff] [blame] | 1410 | if (!processor_->IsRunning()) { |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1411 | auto ping_action = std::make_unique<OmahaRequestAction>( |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 1412 | system_state_, |
| 1413 | nullptr, |
Ben Chan | ab5a0af | 2017-10-12 14:57:50 -0700 | [diff] [blame] | 1414 | std::make_unique<LibcurlHttpFetcher>(GetProxyResolver(), |
Ben Chan | 5c02c13 | 2017-06-27 07:10:36 -0700 | [diff] [blame] | 1415 | system_state_->hardware()), |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1416 | true); |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1417 | processor_->set_delegate(nullptr); |
Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 1418 | processor_->EnqueueAction(std::move(ping_action)); |
Thieu Le | d88a857 | 2011-05-26 09:09:19 -0700 | [diff] [blame] | 1419 | // Call StartProcessing() synchronously here to avoid any race conditions |
| 1420 | // caused by multiple outstanding ping Omaha requests. If we call |
| 1421 | // StartProcessing() asynchronously, the device can be suspended before we |
| 1422 | // get a chance to callback to StartProcessing(). When the device resumes |
| 1423 | // (assuming the device sleeps longer than the next update check period), |
| 1424 | // StartProcessing() is called back and at the same time, the next update |
| 1425 | // check is fired which eventually invokes StartProcessing(). A crash |
| 1426 | // can occur because StartProcessing() checks to make sure that the |
| 1427 | // processor is idle which it isn't due to the two concurrent ping Omaha |
| 1428 | // requests. |
| 1429 | processor_->StartProcessing(); |
| 1430 | } else { |
Darin Petkov | 58dd134 | 2011-05-06 12:05:13 -0700 | [diff] [blame] | 1431 | LOG(WARNING) << "Action processor running, Omaha ping suppressed."; |
Darin Petkov | 58dd134 | 2011-05-06 12:05:13 -0700 | [diff] [blame] | 1432 | } |
Thieu Le | d88a857 | 2011-05-26 09:09:19 -0700 | [diff] [blame] | 1433 | |
Gilad Arnold | ec7f916 | 2014-07-15 13:24:46 -0700 | [diff] [blame] | 1434 | // Update the last check time here; it may be re-updated when an Omaha |
| 1435 | // response is received, but this will prevent us from repeatedly scheduling |
| 1436 | // checks in the case where a response is not received. |
| 1437 | UpdateLastCheckedTime(); |
| 1438 | |
Thieu Le | d88a857 | 2011-05-26 09:09:19 -0700 | [diff] [blame] | 1439 | // Update the status which will schedule the next update check |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1440 | SetStatusAndNotify(UpdateStatus::UPDATED_NEED_REBOOT); |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1441 | ScheduleUpdates(); |
Thieu Le | 116fda3 | 2011-04-19 11:01:54 -0700 | [diff] [blame] | 1442 | } |
| 1443 | |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 1444 | |
| 1445 | bool UpdateAttempter::DecrementUpdateCheckCount() { |
Ben Chan | 9abb763 | 2014-08-07 00:10:53 -0700 | [diff] [blame] | 1446 | int64_t update_check_count_value; |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 1447 | |
| 1448 | if (!prefs_->Exists(kPrefsUpdateCheckCount)) { |
| 1449 | // This file does not exist. This means we haven't started our update |
| 1450 | // check count down yet, so nothing more to do. This file will be created |
| 1451 | // later when we first satisfy the wall-clock-based-wait period. |
| 1452 | LOG(INFO) << "No existing update check count. That's normal."; |
| 1453 | return true; |
| 1454 | } |
| 1455 | |
| 1456 | if (prefs_->GetInt64(kPrefsUpdateCheckCount, &update_check_count_value)) { |
| 1457 | // Only if we're able to read a proper integer value, then go ahead |
| 1458 | // and decrement and write back the result in the same file, if needed. |
| 1459 | LOG(INFO) << "Update check count = " << update_check_count_value; |
| 1460 | |
| 1461 | if (update_check_count_value == 0) { |
| 1462 | // It could be 0, if, for some reason, the file didn't get deleted |
| 1463 | // when we set our status to waiting for reboot. so we just leave it |
| 1464 | // as is so that we can prevent another update_check wait for this client. |
| 1465 | LOG(INFO) << "Not decrementing update check count as it's already 0."; |
| 1466 | return true; |
| 1467 | } |
| 1468 | |
| 1469 | if (update_check_count_value > 0) |
| 1470 | update_check_count_value--; |
| 1471 | else |
| 1472 | update_check_count_value = 0; |
| 1473 | |
| 1474 | // Write out the new value of update_check_count_value. |
| 1475 | if (prefs_->SetInt64(kPrefsUpdateCheckCount, update_check_count_value)) { |
Sen Jiang | 771f648 | 2018-04-04 17:59:10 -0700 | [diff] [blame] | 1476 | // We successfully wrote out the new value, so enable the |
Jay Srinivasan | 480ddfa | 2012-06-01 19:15:26 -0700 | [diff] [blame] | 1477 | // update check based wait. |
| 1478 | LOG(INFO) << "New update check count = " << update_check_count_value; |
| 1479 | return true; |
| 1480 | } |
| 1481 | } |
| 1482 | |
| 1483 | LOG(INFO) << "Deleting update check count state due to read/write errors."; |
| 1484 | |
| 1485 | // We cannot read/write to the file, so disable the update check based wait |
| 1486 | // so that we don't get stuck in this OS version by any chance (which could |
| 1487 | // happen if there's some bug that causes to read/write incorrectly). |
| 1488 | // Also attempt to delete the file to do our best effort to cleanup. |
| 1489 | prefs_->Delete(kPrefsUpdateCheckCount); |
| 1490 | return false; |
| 1491 | } |
Chris Sosa | d317e40 | 2013-06-12 13:47:09 -0700 | [diff] [blame] | 1492 | |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 1493 | |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 1494 | void UpdateAttempter::UpdateEngineStarted() { |
Alex Vakulenko | dea2eac | 2014-03-14 15:56:59 -0700 | [diff] [blame] | 1495 | // If we just booted into a new update, keep the previous OS version |
| 1496 | // in case we rebooted because of a crash of the old version, so we |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1497 | // can do a proper crash report with correct information. |
Alex Vakulenko | dea2eac | 2014-03-14 15:56:59 -0700 | [diff] [blame] | 1498 | // This must be done before calling |
| 1499 | // system_state_->payload_state()->UpdateEngineStarted() since it will |
| 1500 | // delete SystemUpdated marker file. |
| 1501 | if (system_state_->system_rebooted() && |
| 1502 | prefs_->Exists(kPrefsSystemUpdatedMarker)) { |
| 1503 | if (!prefs_->GetString(kPrefsPreviousVersion, &prev_version_)) { |
| 1504 | // If we fail to get the version string, make sure it stays empty. |
| 1505 | prev_version_.clear(); |
| 1506 | } |
| 1507 | } |
| 1508 | |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 1509 | system_state_->payload_state()->UpdateEngineStarted(); |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 1510 | StartP2PAtStartup(); |
| 1511 | } |
| 1512 | |
| 1513 | bool UpdateAttempter::StartP2PAtStartup() { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1514 | if (system_state_ == nullptr || |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 1515 | !system_state_->p2p_manager()->IsP2PEnabled()) { |
| 1516 | LOG(INFO) << "Not starting p2p at startup since it's not enabled."; |
| 1517 | return false; |
| 1518 | } |
| 1519 | |
| 1520 | if (system_state_->p2p_manager()->CountSharedFiles() < 1) { |
| 1521 | LOG(INFO) << "Not starting p2p at startup since our application " |
| 1522 | << "is not sharing any files."; |
| 1523 | return false; |
| 1524 | } |
| 1525 | |
| 1526 | return StartP2PAndPerformHousekeeping(); |
| 1527 | } |
| 1528 | |
| 1529 | bool UpdateAttempter::StartP2PAndPerformHousekeeping() { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 1530 | if (system_state_ == nullptr) |
David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 1531 | return false; |
| 1532 | |
| 1533 | if (!system_state_->p2p_manager()->IsP2PEnabled()) { |
| 1534 | LOG(INFO) << "Not starting p2p since it's not enabled."; |
| 1535 | return false; |
| 1536 | } |
| 1537 | |
| 1538 | LOG(INFO) << "Ensuring that p2p is running."; |
| 1539 | if (!system_state_->p2p_manager()->EnsureP2PRunning()) { |
| 1540 | LOG(ERROR) << "Error starting p2p."; |
| 1541 | return false; |
| 1542 | } |
| 1543 | |
| 1544 | LOG(INFO) << "Performing p2p housekeeping."; |
| 1545 | if (!system_state_->p2p_manager()->PerformHousekeeping()) { |
| 1546 | LOG(ERROR) << "Error performing housekeeping for p2p."; |
| 1547 | return false; |
| 1548 | } |
| 1549 | |
| 1550 | LOG(INFO) << "Done performing p2p housekeeping."; |
| 1551 | return true; |
David Zeuthen | e4c58bf | 2013-06-18 17:26:50 -0700 | [diff] [blame] | 1552 | } |
| 1553 | |
Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 1554 | bool UpdateAttempter::GetBootTimeAtUpdate(Time *out_boot_time) { |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 1555 | // In case of an update_engine restart without a reboot, we stored the boot_id |
| 1556 | // when the update was completed by setting a pref, so we can check whether |
| 1557 | // the last update was on this boot or a previous one. |
| 1558 | string boot_id; |
| 1559 | TEST_AND_RETURN_FALSE(utils::GetBootId(&boot_id)); |
| 1560 | |
| 1561 | string update_completed_on_boot_id; |
| 1562 | if (!prefs_->Exists(kPrefsUpdateCompletedOnBootId) || |
| 1563 | !prefs_->GetString(kPrefsUpdateCompletedOnBootId, |
| 1564 | &update_completed_on_boot_id) || |
| 1565 | update_completed_on_boot_id != boot_id) |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 1566 | return false; |
| 1567 | |
Alex Deymo | 906191f | 2015-10-12 12:22:44 -0700 | [diff] [blame] | 1568 | // Short-circuit avoiding the read in case out_boot_time is nullptr. |
| 1569 | if (out_boot_time) { |
| 1570 | int64_t boot_time = 0; |
| 1571 | // Since the kPrefsUpdateCompletedOnBootId was correctly set, this pref |
| 1572 | // should not fail. |
| 1573 | TEST_AND_RETURN_FALSE( |
| 1574 | prefs_->GetInt64(kPrefsUpdateCompletedBootTime, &boot_time)); |
| 1575 | *out_boot_time = Time::FromInternalValue(boot_time); |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 1576 | } |
David Zeuthen | 3c55abd | 2013-10-14 12:48:03 -0700 | [diff] [blame] | 1577 | return true; |
| 1578 | } |
| 1579 | |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1580 | bool UpdateAttempter::IsUpdateRunningOrScheduled() { |
Christopher Wiley | cc8ce0e | 2015-10-01 16:48:47 -0700 | [diff] [blame] | 1581 | return ((status_ != UpdateStatus::IDLE && |
| 1582 | status_ != UpdateStatus::UPDATED_NEED_REBOOT) || |
Gilad Arnold | 54fa66d | 2014-09-29 13:14:29 -0700 | [diff] [blame] | 1583 | waiting_for_scheduled_check_); |
| 1584 | } |
| 1585 | |
Sen Jiang | daeaa43 | 2018-10-09 18:18:45 -0700 | [diff] [blame] | 1586 | bool UpdateAttempter::IsAnyUpdateSourceAllowed() const { |
David Pursell | 907b4fa | 2015-01-27 10:27:38 -0800 | [diff] [blame] | 1587 | // We allow updates from any source if either of these are true: |
| 1588 | // * The device is running an unofficial (dev/test) image. |
| 1589 | // * The debugd dev features are accessible (i.e. in devmode with no owner). |
| 1590 | // This protects users running a base image, while still allowing a specific |
| 1591 | // window (gated by the debug dev features) where `cros flash` is usable. |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 1592 | if (!system_state_->hardware()->IsOfficialBuild()) { |
| 1593 | LOG(INFO) << "Non-official build; allowing any update source."; |
| 1594 | return true; |
| 1595 | } |
| 1596 | |
Sen Jiang | e67bb5b | 2016-06-20 15:53:56 -0700 | [diff] [blame] | 1597 | if (system_state_->hardware()->AreDevFeaturesEnabled()) { |
| 1598 | LOG(INFO) << "Developer features enabled; allowing custom update sources."; |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 1599 | return true; |
| 1600 | } |
Sen Jiang | e67bb5b | 2016-06-20 15:53:56 -0700 | [diff] [blame] | 1601 | |
| 1602 | LOG(INFO) |
| 1603 | << "Developer features disabled; disallowing custom update sources."; |
David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 1604 | return false; |
| 1605 | } |
| 1606 | |
Andrew de los Reyes | 4e9b9f4 | 2010-04-26 15:06:43 -0700 | [diff] [blame] | 1607 | } // namespace chromeos_update_engine |