| Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // | 
|  | 2 | // Copyright (C) 2011 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 | // | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 16 |  | 
| Amin Hassani | ec7bc11 | 2020-10-29 16:47:58 -0700 | [diff] [blame] | 17 | #include "update_engine/common/download_action.h" | 
| Alex Deymo | aab50e3 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 18 |  | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 19 | #include <errno.h> | 
| Alex Deymo | e5e5fe9 | 2015-10-05 09:28:19 -0700 | [diff] [blame] | 20 |  | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 21 | #include <algorithm> | 
| Andrew de los Reyes | f971443 | 2010-05-04 10:21:23 -0700 | [diff] [blame] | 22 | #include <string> | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 23 |  | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 24 | #include <base/files/file_path.h> | 
| Lann Martin | 39f5714 | 2017-07-14 09:18:42 -0600 | [diff] [blame] | 25 | #include <base/metrics/statistics_recorder.h> | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 26 | #include <base/strings/stringprintf.h> | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 27 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 28 | #include "update_engine/common/action_pipe.h" | 
|  | 29 | #include "update_engine/common/boot_control_interface.h" | 
| Alex Deymo | ed9cc18 | 2016-06-15 16:19:29 -0700 | [diff] [blame] | 30 | #include "update_engine/common/error_code_utils.h" | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 31 | #include "update_engine/common/multi_range_http_fetcher.h" | 
| Kelvin Zhang | 5c1451f | 2020-12-02 17:01:54 -0500 | [diff] [blame] | 32 | #include "update_engine/common/prefs_interface.h" | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 33 | #include "update_engine/common/utils.h" | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 34 |  | 
| Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 35 | using base::FilePath; | 
| Andrew de los Reyes | f971443 | 2010-05-04 10:21:23 -0700 | [diff] [blame] | 36 | using std::string; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 37 |  | 
|  | 38 | namespace chromeos_update_engine { | 
|  | 39 |  | 
| Darin Petkov | 73058b4 | 2010-10-06 16:32:19 -0700 | [diff] [blame] | 40 | DownloadAction::DownloadAction(PrefsInterface* prefs, | 
| Alex Deymo | 1b3556c | 2016-02-03 09:54:02 -0800 | [diff] [blame] | 41 | BootControlInterface* boot_control, | 
|  | 42 | HardwareInterface* hardware, | 
| Amin Hassani | 7ecda26 | 2017-07-11 17:10:50 -0700 | [diff] [blame] | 43 | HttpFetcher* http_fetcher, | 
| Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 44 | bool interactive) | 
| Darin Petkov | 73058b4 | 2010-10-06 16:32:19 -0700 | [diff] [blame] | 45 | : prefs_(prefs), | 
| Alex Deymo | 1b3556c | 2016-02-03 09:54:02 -0800 | [diff] [blame] | 46 | boot_control_(boot_control), | 
|  | 47 | hardware_(hardware), | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 48 | http_fetcher_(new MultiRangeHttpFetcher(http_fetcher)), | 
| Amin Hassani | ed37d68 | 2018-04-06 13:22:00 -0700 | [diff] [blame] | 49 | interactive_(interactive), | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 50 | code_(ErrorCode::kSuccess), | 
| Kelvin Zhang | 5c1451f | 2020-12-02 17:01:54 -0500 | [diff] [blame] | 51 | delegate_(nullptr) {} | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 52 |  | 
|  | 53 | DownloadAction::~DownloadAction() {} | 
|  | 54 |  | 
|  | 55 | void DownloadAction::PerformAction() { | 
|  | 56 | http_fetcher_->set_delegate(this); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 57 |  | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 58 | // Get the InstallPlan and read it | 
|  | 59 | CHECK(HasInputObject()); | 
| Andrew de los Reyes | f918517 | 2010-05-03 11:07:05 -0700 | [diff] [blame] | 60 | install_plan_ = GetInputObject(); | 
| Andrew de los Reyes | f918517 | 2010-05-03 11:07:05 -0700 | [diff] [blame] | 61 | install_plan_.Dump(); | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 62 |  | 
|  | 63 | bytes_received_ = 0; | 
| Aaron Wood | 9321f50 | 2017-09-07 11:18:54 -0700 | [diff] [blame] | 64 | bytes_received_previous_payloads_ = 0; | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 65 | bytes_total_ = 0; | 
|  | 66 | for (const auto& payload : install_plan_.payloads) | 
|  | 67 | bytes_total_ += payload.size; | 
|  | 68 |  | 
|  | 69 | if (install_plan_.is_resume) { | 
|  | 70 | int64_t payload_index = 0; | 
|  | 71 | if (prefs_->GetInt64(kPrefsUpdateStatePayloadIndex, &payload_index) && | 
|  | 72 | static_cast<size_t>(payload_index) < install_plan_.payloads.size()) { | 
|  | 73 | // Save the index for the resume payload before downloading any previous | 
|  | 74 | // payload, otherwise it will be overwritten. | 
|  | 75 | resume_payload_index_ = payload_index; | 
|  | 76 | for (int i = 0; i < payload_index; i++) | 
|  | 77 | install_plan_.payloads[i].already_applied = true; | 
|  | 78 | } | 
|  | 79 | } | 
| Kelvin Zhang | 5c1451f | 2020-12-02 17:01:54 -0500 | [diff] [blame] | 80 | CHECK_GE(install_plan_.payloads.size(), 1UL); | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 81 | if (!payload_) | 
|  | 82 | payload_ = &install_plan_.payloads[0]; | 
| adlr@google.com | c98a7ed | 2009-12-04 18:54:03 +0000 | [diff] [blame] | 83 |  | 
| Alex Deymo | 5ed695e | 2015-10-05 16:59:23 -0700 | [diff] [blame] | 84 | LOG(INFO) << "Marking new slot as unbootable"; | 
| Alex Deymo | 1b3556c | 2016-02-03 09:54:02 -0800 | [diff] [blame] | 85 | if (!boot_control_->MarkSlotUnbootable(install_plan_.target_slot)) { | 
| Alex Deymo | 5ed695e | 2015-10-05 16:59:23 -0700 | [diff] [blame] | 86 | LOG(WARNING) << "Unable to mark new slot " | 
|  | 87 | << BootControlInterface::SlotName(install_plan_.target_slot) | 
|  | 88 | << ". Proceeding with the update anyway."; | 
|  | 89 | } | 
|  | 90 |  | 
| Sen Jiang | 6c73668 | 2017-03-10 15:01:36 -0800 | [diff] [blame] | 91 | StartDownloading(); | 
|  | 92 | } | 
|  | 93 |  | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 94 | bool DownloadAction::LoadCachedManifest(int64_t manifest_size) { | 
|  | 95 | std::string cached_manifest_bytes; | 
|  | 96 | if (!prefs_->GetString(kPrefsManifestBytes, &cached_manifest_bytes) || | 
|  | 97 | cached_manifest_bytes.size() <= 0) { | 
|  | 98 | LOG(INFO) << "Cached Manifest data not found"; | 
|  | 99 | return false; | 
|  | 100 | } | 
|  | 101 | if (static_cast<int64_t>(cached_manifest_bytes.size()) != manifest_size) { | 
|  | 102 | LOG(WARNING) << "Cached metadata has unexpected size: " | 
|  | 103 | << cached_manifest_bytes.size() << " vs. " << manifest_size; | 
|  | 104 | return false; | 
|  | 105 | } | 
|  | 106 |  | 
|  | 107 | ErrorCode error; | 
|  | 108 | const bool success = | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 109 | delta_performer_->Write( | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 110 | cached_manifest_bytes.data(), cached_manifest_bytes.size(), &error) && | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 111 | delta_performer_->IsManifestValid(); | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 112 | if (success) { | 
|  | 113 | LOG(INFO) << "Successfully parsed cached manifest"; | 
|  | 114 | } else { | 
|  | 115 | // If parsing of cached data failed, fall back to fetch them using HTTP | 
|  | 116 | LOG(WARNING) << "Cached manifest data fails to load, error code:" | 
|  | 117 | << static_cast<int>(error) << "," << error; | 
|  | 118 | } | 
|  | 119 | return success; | 
|  | 120 | } | 
|  | 121 |  | 
| Sen Jiang | 6c73668 | 2017-03-10 15:01:36 -0800 | [diff] [blame] | 122 | void DownloadAction::StartDownloading() { | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 123 | download_active_ = true; | 
|  | 124 | http_fetcher_->ClearRanges(); | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 125 |  | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 126 | if (delta_performer_ != nullptr) { | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 127 | LOG(INFO) << "Using writer for test."; | 
|  | 128 | } else { | 
|  | 129 | delta_performer_.reset(new DeltaPerformer(prefs_, | 
|  | 130 | boot_control_, | 
|  | 131 | hardware_, | 
|  | 132 | delegate_, | 
|  | 133 | &install_plan_, | 
|  | 134 | payload_, | 
|  | 135 | interactive_)); | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 136 | } | 
|  | 137 |  | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 138 | if (install_plan_.is_resume && | 
|  | 139 | payload_ == &install_plan_.payloads[resume_payload_index_]) { | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 140 | // Resuming an update so parse the cached manifest first | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 141 | int64_t manifest_metadata_size = 0; | 
|  | 142 | int64_t manifest_signature_size = 0; | 
|  | 143 | prefs_->GetInt64(kPrefsManifestMetadataSize, &manifest_metadata_size); | 
|  | 144 | prefs_->GetInt64(kPrefsManifestSignatureSize, &manifest_signature_size); | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 145 |  | 
|  | 146 | // TODO(zhangkelvin) Add unittest for success and fallback route | 
|  | 147 | if (!LoadCachedManifest(manifest_metadata_size + manifest_signature_size)) { | 
|  | 148 | if (delta_performer_) { | 
|  | 149 | // Create a new DeltaPerformer to reset all its state | 
|  | 150 | delta_performer_ = std::make_unique<DeltaPerformer>(prefs_, | 
|  | 151 | boot_control_, | 
|  | 152 | hardware_, | 
|  | 153 | delegate_, | 
|  | 154 | &install_plan_, | 
|  | 155 | payload_, | 
|  | 156 | interactive_); | 
| Kelvin Zhang | cc011d3 | 2020-07-10 18:20:08 -0400 | [diff] [blame] | 157 | } | 
|  | 158 | http_fetcher_->AddRange(base_offset_, | 
|  | 159 | manifest_metadata_size + manifest_signature_size); | 
|  | 160 | } | 
|  | 161 |  | 
| Kelvin Zhang | ead9fd7 | 2020-12-03 12:57:35 -0500 | [diff] [blame] | 162 | // If there're remaining unprocessed data blobs, fetch them. Be careful | 
|  | 163 | // not to request data beyond the end of the payload to avoid 416 HTTP | 
|  | 164 | // response error codes. | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 165 | int64_t next_data_offset = 0; | 
|  | 166 | prefs_->GetInt64(kPrefsUpdateStateNextDataOffset, &next_data_offset); | 
|  | 167 | uint64_t resume_offset = | 
|  | 168 | manifest_metadata_size + manifest_signature_size + next_data_offset; | 
|  | 169 | if (!payload_->size) { | 
|  | 170 | http_fetcher_->AddRange(base_offset_ + resume_offset); | 
|  | 171 | } else if (resume_offset < payload_->size) { | 
|  | 172 | http_fetcher_->AddRange(base_offset_ + resume_offset, | 
|  | 173 | payload_->size - resume_offset); | 
|  | 174 | } | 
|  | 175 | } else { | 
|  | 176 | if (payload_->size) { | 
|  | 177 | http_fetcher_->AddRange(base_offset_, payload_->size); | 
|  | 178 | } else { | 
|  | 179 | // If no payload size is passed we assume we read until the end of the | 
|  | 180 | // stream. | 
|  | 181 | http_fetcher_->AddRange(base_offset_); | 
|  | 182 | } | 
|  | 183 | } | 
|  | 184 |  | 
| Andrew de los Reyes | f918517 | 2010-05-03 11:07:05 -0700 | [diff] [blame] | 185 | http_fetcher_->BeginTransfer(install_plan_.download_url); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 186 | } | 
|  | 187 |  | 
| Alex Deymo | f285857 | 2016-02-25 11:20:13 -0800 | [diff] [blame] | 188 | void DownloadAction::SuspendAction() { | 
|  | 189 | http_fetcher_->Pause(); | 
|  | 190 | } | 
|  | 191 |  | 
|  | 192 | void DownloadAction::ResumeAction() { | 
|  | 193 | http_fetcher_->Unpause(); | 
|  | 194 | } | 
|  | 195 |  | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 196 | void DownloadAction::TerminateProcessing() { | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 197 | if (delta_performer_) { | 
|  | 198 | delta_performer_->Close(); | 
|  | 199 | delta_performer_.reset(); | 
| Darin Petkov | 698d041 | 2010-10-13 10:59:44 -0700 | [diff] [blame] | 200 | } | 
| Alex Deymo | 1b3556c | 2016-02-03 09:54:02 -0800 | [diff] [blame] | 201 | download_active_ = false; | 
| Darin Petkov | 9ce452b | 2010-11-17 14:33:28 -0800 | [diff] [blame] | 202 | // Terminates the transfer. The action is terminated, if necessary, when the | 
|  | 203 | // TransferTerminated callback is received. | 
|  | 204 | http_fetcher_->TerminateTransfer(); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 205 | } | 
|  | 206 |  | 
| Andrew de los Reyes | 34e41a1 | 2010-10-26 20:07:58 -0700 | [diff] [blame] | 207 | void DownloadAction::SeekToOffset(off_t offset) { | 
|  | 208 | bytes_received_ = offset; | 
|  | 209 | } | 
|  | 210 |  | 
| Amin Hassani | 0cd9d77 | 2018-07-31 23:55:43 -0700 | [diff] [blame] | 211 | bool DownloadAction::ReceivedBytes(HttpFetcher* fetcher, | 
| Alex Vakulenko | f68bbbc | 2015-02-09 12:53:18 -0800 | [diff] [blame] | 212 | const void* bytes, | 
|  | 213 | size_t length) { | 
| Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 214 | bytes_received_ += length; | 
| Aaron Wood | 9321f50 | 2017-09-07 11:18:54 -0700 | [diff] [blame] | 215 | uint64_t bytes_downloaded_total = | 
|  | 216 | bytes_received_previous_payloads_ + bytes_received_; | 
| Alex Deymo | 542c19b | 2015-12-03 07:43:31 -0300 | [diff] [blame] | 217 | if (delegate_ && download_active_) { | 
| Aaron Wood | 9321f50 | 2017-09-07 11:18:54 -0700 | [diff] [blame] | 218 | delegate_->BytesReceived(length, bytes_downloaded_total, bytes_total_); | 
| Alex Deymo | 542c19b | 2015-12-03 07:43:31 -0300 | [diff] [blame] | 219 | } | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 220 | if (delta_performer_ && !delta_performer_->Write(bytes, length, &code_)) { | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 221 | if (code_ != ErrorCode::kSuccess) { | 
|  | 222 | LOG(ERROR) << "Error " << utils::ErrorCodeToString(code_) << " (" << code_ | 
|  | 223 | << ") in DeltaPerformer's Write method when " | 
|  | 224 | << "processing the received payload -- Terminating processing"; | 
|  | 225 | } | 
| Darin Petkov | 9ce452b | 2010-11-17 14:33:28 -0800 | [diff] [blame] | 226 | // Don't tell the action processor that the action is complete until we get | 
|  | 227 | // the TransferTerminated callback. Otherwise, this and the HTTP fetcher | 
|  | 228 | // objects may get destroyed before all callbacks are complete. | 
| Darin Petkov | 698d041 | 2010-10-13 10:59:44 -0700 | [diff] [blame] | 229 | TerminateProcessing(); | 
| Amin Hassani | 0cd9d77 | 2018-07-31 23:55:43 -0700 | [diff] [blame] | 230 | return false; | 
| Darin Petkov | 698d041 | 2010-10-13 10:59:44 -0700 | [diff] [blame] | 231 | } | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 232 |  | 
| Amin Hassani | 0cd9d77 | 2018-07-31 23:55:43 -0700 | [diff] [blame] | 233 | return true; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 234 | } | 
|  | 235 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 236 | void DownloadAction::TransferComplete(HttpFetcher* fetcher, bool successful) { | 
| Kelvin Zhang | 22b62e4 | 2020-12-14 14:50:40 -0500 | [diff] [blame] | 237 | if (delta_performer_) { | 
|  | 238 | LOG_IF(WARNING, delta_performer_->Close() != 0) | 
|  | 239 | << "Error closing the writer."; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 240 | } | 
| Alex Deymo | 1b3556c | 2016-02-03 09:54:02 -0800 | [diff] [blame] | 241 | download_active_ = false; | 
| David Zeuthen | a99981f | 2013-04-29 13:42:47 -0700 | [diff] [blame] | 242 | ErrorCode code = | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 243 | successful ? ErrorCode::kSuccess : ErrorCode::kDownloadTransferError; | 
| Aaron Wood | 9321f50 | 2017-09-07 11:18:54 -0700 | [diff] [blame] | 244 | if (code == ErrorCode::kSuccess) { | 
|  | 245 | if (delta_performer_ && !payload_->already_applied) | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 246 | code = delta_performer_->VerifyPayload(payload_->hash, payload_->size); | 
| Lann Martin | 39f5714 | 2017-07-14 09:18:42 -0600 | [diff] [blame] | 247 | if (code == ErrorCode::kSuccess) { | 
| Kelvin Zhang | 5c1451f | 2020-12-02 17:01:54 -0500 | [diff] [blame] | 248 | CHECK_EQ(install_plan_.payloads.size(), 1UL); | 
| Sen Jiang | 7ebfccf | 2018-03-15 13:55:55 -0700 | [diff] [blame] | 249 | // All payloads have been applied and verified. | 
|  | 250 | if (delegate_) | 
|  | 251 | delegate_->DownloadComplete(); | 
|  | 252 |  | 
| Lann Martin | 39f5714 | 2017-07-14 09:18:42 -0600 | [diff] [blame] | 253 | // Log UpdateEngine.DownloadAction.* histograms to help diagnose | 
| Sen Jiang | 7ebfccf | 2018-03-15 13:55:55 -0700 | [diff] [blame] | 254 | // long-blocking operations. | 
| Lann Martin | 39f5714 | 2017-07-14 09:18:42 -0600 | [diff] [blame] | 255 | std::string histogram_output; | 
| Amin Hassani | 008c458 | 2019-01-13 16:22:47 -0800 | [diff] [blame] | 256 | base::StatisticsRecorder::WriteGraph("UpdateEngine.DownloadAction.", | 
|  | 257 | &histogram_output); | 
| Lann Martin | 39f5714 | 2017-07-14 09:18:42 -0600 | [diff] [blame] | 258 | LOG(INFO) << histogram_output; | 
|  | 259 | } else { | 
| Darin Petkov | 7ed561b | 2011-10-04 02:59:03 -0700 | [diff] [blame] | 260 | LOG(ERROR) << "Download of " << install_plan_.download_url | 
|  | 261 | << " failed due to payload verification error."; | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 262 | } | 
|  | 263 | } | 
| Darin Petkov | c1a8b42 | 2010-07-19 11:34:49 -0700 | [diff] [blame] | 264 |  | 
| Darin Petkov | c97435c | 2010-07-20 12:37:43 -0700 | [diff] [blame] | 265 | // Write the path to the output pipe if we're successful. | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 266 | if (code == ErrorCode::kSuccess && HasOutputPipe()) | 
| Darin Petkov | 3aefa86 | 2010-12-07 14:45:00 -0800 | [diff] [blame] | 267 | SetOutputObject(install_plan_); | 
| Darin Petkov | c97435c | 2010-07-20 12:37:43 -0700 | [diff] [blame] | 268 | processor_->ActionComplete(this, code); | 
| rspangler@google.com | 49fdf18 | 2009-10-10 00:57:34 +0000 | [diff] [blame] | 269 | } | 
|  | 270 |  | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 271 | void DownloadAction::TransferTerminated(HttpFetcher* fetcher) { | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 272 | if (code_ != ErrorCode::kSuccess) { | 
| Darin Petkov | 9ce452b | 2010-11-17 14:33:28 -0800 | [diff] [blame] | 273 | processor_->ActionComplete(this, code_); | 
| Sen Jiang | 5ae865b | 2017-04-18 14:24:40 -0700 | [diff] [blame] | 274 | } else if (payload_->already_applied) { | 
|  | 275 | LOG(INFO) << "TransferTerminated with ErrorCode::kSuccess when the current " | 
|  | 276 | "payload has already applied, treating as TransferComplete."; | 
|  | 277 | TransferComplete(fetcher, true); | 
| Darin Petkov | 9ce452b | 2010-11-17 14:33:28 -0800 | [diff] [blame] | 278 | } | 
|  | 279 | } | 
|  | 280 |  | 
| Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 281 | }  // namespace chromeos_update_engine |