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