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 | // |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 16 | |
Alex Deymo | 923d8fa | 2014-07-15 17:58:51 -0700 | [diff] [blame] | 17 | #include "update_engine/payload_generator/payload_signer.h" |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 18 | |
Alex Deymo | 6f20dd4 | 2015-08-18 16:42:46 -0700 | [diff] [blame] | 19 | #include <endian.h> |
| 20 | |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 21 | #include <memory> |
Sen Jiang | b5f601d | 2018-02-02 13:51:21 -0800 | [diff] [blame] | 22 | #include <utility> |
| 23 | |
Darin Petkov | b039d50 | 2010-12-03 09:08:04 -0800 | [diff] [blame] | 24 | #include <base/logging.h> |
Alex Deymo | 98e691c | 2016-02-04 21:05:45 -0800 | [diff] [blame] | 25 | #include <base/strings/string_number_conversions.h> |
Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 26 | #include <base/strings/string_split.h> |
| 27 | #include <base/strings/string_util.h> |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 28 | #include <brillo/data_encoding.h> |
Sen Jiang | 923886a | 2016-03-14 15:04:28 -0700 | [diff] [blame] | 29 | #include <openssl/err.h> |
Darin Petkov | b039d50 | 2010-12-03 09:08:04 -0800 | [diff] [blame] | 30 | #include <openssl/pem.h> |
Kelvin Zhang | fc81688 | 2022-11-17 15:28:51 -0800 | [diff] [blame^] | 31 | #include <unistd.h> |
Darin Petkov | b039d50 | 2010-12-03 09:08:04 -0800 | [diff] [blame] | 32 | |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 33 | #include "update_engine/common/constants.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 34 | #include "update_engine/common/hash_calculator.h" |
| 35 | #include "update_engine/common/subprocess.h" |
| 36 | #include "update_engine/common/utils.h" |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 37 | #include "update_engine/payload_consumer/payload_metadata.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 38 | #include "update_engine/payload_consumer/payload_verifier.h" |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 39 | #include "update_engine/update_metadata.pb.h" |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 40 | |
| 41 | using std::string; |
| 42 | using std::vector; |
| 43 | |
| 44 | namespace chromeos_update_engine { |
| 45 | |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 46 | namespace { |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 47 | // Given raw |signatures|, packs them into a protobuf and serializes it into a |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 48 | // string. Returns true on success, false otherwise. |
| 49 | bool ConvertSignaturesToProtobuf(const vector<brillo::Blob>& signatures, |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 50 | const vector<size_t>& padded_signature_sizes, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 51 | string* out_serialized_signature) { |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 52 | TEST_AND_RETURN_FALSE(signatures.size() == padded_signature_sizes.size()); |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 53 | // Pack it into a protobuf |
| 54 | Signatures out_message; |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 55 | for (size_t i = 0; i < signatures.size(); i++) { |
| 56 | const auto& signature = signatures[i]; |
| 57 | const auto& padded_signature_size = padded_signature_sizes[i]; |
| 58 | TEST_AND_RETURN_FALSE(padded_signature_size >= signature.size()); |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 59 | Signatures::Signature* sig_message = out_message.add_signatures(); |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 60 | // Skip assigning the same version number because we don't need to be |
| 61 | // compatible with old major version 1 client anymore. |
| 62 | |
| 63 | // TODO(Xunchang) don't need to set the unpadded_signature_size field for |
| 64 | // RSA key signed signatures. |
| 65 | sig_message->set_unpadded_signature_size(signature.size()); |
| 66 | brillo::Blob padded_signature = signature; |
| 67 | padded_signature.insert( |
| 68 | padded_signature.end(), padded_signature_size - signature.size(), 0); |
| 69 | sig_message->set_data(padded_signature.data(), padded_signature.size()); |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 70 | } |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 71 | |
| 72 | // Serialize protobuf |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 73 | TEST_AND_RETURN_FALSE( |
| 74 | out_message.SerializeToString(out_serialized_signature)); |
| 75 | LOG(INFO) << "Signature blob size: " << out_serialized_signature->size(); |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 76 | return true; |
| 77 | } |
| 78 | |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 79 | // Given an unsigned payload under |payload_path| and the |payload_signature| |
| 80 | // and |metadata_signature| generates an updated payload that includes the |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 81 | // signatures. It populates |out_metadata_size| with the size of the final |
Tianjie | d60dc39 | 2020-07-29 11:27:35 -0700 | [diff] [blame] | 82 | // manifest after adding the fake signature operation, and |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 83 | // |out_signatures_offset| with the expected offset for the new blob, and |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 84 | // |out_metadata_signature_size| which will be size of |metadata_signature| |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 85 | // if the payload major version supports metadata signature, 0 otherwise. |
| 86 | // Returns true on success, false otherwise. |
| 87 | bool AddSignatureBlobToPayload(const string& payload_path, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 88 | const string& payload_signature, |
| 89 | const string& metadata_signature, |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 90 | brillo::Blob* out_payload, |
| 91 | uint64_t* out_metadata_size, |
| 92 | uint32_t* out_metadata_signature_size, |
| 93 | uint64_t* out_signatures_offset) { |
| 94 | uint64_t manifest_offset = 20; |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 95 | const int kProtobufSizeOffset = 12; |
| 96 | |
Alex Deymo | 98e691c | 2016-02-04 21:05:45 -0800 | [diff] [blame] | 97 | brillo::Blob payload; |
| 98 | TEST_AND_RETURN_FALSE(utils::ReadFile(payload_path, &payload)); |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 99 | PayloadMetadata payload_metadata; |
Amin Hassani | 822d485 | 2020-03-10 01:50:42 +0000 | [diff] [blame] | 100 | TEST_AND_RETURN_FALSE(payload_metadata.ParsePayloadHeader(payload)); |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 101 | uint64_t metadata_size = payload_metadata.GetMetadataSize(); |
| 102 | uint32_t metadata_signature_size = |
| 103 | payload_metadata.GetMetadataSignatureSize(); |
Amin Hassani | 55c7541 | 2019-10-07 11:20:39 -0700 | [diff] [blame] | 104 | // Write metadata signature size in header. |
Tianjie | 55abd3c | 2020-06-19 00:22:59 -0700 | [diff] [blame] | 105 | uint32_t metadata_signature_size_be = htobe32(metadata_signature.size()); |
Amin Hassani | 55c7541 | 2019-10-07 11:20:39 -0700 | [diff] [blame] | 106 | memcpy(payload.data() + manifest_offset, |
| 107 | &metadata_signature_size_be, |
| 108 | sizeof(metadata_signature_size_be)); |
| 109 | manifest_offset += sizeof(metadata_signature_size_be); |
| 110 | // Replace metadata signature. |
| 111 | payload.erase(payload.begin() + metadata_size, |
| 112 | payload.begin() + metadata_size + metadata_signature_size); |
| 113 | payload.insert(payload.begin() + metadata_size, |
Tianjie | 55abd3c | 2020-06-19 00:22:59 -0700 | [diff] [blame] | 114 | metadata_signature.begin(), |
| 115 | metadata_signature.end()); |
| 116 | metadata_signature_size = metadata_signature.size(); |
Amin Hassani | 55c7541 | 2019-10-07 11:20:39 -0700 | [diff] [blame] | 117 | LOG(INFO) << "Metadata signature size: " << metadata_signature_size; |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 118 | |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 119 | DeltaArchiveManifest manifest; |
| 120 | TEST_AND_RETURN_FALSE(payload_metadata.GetManifest(payload, &manifest)); |
| 121 | |
Kelvin Zhang | fc81688 | 2022-11-17 15:28:51 -0800 | [diff] [blame^] | 122 | // Erase existing signatures. |
| 123 | if (manifest.has_signatures_offset()) { |
| 124 | payload.resize(manifest.signatures_offset() + metadata_size + |
| 125 | metadata_signature_size); |
Don Garrett | 2ae3787 | 2013-10-25 13:33:20 -0700 | [diff] [blame] | 126 | } |
Kelvin Zhang | fc81688 | 2022-11-17 15:28:51 -0800 | [diff] [blame^] | 127 | |
| 128 | // Updates the manifest to include the signature operation. |
| 129 | PayloadSigner::AddSignatureToManifest( |
| 130 | payload.size() - metadata_size - metadata_signature_size, |
| 131 | payload_signature.size(), |
| 132 | &manifest); |
| 133 | |
| 134 | // Updates the payload to include the new manifest. |
| 135 | string serialized_manifest; |
| 136 | TEST_AND_RETURN_FALSE(manifest.AppendToString(&serialized_manifest)); |
| 137 | LOG(INFO) << "Updated protobuf size: " << serialized_manifest.size(); |
| 138 | payload.erase(payload.begin() + manifest_offset, |
| 139 | payload.begin() + metadata_size); |
| 140 | payload.insert(payload.begin() + manifest_offset, |
| 141 | serialized_manifest.begin(), |
| 142 | serialized_manifest.end()); |
| 143 | |
| 144 | // Updates the protobuf size. |
| 145 | uint64_t size_be = htobe64(serialized_manifest.size()); |
| 146 | memcpy(&payload[kProtobufSizeOffset], &size_be, sizeof(size_be)); |
| 147 | metadata_size = serialized_manifest.size() + manifest_offset; |
| 148 | |
| 149 | LOG(INFO) << "Updated payload size: " << payload.size(); |
| 150 | LOG(INFO) << "Updated metadata size: " << metadata_size; |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 151 | uint64_t signatures_offset = |
| 152 | metadata_size + metadata_signature_size + manifest.signatures_offset(); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 153 | LOG(INFO) << "Signature Blob Offset: " << signatures_offset; |
| 154 | payload.resize(signatures_offset); |
| 155 | payload.insert(payload.begin() + signatures_offset, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 156 | payload_signature.begin(), |
| 157 | payload_signature.end()); |
Don Garrett | 2ae3787 | 2013-10-25 13:33:20 -0700 | [diff] [blame] | 158 | |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 159 | *out_payload = std::move(payload); |
Don Garrett | 2ae3787 | 2013-10-25 13:33:20 -0700 | [diff] [blame] | 160 | *out_metadata_size = metadata_size; |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 161 | *out_metadata_signature_size = metadata_signature_size; |
| 162 | *out_signatures_offset = signatures_offset; |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 163 | return true; |
| 164 | } |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 165 | |
| 166 | // Given a |payload| with correct signature op and metadata signature size in |
| 167 | // header and |metadata_size|, |metadata_signature_size|, |signatures_offset|, |
| 168 | // calculate hash for payload and metadata, save it to |out_hash_data| and |
| 169 | // |out_metadata_hash|. |
| 170 | bool CalculateHashFromPayload(const brillo::Blob& payload, |
| 171 | const uint64_t metadata_size, |
| 172 | const uint32_t metadata_signature_size, |
| 173 | const uint64_t signatures_offset, |
| 174 | brillo::Blob* out_hash_data, |
| 175 | brillo::Blob* out_metadata_hash) { |
| 176 | if (out_metadata_hash) { |
| 177 | // Calculates the hash on the manifest. |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 178 | TEST_AND_RETURN_FALSE(HashCalculator::RawHashOfBytes( |
| 179 | payload.data(), metadata_size, out_metadata_hash)); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 180 | } |
| 181 | if (out_hash_data) { |
| 182 | // Calculates the hash on the updated payload. Note that we skip metadata |
| 183 | // signature and payload signature. |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 184 | HashCalculator calc; |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 185 | TEST_AND_RETURN_FALSE(calc.Update(payload.data(), metadata_size)); |
Alex Deymo | 98e691c | 2016-02-04 21:05:45 -0800 | [diff] [blame] | 186 | TEST_AND_RETURN_FALSE(signatures_offset >= |
| 187 | metadata_size + metadata_signature_size); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 188 | TEST_AND_RETURN_FALSE(calc.Update( |
| 189 | payload.data() + metadata_size + metadata_signature_size, |
| 190 | signatures_offset - metadata_size - metadata_signature_size)); |
| 191 | TEST_AND_RETURN_FALSE(calc.Finalize()); |
| 192 | *out_hash_data = calc.raw_hash(); |
| 193 | } |
| 194 | return true; |
| 195 | } |
| 196 | |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 197 | std::unique_ptr<EVP_PKEY, decltype(&EVP_PKEY_free)> CreatePrivateKeyFromPath( |
| 198 | const string& private_key_path) { |
| 199 | FILE* fprikey = fopen(private_key_path.c_str(), "rb"); |
| 200 | if (!fprikey) { |
| 201 | PLOG(ERROR) << "Failed to read " << private_key_path; |
| 202 | return {nullptr, nullptr}; |
| 203 | } |
| 204 | |
| 205 | auto private_key = std::unique_ptr<EVP_PKEY, decltype(&EVP_PKEY_free)>( |
| 206 | PEM_read_PrivateKey(fprikey, nullptr, nullptr, nullptr), EVP_PKEY_free); |
| 207 | fclose(fprikey); |
| 208 | return private_key; |
| 209 | } |
| 210 | |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 211 | } // namespace |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 212 | |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 213 | bool PayloadSigner::GetMaximumSignatureSize(const string& private_key_path, |
| 214 | size_t* signature_size) { |
| 215 | *signature_size = 0; |
| 216 | auto private_key = CreatePrivateKeyFromPath(private_key_path); |
| 217 | if (!private_key) { |
| 218 | LOG(ERROR) << "Failed to create private key from " << private_key_path; |
| 219 | return false; |
| 220 | } |
| 221 | |
| 222 | *signature_size = EVP_PKEY_size(private_key.get()); |
| 223 | return true; |
| 224 | } |
| 225 | |
Sen Jiang | 3e728fe | 2015-11-05 11:37:23 -0800 | [diff] [blame] | 226 | void PayloadSigner::AddSignatureToManifest(uint64_t signature_blob_offset, |
| 227 | uint64_t signature_blob_length, |
Sen Jiang | 3e728fe | 2015-11-05 11:37:23 -0800 | [diff] [blame] | 228 | DeltaArchiveManifest* manifest) { |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 229 | LOG(INFO) << "Making room for signature in file"; |
| 230 | manifest->set_signatures_offset(signature_blob_offset); |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 231 | manifest->set_signatures_size(signature_blob_length); |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 232 | } |
| 233 | |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 234 | bool PayloadSigner::VerifySignedPayload(const string& payload_path, |
| 235 | const string& public_key_path) { |
Alex Deymo | 98e691c | 2016-02-04 21:05:45 -0800 | [diff] [blame] | 236 | brillo::Blob payload; |
| 237 | TEST_AND_RETURN_FALSE(utils::ReadFile(payload_path, &payload)); |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 238 | PayloadMetadata payload_metadata; |
Amin Hassani | 822d485 | 2020-03-10 01:50:42 +0000 | [diff] [blame] | 239 | TEST_AND_RETURN_FALSE(payload_metadata.ParsePayloadHeader(payload)); |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 240 | DeltaArchiveManifest manifest; |
| 241 | TEST_AND_RETURN_FALSE(payload_metadata.GetManifest(payload, &manifest)); |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 242 | TEST_AND_RETURN_FALSE(manifest.has_signatures_offset() && |
| 243 | manifest.has_signatures_size()); |
Sen Jiang | 44ac3ea | 2018-10-18 15:10:20 -0700 | [diff] [blame] | 244 | uint64_t metadata_size = payload_metadata.GetMetadataSize(); |
| 245 | uint32_t metadata_signature_size = |
| 246 | payload_metadata.GetMetadataSignatureSize(); |
| 247 | uint64_t signatures_offset = |
| 248 | metadata_size + metadata_signature_size + manifest.signatures_offset(); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 249 | CHECK_EQ(payload.size(), signatures_offset + manifest.signatures_size()); |
| 250 | brillo::Blob payload_hash, metadata_hash; |
| 251 | TEST_AND_RETURN_FALSE(CalculateHashFromPayload(payload, |
| 252 | metadata_size, |
| 253 | metadata_signature_size, |
| 254 | signatures_offset, |
| 255 | &payload_hash, |
| 256 | &metadata_hash)); |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 257 | string signature(payload.begin() + signatures_offset, payload.end()); |
Sen Jiang | 08c6da1 | 2019-01-07 18:28:56 -0800 | [diff] [blame] | 258 | string public_key; |
| 259 | TEST_AND_RETURN_FALSE(utils::ReadFile(public_key_path, &public_key)); |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 260 | TEST_AND_RETURN_FALSE(payload_hash.size() == kSHA256Size); |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 261 | |
| 262 | auto payload_verifier = PayloadVerifier::CreateInstance(public_key); |
| 263 | TEST_AND_RETURN_FALSE(payload_verifier != nullptr); |
| 264 | |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 265 | TEST_AND_RETURN_FALSE( |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 266 | payload_verifier->VerifySignature(signature, payload_hash)); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 267 | if (metadata_signature_size) { |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 268 | signature.assign(payload.begin() + metadata_size, |
| 269 | payload.begin() + metadata_size + metadata_signature_size); |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 270 | TEST_AND_RETURN_FALSE(metadata_hash.size() == kSHA256Size); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 271 | TEST_AND_RETURN_FALSE( |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 272 | payload_verifier->VerifySignature(signature, metadata_hash)); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 273 | } |
Sen Jiang | aef1c6f | 2015-10-07 10:05:32 -0700 | [diff] [blame] | 274 | return true; |
| 275 | } |
| 276 | |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 277 | bool PayloadSigner::SignHash(const brillo::Blob& hash, |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 278 | const string& private_key_path, |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 279 | brillo::Blob* out_signature) { |
Jay Srinivasan | 51dcf26 | 2012-09-13 17:24:32 -0700 | [diff] [blame] | 280 | LOG(INFO) << "Signing hash with private key: " << private_key_path; |
Andrew de los Reyes | bdfaaf0 | 2011-03-30 10:35:12 -0700 | [diff] [blame] | 281 | // We expect unpadded SHA256 hash coming in |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 282 | TEST_AND_RETURN_FALSE(hash.size() == kSHA256Size); |
Sen Jiang | 923886a | 2016-03-14 15:04:28 -0700 | [diff] [blame] | 283 | // The code below executes the equivalent of: |
| 284 | // |
| 285 | // openssl rsautl -raw -sign -inkey |private_key_path| |
| 286 | // -in |padded_hash| -out |out_signature| |
Darin Petkov | d22cb29 | 2010-09-29 10:02:29 -0700 | [diff] [blame] | 287 | |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 288 | auto private_key = CreatePrivateKeyFromPath(private_key_path); |
| 289 | if (!private_key) { |
| 290 | LOG(ERROR) << "Failed to create private key from " << private_key_path; |
Sen Jiang | 923886a | 2016-03-14 15:04:28 -0700 | [diff] [blame] | 291 | return false; |
| 292 | } |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 293 | |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 294 | int key_type = EVP_PKEY_id(private_key.get()); |
| 295 | brillo::Blob signature; |
| 296 | if (key_type == EVP_PKEY_RSA) { |
Amin Hassani | 7d19220 | 2020-09-29 15:58:37 -0700 | [diff] [blame] | 297 | RSA* rsa = EVP_PKEY_get0_RSA(private_key.get()); |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 298 | TEST_AND_RETURN_FALSE(rsa != nullptr); |
xunchang | cda3c03 | 2019-03-26 15:41:14 -0700 | [diff] [blame] | 299 | |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 300 | brillo::Blob padded_hash = hash; |
| 301 | PayloadVerifier::PadRSASHA256Hash(&padded_hash, RSA_size(rsa)); |
| 302 | |
| 303 | signature.resize(RSA_size(rsa)); |
| 304 | ssize_t signature_size = RSA_private_encrypt(padded_hash.size(), |
| 305 | padded_hash.data(), |
| 306 | signature.data(), |
| 307 | rsa, |
| 308 | RSA_NO_PADDING); |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 309 | if (signature_size < 0) { |
| 310 | LOG(ERROR) << "Signing hash failed: " |
| 311 | << ERR_error_string(ERR_get_error(), nullptr); |
| 312 | return false; |
| 313 | } |
| 314 | TEST_AND_RETURN_FALSE(static_cast<size_t>(signature_size) == |
| 315 | signature.size()); |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 316 | } else if (key_type == EVP_PKEY_EC) { |
Amin Hassani | 7d19220 | 2020-09-29 15:58:37 -0700 | [diff] [blame] | 317 | EC_KEY* ec_key = EVP_PKEY_get0_EC_KEY(private_key.get()); |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 318 | TEST_AND_RETURN_FALSE(ec_key != nullptr); |
| 319 | |
| 320 | signature.resize(ECDSA_size(ec_key)); |
| 321 | unsigned int signature_size; |
| 322 | if (ECDSA_sign(0, |
| 323 | hash.data(), |
| 324 | hash.size(), |
| 325 | signature.data(), |
| 326 | &signature_size, |
| 327 | ec_key) != 1) { |
| 328 | LOG(ERROR) << "Signing hash failed: " |
| 329 | << ERR_error_string(ERR_get_error(), nullptr); |
| 330 | return false; |
| 331 | } |
| 332 | |
| 333 | // NIST P-256 |
| 334 | LOG(ERROR) << "signature max size " << signature.size() << " size " |
| 335 | << signature_size; |
| 336 | TEST_AND_RETURN_FALSE(signature.size() >= signature_size); |
| 337 | signature.resize(signature_size); |
Tianjie Xu | 6cf830b | 2019-09-30 11:31:49 -0700 | [diff] [blame] | 338 | } else { |
| 339 | LOG(ERROR) << "key_type " << key_type << " isn't supported for signing"; |
Sen Jiang | 923886a | 2016-03-14 15:04:28 -0700 | [diff] [blame] | 340 | return false; |
| 341 | } |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 342 | out_signature->swap(signature); |
| 343 | return true; |
| 344 | } |
Darin Petkov | d22cb29 | 2010-09-29 10:02:29 -0700 | [diff] [blame] | 345 | |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 346 | bool PayloadSigner::SignHashWithKeys(const brillo::Blob& hash_data, |
| 347 | const vector<string>& private_key_paths, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 348 | string* out_serialized_signature) { |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 349 | vector<brillo::Blob> signatures; |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 350 | vector<size_t> padded_signature_sizes; |
Alex Deymo | 020600d | 2014-11-05 21:05:55 -0800 | [diff] [blame] | 351 | for (const string& path : private_key_paths) { |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 352 | brillo::Blob signature; |
Alex Deymo | 020600d | 2014-11-05 21:05:55 -0800 | [diff] [blame] | 353 | TEST_AND_RETURN_FALSE(SignHash(hash_data, path, &signature)); |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 354 | signatures.push_back(signature); |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 355 | |
| 356 | size_t padded_signature_size; |
| 357 | TEST_AND_RETURN_FALSE( |
| 358 | GetMaximumSignatureSize(path, &padded_signature_size)); |
| 359 | padded_signature_sizes.push_back(padded_signature_size); |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 360 | } |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 361 | TEST_AND_RETURN_FALSE(ConvertSignaturesToProtobuf( |
| 362 | signatures, padded_signature_sizes, out_serialized_signature)); |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 363 | return true; |
| 364 | } |
| 365 | |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 366 | bool PayloadSigner::SignPayload(const string& unsigned_payload_path, |
| 367 | const vector<string>& private_key_paths, |
| 368 | const uint64_t metadata_size, |
| 369 | const uint32_t metadata_signature_size, |
| 370 | const uint64_t signatures_offset, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 371 | string* out_serialized_signature) { |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 372 | brillo::Blob payload; |
| 373 | TEST_AND_RETURN_FALSE(utils::ReadFile(unsigned_payload_path, &payload)); |
| 374 | brillo::Blob hash_data; |
| 375 | TEST_AND_RETURN_FALSE(CalculateHashFromPayload(payload, |
| 376 | metadata_size, |
| 377 | metadata_signature_size, |
| 378 | signatures_offset, |
| 379 | &hash_data, |
| 380 | nullptr)); |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 381 | TEST_AND_RETURN_FALSE( |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 382 | SignHashWithKeys(hash_data, private_key_paths, out_serialized_signature)); |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 383 | return true; |
| 384 | } |
| 385 | |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 386 | bool PayloadSigner::SignatureBlobLength(const vector<string>& private_key_paths, |
| 387 | uint64_t* out_length) { |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 388 | DCHECK(out_length); |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 389 | brillo::Blob hash_blob; |
| 390 | TEST_AND_RETURN_FALSE(HashCalculator::RawHashOfData({'x'}, &hash_blob)); |
| 391 | string sig_blob; |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 392 | TEST_AND_RETURN_FALSE( |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 393 | SignHashWithKeys(hash_blob, private_key_paths, &sig_blob)); |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 394 | *out_length = sig_blob.size(); |
| 395 | return true; |
| 396 | } |
| 397 | |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 398 | bool PayloadSigner::HashPayloadForSigning(const string& payload_path, |
Tianjie Xu | dac04b7 | 2019-09-30 11:48:17 -0700 | [diff] [blame] | 399 | const vector<size_t>& signature_sizes, |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 400 | brillo::Blob* out_payload_hash_data, |
| 401 | brillo::Blob* out_metadata_hash) { |
| 402 | // Create a signature blob with signatures filled with 0. |
| 403 | // Will be used for both payload signature and metadata signature. |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 404 | vector<brillo::Blob> signatures; |
Alex Deymo | 020600d | 2014-11-05 21:05:55 -0800 | [diff] [blame] | 405 | for (int signature_size : signature_sizes) { |
| 406 | signatures.emplace_back(signature_size, 0); |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 407 | } |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 408 | string signature; |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 409 | TEST_AND_RETURN_FALSE( |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 410 | ConvertSignaturesToProtobuf(signatures, signature_sizes, &signature)); |
Don Garrett | 2ae3787 | 2013-10-25 13:33:20 -0700 | [diff] [blame] | 411 | |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 412 | brillo::Blob payload; |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 413 | uint64_t metadata_size, signatures_offset; |
| 414 | uint32_t metadata_signature_size; |
| 415 | // Prepare payload for hashing. |
| 416 | TEST_AND_RETURN_FALSE(AddSignatureBlobToPayload(payload_path, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 417 | signature, |
| 418 | signature, |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 419 | &payload, |
| 420 | &metadata_size, |
| 421 | &metadata_signature_size, |
| 422 | &signatures_offset)); |
| 423 | TEST_AND_RETURN_FALSE(CalculateHashFromPayload(payload, |
| 424 | metadata_size, |
| 425 | metadata_signature_size, |
| 426 | signatures_offset, |
| 427 | out_payload_hash_data, |
| 428 | out_metadata_hash)); |
Jay Srinivasan | f431870 | 2012-09-24 11:56:24 -0700 | [diff] [blame] | 429 | return true; |
| 430 | } |
| 431 | |
Andrew de los Reyes | c24e3f3 | 2011-08-30 15:45:20 -0700 | [diff] [blame] | 432 | bool PayloadSigner::AddSignatureToPayload( |
| 433 | const string& payload_path, |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 434 | const vector<size_t>& padded_signature_sizes, |
Sen Jiang | 644f618 | 2015-10-06 16:45:57 -0700 | [diff] [blame] | 435 | const vector<brillo::Blob>& payload_signatures, |
| 436 | const vector<brillo::Blob>& metadata_signatures, |
Jay Srinivasan | 738fdf3 | 2012-12-07 17:40:54 -0800 | [diff] [blame] | 437 | const string& signed_payload_path, |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 438 | uint64_t* out_metadata_size) { |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 439 | // TODO(petkov): Reduce memory usage -- the payload is manipulated in memory. |
| 440 | |
| 441 | // Loads the payload and adds the signature op to it. |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 442 | string payload_signature, metadata_signature; |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 443 | TEST_AND_RETURN_FALSE(ConvertSignaturesToProtobuf( |
| 444 | payload_signatures, padded_signature_sizes, &payload_signature)); |
Sen Jiang | 644f618 | 2015-10-06 16:45:57 -0700 | [diff] [blame] | 445 | if (!metadata_signatures.empty()) { |
Tianjie Xu | 7bbe015 | 2019-10-09 18:11:15 -0700 | [diff] [blame] | 446 | TEST_AND_RETURN_FALSE(ConvertSignaturesToProtobuf( |
| 447 | metadata_signatures, padded_signature_sizes, &metadata_signature)); |
Sen Jiang | 644f618 | 2015-10-06 16:45:57 -0700 | [diff] [blame] | 448 | } |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 449 | brillo::Blob payload; |
Don Garrett | 2ae3787 | 2013-10-25 13:33:20 -0700 | [diff] [blame] | 450 | uint64_t signatures_offset; |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 451 | uint32_t metadata_signature_size; |
| 452 | TEST_AND_RETURN_FALSE(AddSignatureBlobToPayload(payload_path, |
Sen Jiang | 9b2f178 | 2019-01-24 14:27:50 -0800 | [diff] [blame] | 453 | payload_signature, |
| 454 | metadata_signature, |
Sen Jiang | 720df3e | 2015-10-01 13:10:44 -0700 | [diff] [blame] | 455 | &payload, |
| 456 | out_metadata_size, |
| 457 | &metadata_signature_size, |
| 458 | &signatures_offset)); |
| 459 | |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 460 | LOG(INFO) << "Signed payload size: " << payload.size(); |
Kelvin Zhang | fc81688 | 2022-11-17 15:28:51 -0800 | [diff] [blame^] | 461 | const auto ret = |
| 462 | HANDLE_EINTR(truncate(signed_payload_path.c_str(), payload.size())); |
| 463 | if (ret < 0) { |
| 464 | PLOG(ERROR) << "Failed to truncate file " << signed_payload_path |
| 465 | << " to size " << payload.size(); |
| 466 | return false; |
| 467 | } |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 468 | TEST_AND_RETURN_FALSE(utils::WriteFile( |
| 469 | signed_payload_path.c_str(), payload.data(), payload.size())); |
Darin Petkov | 9574f7e | 2011-01-13 10:48:12 -0800 | [diff] [blame] | 470 | return true; |
| 471 | } |
| 472 | |
Alex Vakulenko | f68bbbc | 2015-02-09 12:53:18 -0800 | [diff] [blame] | 473 | bool PayloadSigner::GetMetadataSignature(const void* const metadata, |
Jay Srinivasan | f431870 | 2012-09-24 11:56:24 -0700 | [diff] [blame] | 474 | size_t metadata_size, |
Jay Srinivasan | 51dcf26 | 2012-09-13 17:24:32 -0700 | [diff] [blame] | 475 | const string& private_key_path, |
| 476 | string* out_signature) { |
| 477 | // Calculates the hash on the updated payload. Note that the payload includes |
| 478 | // the signature op but doesn't include the signature blob at the end. |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 479 | brillo::Blob metadata_hash; |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 480 | TEST_AND_RETURN_FALSE( |
| 481 | HashCalculator::RawHashOfBytes(metadata, metadata_size, &metadata_hash)); |
Jay Srinivasan | 51dcf26 | 2012-09-13 17:24:32 -0700 | [diff] [blame] | 482 | |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 483 | brillo::Blob signature; |
Amin Hassani | 232f8f9 | 2019-01-14 16:15:31 -0800 | [diff] [blame] | 484 | TEST_AND_RETURN_FALSE(SignHash(metadata_hash, private_key_path, &signature)); |
Jay Srinivasan | 51dcf26 | 2012-09-13 17:24:32 -0700 | [diff] [blame] | 485 | |
Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 486 | *out_signature = brillo::data_encoding::Base64Encode(signature); |
Jay Srinivasan | 51dcf26 | 2012-09-13 17:24:32 -0700 | [diff] [blame] | 487 | return true; |
| 488 | } |
| 489 | |
Andrew de los Reyes | 0c44005 | 2010-08-20 11:25:54 -0700 | [diff] [blame] | 490 | } // namespace chromeos_update_engine |