| 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 | // | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 16 |  | 
|  | 17 | #include "update_engine/omaha_response_handler_action.h" | 
| Darin Petkov | 73058b4 | 2010-10-06 16:32:19 -0700 | [diff] [blame] | 18 |  | 
| Zentaro Kavanagh | 0ff621c | 2018-07-13 13:06:56 -0700 | [diff] [blame] | 19 | #include <limits> | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 20 | #include <string> | 
| Darin Petkov | 73058b4 | 2010-10-06 16:32:19 -0700 | [diff] [blame] | 21 |  | 
|  | 22 | #include <base/logging.h> | 
| Sen Jiang | 2703ef4 | 2017-03-16 13:36:21 -0700 | [diff] [blame] | 23 | #include <base/strings/string_number_conversions.h> | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 24 | #include <base/strings/string_util.h> | 
| Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 25 | #include <policy/device_policy.h> | 
| Darin Petkov | 73058b4 | 2010-10-06 16:32:19 -0700 | [diff] [blame] | 26 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 27 | #include "update_engine/common/constants.h" | 
|  | 28 | #include "update_engine/common/hardware_interface.h" | 
|  | 29 | #include "update_engine/common/prefs_interface.h" | 
|  | 30 | #include "update_engine/common/utils.h" | 
| Alex Deymo | f6ee016 | 2015-07-31 12:35:22 -0700 | [diff] [blame] | 31 | #include "update_engine/connection_manager_interface.h" | 
| Gilad Arnold | 1f84723 | 2014-04-07 12:07:49 -0700 | [diff] [blame] | 32 | #include "update_engine/omaha_request_params.h" | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 33 | #include "update_engine/payload_consumer/delta_performer.h" | 
| Jay Srinivasan | 55f50c2 | 2013-01-10 19:24:35 -0800 | [diff] [blame] | 34 | #include "update_engine/payload_state_interface.h" | 
| Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 35 | #include "update_engine/update_manager/policy.h" | 
|  | 36 | #include "update_engine/update_manager/update_manager.h" | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 37 |  | 
| Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 38 | using chromeos_update_manager::Policy; | 
|  | 39 | using chromeos_update_manager::UpdateManager; | 
| Zentaro Kavanagh | 0ff621c | 2018-07-13 13:06:56 -0700 | [diff] [blame] | 40 | using std::numeric_limits; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 41 | using std::string; | 
|  | 42 |  | 
|  | 43 | namespace chromeos_update_engine { | 
|  | 44 |  | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 45 | OmahaResponseHandlerAction::OmahaResponseHandlerAction( | 
|  | 46 | SystemState* system_state) | 
| Gilad Arnold | 4dbd47e | 2013-07-22 05:39:26 -0700 | [diff] [blame] | 47 | : system_state_(system_state), | 
| Amin Hassani | d3f4bea | 2018-04-30 14:52:40 -0700 | [diff] [blame] | 48 | deadline_file_(constants::kOmahaResponseDeadlineFile) {} | 
| Darin Petkov | abc7bc0 | 2011-02-23 14:39:43 -0800 | [diff] [blame] | 49 |  | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 50 | void OmahaResponseHandlerAction::PerformAction() { | 
|  | 51 | CHECK(HasInputObject()); | 
|  | 52 | ScopedActionCompleter completer(processor_, this); | 
| Darin Petkov | 6a5b322 | 2010-07-13 14:55:28 -0700 | [diff] [blame] | 53 | const OmahaResponse& response = GetInputObject(); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 54 | if (!response.update_exists) { | 
|  | 55 | LOG(INFO) << "There are no updates. Aborting."; | 
| Sen Jiang | 3978ddd | 2018-03-22 18:05:44 -0700 | [diff] [blame] | 56 | completer.set_code(ErrorCode::kNoUpdate); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 57 | return; | 
|  | 58 | } | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 59 |  | 
|  | 60 | // All decisions as to which URL should be used have already been done. So, | 
| Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 61 | // make the current URL as the download URL. | 
|  | 62 | string current_url = system_state_->payload_state()->GetCurrentUrl(); | 
|  | 63 | if (current_url.empty()) { | 
|  | 64 | // This shouldn't happen as we should always supply the HTTPS backup URL. | 
|  | 65 | // Handling this anyway, just in case. | 
|  | 66 | LOG(ERROR) << "There are no suitable URLs in the response to use."; | 
| Gilad Arnold | d1c4d2d | 2014-06-05 14:07:53 -0700 | [diff] [blame] | 67 | completer.set_code(ErrorCode::kOmahaResponseInvalid); | 
| Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 68 | return; | 
|  | 69 | } | 
|  | 70 |  | 
| Aaron Wood | 7f92e2b | 2017-08-28 14:51:21 -0700 | [diff] [blame] | 71 | // This is the url to the first package, not all packages. | 
| Jay Srinivasan | 53173b9 | 2013-05-17 17:13:01 -0700 | [diff] [blame] | 72 | install_plan_.download_url = current_url; | 
| Chris Sosa | fb1020e | 2013-07-29 17:27:33 -0700 | [diff] [blame] | 73 | install_plan_.version = response.version; | 
| Aaron Wood | 7dcdedf | 2017-09-06 17:17:41 -0700 | [diff] [blame] | 74 | install_plan_.system_version = response.system_version; | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 75 |  | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 76 | OmahaRequestParams* const params = system_state_->request_params(); | 
|  | 77 | PayloadStateInterface* const payload_state = system_state_->payload_state(); | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 78 |  | 
|  | 79 | // If we're using p2p to download and there is a local peer, use it. | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 80 | if (payload_state->GetUsingP2PForDownloading() && | 
|  | 81 | !payload_state->GetP2PUrl().empty()) { | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 82 | LOG(INFO) << "Replacing URL " << install_plan_.download_url | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 83 | << " with local URL " << payload_state->GetP2PUrl() | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 84 | << " since p2p is enabled."; | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 85 | install_plan_.download_url = payload_state->GetP2PUrl(); | 
|  | 86 | payload_state->SetUsingP2PForDownloading(true); | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 87 | } | 
|  | 88 |  | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 89 | // Fill up the other properties based on the response. | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 90 | string update_check_response_hash; | 
|  | 91 | for (const auto& package : response.packages) { | 
|  | 92 | brillo::Blob raw_hash; | 
|  | 93 | if (!base::HexStringToBytes(package.hash, &raw_hash)) { | 
|  | 94 | LOG(ERROR) << "Failed to convert payload hash from hex string to bytes: " | 
|  | 95 | << package.hash; | 
|  | 96 | completer.set_code(ErrorCode::kOmahaResponseInvalid); | 
|  | 97 | return; | 
|  | 98 | } | 
|  | 99 | install_plan_.payloads.push_back( | 
|  | 100 | {.size = package.size, | 
|  | 101 | .metadata_size = package.metadata_size, | 
|  | 102 | .metadata_signature = package.metadata_signature, | 
| Sen Jiang | cdd5206 | 2017-05-18 15:33:10 -0700 | [diff] [blame] | 103 | .hash = raw_hash, | 
|  | 104 | .type = package.is_delta ? InstallPayloadType::kDelta | 
|  | 105 | : InstallPayloadType::kFull}); | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 106 | update_check_response_hash += package.hash + ":"; | 
| Sen Jiang | 2703ef4 | 2017-03-16 13:36:21 -0700 | [diff] [blame] | 107 | } | 
| David Zeuthen | e7f8917 | 2013-10-31 10:21:04 -0700 | [diff] [blame] | 108 | install_plan_.public_key_rsa = response.public_key_rsa; | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 109 | install_plan_.hash_checks_mandatory = AreHashChecksMandatory(response); | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 110 | install_plan_.is_resume = DeltaPerformer::CanResumeUpdate( | 
|  | 111 | system_state_->prefs(), update_check_response_hash); | 
| Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 112 | if (install_plan_.is_resume) { | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 113 | payload_state->UpdateResumed(); | 
| Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 114 | } else { | 
| Gilad Arnold | 74b5f55 | 2014-10-07 08:17:16 -0700 | [diff] [blame] | 115 | payload_state->UpdateRestarted(); | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 116 | LOG_IF(WARNING, | 
|  | 117 | !DeltaPerformer::ResetUpdateProgress(system_state_->prefs(), false)) | 
| Darin Petkov | 0406e40 | 2010-10-06 21:33:11 -0700 | [diff] [blame] | 118 | << "Unable to reset the update progress."; | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 119 | LOG_IF(WARNING, | 
|  | 120 | !system_state_->prefs()->SetString(kPrefsUpdateCheckResponseHash, | 
|  | 121 | update_check_response_hash)) | 
| Darin Petkov | 0406e40 | 2010-10-06 21:33:11 -0700 | [diff] [blame] | 122 | << "Unable to save the update check response hash."; | 
|  | 123 | } | 
|  | 124 |  | 
| Alex Deymo | 763e7db | 2015-08-27 21:08:08 -0700 | [diff] [blame] | 125 | install_plan_.source_slot = system_state_->boot_control()->GetCurrentSlot(); | 
|  | 126 | install_plan_.target_slot = install_plan_.source_slot == 0 ? 1 : 0; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 127 |  | 
| Alex Deymo | 8561665 | 2015-10-15 18:48:31 -0700 | [diff] [blame] | 128 | // The Omaha response doesn't include the channel name for this image, so we | 
|  | 129 | // use the download_channel we used during the request to tag the target slot. | 
|  | 130 | // This will be used in the next boot to know the channel the image was | 
|  | 131 | // downloaded from. | 
|  | 132 | string current_channel_key = | 
|  | 133 | kPrefsChannelOnSlotPrefix + std::to_string(install_plan_.target_slot); | 
|  | 134 | system_state_->prefs()->SetString(current_channel_key, | 
|  | 135 | params->download_channel()); | 
|  | 136 |  | 
| Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 137 | // Checking whether device is able to boot up the returned rollback image. | 
|  | 138 | if (response.is_rollback) { | 
|  | 139 | if (!params->rollback_allowed()) { | 
|  | 140 | LOG(ERROR) << "Received rollback image but rollback is not allowed."; | 
|  | 141 | completer.set_code(ErrorCode::kOmahaResponseInvalid); | 
|  | 142 | return; | 
|  | 143 | } | 
|  | 144 | auto min_kernel_key_version = static_cast<uint32_t>( | 
|  | 145 | system_state_->hardware()->GetMinKernelKeyVersion()); | 
|  | 146 | auto min_firmware_key_version = static_cast<uint32_t>( | 
|  | 147 | system_state_->hardware()->GetMinFirmwareKeyVersion()); | 
| Zentaro Kavanagh | 0ff621c | 2018-07-13 13:06:56 -0700 | [diff] [blame] | 148 | uint32_t kernel_key_version = | 
|  | 149 | static_cast<uint32_t>(response.rollback_key_version.kernel_key) << 16 | | 
|  | 150 | static_cast<uint32_t>(response.rollback_key_version.kernel); | 
|  | 151 | uint32_t firmware_key_version = | 
|  | 152 | static_cast<uint32_t>(response.rollback_key_version.firmware_key) | 
|  | 153 | << 16 | | 
|  | 154 | static_cast<uint32_t>(response.rollback_key_version.firmware); | 
|  | 155 |  | 
|  | 156 | // Don't attempt a rollback if the versions are incompatible or the | 
|  | 157 | // target image does not specify the version information. | 
|  | 158 | if (kernel_key_version == numeric_limits<uint32_t>::max() || | 
|  | 159 | firmware_key_version == numeric_limits<uint32_t>::max() || | 
|  | 160 | kernel_key_version < min_kernel_key_version || | 
|  | 161 | firmware_key_version < min_firmware_key_version) { | 
| Marton Hunyady | 199152d | 2018-05-07 19:08:48 +0200 | [diff] [blame] | 162 | LOG(ERROR) << "Device won't be able to boot up the rollback image."; | 
|  | 163 | completer.set_code(ErrorCode::kRollbackNotPossible); | 
|  | 164 | return; | 
|  | 165 | } | 
|  | 166 | install_plan_.is_rollback = true; | 
|  | 167 | } | 
|  | 168 |  | 
| Sen Jiang | fe28440 | 2018-03-21 14:03:50 -0700 | [diff] [blame] | 169 | if (response.powerwash_required || params->ShouldPowerwash()) | 
| Jay Srinivasan | ae4697c | 2013-03-18 17:08:08 -0700 | [diff] [blame] | 170 | install_plan_.powerwash_required = true; | 
| Jay Srinivasan | 1c0fe79 | 2013-03-28 16:45:25 -0700 | [diff] [blame] | 171 |  | 
| Andrew de los Reyes | f98bff8 | 2010-05-06 13:33:25 -0700 | [diff] [blame] | 172 | TEST_AND_RETURN(HasOutputPipe()); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 173 | if (HasOutputPipe()) | 
| Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 174 | SetOutputObject(install_plan_); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 175 | LOG(INFO) << "Using this install plan:"; | 
| Andrew de los Reyes | 63b96d7 | 2010-05-10 13:08:54 -0700 | [diff] [blame] | 176 | install_plan_.Dump(); | 
| Darin Petkov | 6a5b322 | 2010-07-13 14:55:28 -0700 | [diff] [blame] | 177 |  | 
| Darin Petkov | 6c11864 | 2010-10-21 12:06:30 -0700 | [diff] [blame] | 178 | // Send the deadline data (if any) to Chrome through a file. This is a pretty | 
|  | 179 | // hacky solution but should be OK for now. | 
|  | 180 | // | 
| Alex Vakulenko | 072359c | 2014-07-18 11:41:07 -0700 | [diff] [blame] | 181 | // TODO(petkov): Re-architect this to avoid communication through a | 
| Chris Sosa | be45bef | 2013-04-09 18:25:12 -0700 | [diff] [blame] | 182 | // file. Ideally, we would include this information in D-Bus's GetStatus | 
| Darin Petkov | 6c11864 | 2010-10-21 12:06:30 -0700 | [diff] [blame] | 183 | // method and UpdateStatus signal. A potential issue is that update_engine may | 
|  | 184 | // be unresponsive during an update download. | 
| Alex Deymo | 6dd160a | 2015-10-09 18:45:14 -0700 | [diff] [blame] | 185 | if (!deadline_file_.empty()) { | 
| Marton Hunyady | e58bddb | 2018-04-10 20:27:26 +0200 | [diff] [blame] | 186 | if (payload_state->GetRollbackHappened()) { | 
|  | 187 | // Don't do forced update if rollback has happened since the last update | 
|  | 188 | // check where policy was present. | 
|  | 189 | LOG(INFO) << "Not forcing update because a rollback happened."; | 
|  | 190 | utils::WriteFile(deadline_file_.c_str(), nullptr, 0); | 
|  | 191 | } else { | 
|  | 192 | utils::WriteFile(deadline_file_.c_str(), | 
|  | 193 | response.deadline.data(), | 
|  | 194 | response.deadline.size()); | 
|  | 195 | } | 
| Alex Deymo | 6dd160a | 2015-10-09 18:45:14 -0700 | [diff] [blame] | 196 | chmod(deadline_file_.c_str(), S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH); | 
|  | 197 | } | 
| Darin Petkov | 6c11864 | 2010-10-21 12:06:30 -0700 | [diff] [blame] | 198 |  | 
| Aaron Wood | 23bd339 | 2017-10-06 14:48:25 -0700 | [diff] [blame] | 199 | // Check the generated install-plan with the Policy to confirm that | 
|  | 200 | // it can be applied at this time (or at all). | 
|  | 201 | UpdateManager* const update_manager = system_state_->update_manager(); | 
|  | 202 | CHECK(update_manager); | 
|  | 203 | auto ec = ErrorCode::kSuccess; | 
|  | 204 | update_manager->PolicyRequest( | 
|  | 205 | &Policy::UpdateCanBeApplied, &ec, &install_plan_); | 
|  | 206 | completer.set_code(ec); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 207 | } | 
|  | 208 |  | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 209 | bool OmahaResponseHandlerAction::AreHashChecksMandatory( | 
|  | 210 | const OmahaResponse& response) { | 
| David Pursell | 907b4fa | 2015-01-27 10:27:38 -0800 | [diff] [blame] | 211 | // We sometimes need to waive the hash checks in order to download from | 
|  | 212 | // sources that don't provide hashes, such as dev server. | 
|  | 213 | // At this point UpdateAttempter::IsAnyUpdateSourceAllowed() has already been | 
|  | 214 | // checked, so an unofficial update URL won't get this far unless it's OK to | 
|  | 215 | // use without a hash. Additionally, we want to always waive hash checks on | 
|  | 216 | // unofficial builds (i.e. dev/test images). | 
|  | 217 | // The end result is this: | 
|  | 218 | //  * Base image: | 
|  | 219 | //    - Official URLs require a hash. | 
|  | 220 | //    - Unofficial URLs only get this far if the IsAnyUpdateSourceAllowed() | 
|  | 221 | //      devmode/debugd checks pass, in which case the hash is waived. | 
|  | 222 | //  * Dev/test image: | 
|  | 223 | //    - Any URL is allowed through with no hash checking. | 
|  | 224 | if (!system_state_->request_params()->IsUpdateUrlOfficial() || | 
|  | 225 | !system_state_->hardware()->IsOfficialBuild()) { | 
| David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 226 | // Still do a hash check if a public key is included. | 
| David Zeuthen | e7f8917 | 2013-10-31 10:21:04 -0700 | [diff] [blame] | 227 | if (!response.public_key_rsa.empty()) { | 
| David Zeuthen | bc27aac | 2013-11-26 11:17:48 -0800 | [diff] [blame] | 228 | // The autoupdate_CatchBadSignatures test checks for this string | 
|  | 229 | // in log-files. Keep in sync. | 
| David Zeuthen | e7f8917 | 2013-10-31 10:21:04 -0700 | [diff] [blame] | 230 | LOG(INFO) << "Mandating payload hash checks since Omaha Response " | 
|  | 231 | << "for unofficial build includes public RSA key."; | 
|  | 232 | return true; | 
|  | 233 | } else { | 
| David Pursell | 02c1864 | 2014-11-06 11:26:11 -0800 | [diff] [blame] | 234 | LOG(INFO) << "Waiving payload hash checks for unofficial update URL."; | 
| David Zeuthen | e7f8917 | 2013-10-31 10:21:04 -0700 | [diff] [blame] | 235 | return false; | 
|  | 236 | } | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 237 | } | 
|  | 238 |  | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 239 | // If we're using p2p, |install_plan_.download_url| may contain a | 
|  | 240 | // HTTP URL even if |response.payload_urls| contain only HTTPS URLs. | 
| Alex Vakulenko | 0103c36 | 2016-01-20 07:56:15 -0800 | [diff] [blame] | 241 | if (!base::StartsWith(install_plan_.download_url, "https://", | 
|  | 242 | base::CompareCase::INSENSITIVE_ASCII)) { | 
| David Zeuthen | 8f191b2 | 2013-08-06 12:27:50 -0700 | [diff] [blame] | 243 | LOG(INFO) << "Mandating hash checks since download_url is not HTTPS."; | 
|  | 244 | return true; | 
|  | 245 | } | 
|  | 246 |  | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 247 | // TODO(jaysri): VALIDATION: For official builds, we currently waive hash | 
|  | 248 | // checks for HTTPS until we have rolled out at least once and are confident | 
|  | 249 | // nothing breaks. chromium-os:37082 tracks turning this on for HTTPS | 
|  | 250 | // eventually. | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 251 |  | 
|  | 252 | // Even if there's a single non-HTTPS URL, make the hash checks as | 
|  | 253 | // mandatory because we could be downloading the payload from any URL later | 
|  | 254 | // on. It's really hard to do book-keeping based on each byte being | 
|  | 255 | // downloaded to see whether we only used HTTPS throughout. | 
| Sen Jiang | 0affc2c | 2017-02-10 15:55:05 -0800 | [diff] [blame] | 256 | for (const auto& package : response.packages) { | 
|  | 257 | for (const string& payload_url : package.payload_urls) { | 
|  | 258 | if (!base::StartsWith( | 
|  | 259 | payload_url, "https://", base::CompareCase::INSENSITIVE_ASCII)) { | 
|  | 260 | LOG(INFO) << "Mandating payload hash checks since Omaha response " | 
|  | 261 | << "contains non-HTTPS URL(s)"; | 
|  | 262 | return true; | 
|  | 263 | } | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 264 | } | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 265 | } | 
|  | 266 |  | 
| Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 267 | LOG(INFO) << "Waiving payload hash checks since Omaha response " | 
|  | 268 | << "only has HTTPS URL(s)"; | 
|  | 269 | return false; | 
| Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 270 | } | 
|  | 271 |  | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 272 | }  // namespace chromeos_update_engine |