The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2008 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 | */ |
| 16 | |
Tao Bao | ac9d94d | 2016-11-03 11:37:15 -0700 | [diff] [blame] | 17 | #include "verifier.h" |
| 18 | |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 19 | #include <errno.h> |
Elliott Hughes | 26dbad2 | 2015-01-28 12:09:05 -0800 | [diff] [blame] | 20 | #include <stdio.h> |
Tao Bao | ac9d94d | 2016-11-03 11:37:15 -0700 | [diff] [blame] | 21 | #include <stdlib.h> |
Elliott Hughes | 26dbad2 | 2015-01-28 12:09:05 -0800 | [diff] [blame] | 22 | #include <string.h> |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 23 | |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 24 | #include <algorithm> |
Tao Bao | d7bf82e | 2017-03-18 09:24:11 -0700 | [diff] [blame] | 25 | #include <functional> |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 26 | #include <memory> |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 27 | #include <vector> |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 28 | |
Tianjie Xu | 7b0ad9c | 2016-08-05 18:00:04 -0700 | [diff] [blame] | 29 | #include <android-base/logging.h> |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 30 | #include <openssl/bio.h> |
David Benjamin | a86392e | 2016-04-15 20:22:09 -0400 | [diff] [blame] | 31 | #include <openssl/bn.h> |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 32 | #include <openssl/ecdsa.h> |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 33 | #include <openssl/evp.h> |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 34 | #include <openssl/obj_mac.h> |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 35 | #include <openssl/pem.h> |
| 36 | #include <openssl/rsa.h> |
Tianjie Xu | 0dd9685 | 2018-10-15 11:44:14 -0700 | [diff] [blame] | 37 | #include <ziparchive/zip_archive.h> |
Elliott Hughes | 8febafa | 2016-04-13 16:39:56 -0700 | [diff] [blame] | 38 | |
| 39 | #include "asn1_decoder.h" |
Tao Bao | 09e468f | 2017-09-29 14:39:33 -0700 | [diff] [blame] | 40 | #include "otautil/print_sha1.h" |
Doug Zongker | 211aebc | 2011-10-28 15:13:10 -0700 | [diff] [blame] | 41 | |
Kenny Root | 7a4adb5 | 2013-10-09 10:14:35 -0700 | [diff] [blame] | 42 | /* |
| 43 | * Simple version of PKCS#7 SignedData extraction. This extracts the |
| 44 | * signature OCTET STRING to be used for signature verification. |
| 45 | * |
| 46 | * For full details, see http://www.ietf.org/rfc/rfc3852.txt |
| 47 | * |
| 48 | * The PKCS#7 structure looks like: |
| 49 | * |
| 50 | * SEQUENCE (ContentInfo) |
| 51 | * OID (ContentType) |
| 52 | * [0] (content) |
| 53 | * SEQUENCE (SignedData) |
| 54 | * INTEGER (version CMSVersion) |
| 55 | * SET (DigestAlgorithmIdentifiers) |
| 56 | * SEQUENCE (EncapsulatedContentInfo) |
| 57 | * [0] (CertificateSet OPTIONAL) |
| 58 | * [1] (RevocationInfoChoices OPTIONAL) |
| 59 | * SET (SignerInfos) |
| 60 | * SEQUENCE (SignerInfo) |
| 61 | * INTEGER (CMSVersion) |
| 62 | * SEQUENCE (SignerIdentifier) |
| 63 | * SEQUENCE (DigestAlgorithmIdentifier) |
| 64 | * SEQUENCE (SignatureAlgorithmIdentifier) |
| 65 | * OCTET STRING (SignatureValue) |
| 66 | */ |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 67 | static bool read_pkcs7(const uint8_t* pkcs7_der, size_t pkcs7_der_len, |
| 68 | std::vector<uint8_t>* sig_der) { |
| 69 | CHECK(sig_der != nullptr); |
| 70 | sig_der->clear(); |
Kenny Root | 7a4adb5 | 2013-10-09 10:14:35 -0700 | [diff] [blame] | 71 | |
Tao Bao | 861c53c | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 72 | asn1_context ctx(pkcs7_der, pkcs7_der_len); |
| 73 | |
| 74 | std::unique_ptr<asn1_context> pkcs7_seq(ctx.asn1_sequence_get()); |
| 75 | if (pkcs7_seq == nullptr || !pkcs7_seq->asn1_sequence_next()) { |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 76 | return false; |
| 77 | } |
| 78 | |
Tao Bao | 861c53c | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 79 | std::unique_ptr<asn1_context> signed_data_app(pkcs7_seq->asn1_constructed_get()); |
| 80 | if (signed_data_app == nullptr) { |
| 81 | return false; |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 82 | } |
Kenny Root | 7a4adb5 | 2013-10-09 10:14:35 -0700 | [diff] [blame] | 83 | |
Tao Bao | 861c53c | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 84 | std::unique_ptr<asn1_context> signed_data_seq(signed_data_app->asn1_sequence_get()); |
| 85 | if (signed_data_seq == nullptr || |
| 86 | !signed_data_seq->asn1_sequence_next() || |
| 87 | !signed_data_seq->asn1_sequence_next() || |
| 88 | !signed_data_seq->asn1_sequence_next() || |
| 89 | !signed_data_seq->asn1_constructed_skip_all()) { |
| 90 | return false; |
| 91 | } |
| 92 | |
| 93 | std::unique_ptr<asn1_context> sig_set(signed_data_seq->asn1_set_get()); |
| 94 | if (sig_set == nullptr) { |
| 95 | return false; |
| 96 | } |
| 97 | |
| 98 | std::unique_ptr<asn1_context> sig_seq(sig_set->asn1_sequence_get()); |
| 99 | if (sig_seq == nullptr || |
| 100 | !sig_seq->asn1_sequence_next() || |
| 101 | !sig_seq->asn1_sequence_next() || |
| 102 | !sig_seq->asn1_sequence_next() || |
| 103 | !sig_seq->asn1_sequence_next()) { |
| 104 | return false; |
| 105 | } |
| 106 | |
| 107 | const uint8_t* sig_der_ptr; |
| 108 | size_t sig_der_length; |
| 109 | if (!sig_seq->asn1_octet_string_get(&sig_der_ptr, &sig_der_length)) { |
| 110 | return false; |
| 111 | } |
| 112 | |
| 113 | sig_der->resize(sig_der_length); |
| 114 | std::copy(sig_der_ptr, sig_der_ptr + sig_der_length, sig_der->begin()); |
| 115 | return true; |
Kenny Root | 7a4adb5 | 2013-10-09 10:14:35 -0700 | [diff] [blame] | 116 | } |
| 117 | |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 118 | int verify_file(VerifierInterface* package, const std::vector<Certificate>& keys) { |
| 119 | CHECK(package); |
| 120 | package->SetProgress(0.0); |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 121 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 122 | // An archive with a whole-file signature will end in six bytes: |
| 123 | // |
| 124 | // (2-byte signature start) $ff $ff (2-byte comment size) |
| 125 | // |
| 126 | // (As far as the ZIP format is concerned, these are part of the archive comment.) We start by |
| 127 | // reading this footer, this tells us how far back from the end we have to start reading to find |
| 128 | // the whole comment. |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 129 | |
| 130 | #define FOOTER_SIZE 6 |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 131 | uint64_t length = package->GetPackageSize(); |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 132 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 133 | if (length < FOOTER_SIZE) { |
| 134 | LOG(ERROR) << "not big enough to contain footer"; |
| 135 | return VERIFY_FAILURE; |
| 136 | } |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 137 | |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 138 | uint8_t footer[FOOTER_SIZE]; |
| 139 | if (!package->ReadFullyAtOffset(footer, FOOTER_SIZE, length - FOOTER_SIZE)) { |
| 140 | LOG(ERROR) << "Failed to read footer"; |
| 141 | return VERIFY_FAILURE; |
| 142 | } |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 143 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 144 | if (footer[2] != 0xff || footer[3] != 0xff) { |
| 145 | LOG(ERROR) << "footer is wrong"; |
| 146 | return VERIFY_FAILURE; |
| 147 | } |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 148 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 149 | size_t comment_size = footer[4] + (footer[5] << 8); |
| 150 | size_t signature_start = footer[0] + (footer[1] << 8); |
| 151 | LOG(INFO) << "comment is " << comment_size << " bytes; signature is " << signature_start |
| 152 | << " bytes from end"; |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 153 | |
Tao Bao | 553c7bd | 2017-03-18 07:33:26 -0700 | [diff] [blame] | 154 | if (signature_start > comment_size) { |
| 155 | LOG(ERROR) << "signature start: " << signature_start << " is larger than comment size: " |
| 156 | << comment_size; |
| 157 | return VERIFY_FAILURE; |
| 158 | } |
Tianjie Xu | 54ea136 | 2016-12-16 16:24:09 -0800 | [diff] [blame] | 159 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 160 | if (signature_start <= FOOTER_SIZE) { |
| 161 | LOG(ERROR) << "Signature start is in the footer"; |
| 162 | return VERIFY_FAILURE; |
| 163 | } |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 164 | |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 165 | #define EOCD_HEADER_SIZE 22 |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 166 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 167 | // The end-of-central-directory record is 22 bytes plus any comment length. |
| 168 | size_t eocd_size = comment_size + EOCD_HEADER_SIZE; |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 169 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 170 | if (length < eocd_size) { |
| 171 | LOG(ERROR) << "not big enough to contain EOCD"; |
Doug Zongker | 54e2e86 | 2009-08-17 13:21:04 -0700 | [diff] [blame] | 172 | return VERIFY_FAILURE; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 173 | } |
| 174 | |
| 175 | // Determine how much of the file is covered by the signature. This is everything except the |
| 176 | // signature data and length, which includes all of the EOCD except for the comment length field |
| 177 | // (2 bytes) and the comment data. |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 178 | uint64_t signed_len = length - eocd_size + EOCD_HEADER_SIZE - 2; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 179 | |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 180 | uint8_t eocd[eocd_size]; |
| 181 | if (!package->ReadFullyAtOffset(eocd, eocd_size, length - eocd_size)) { |
| 182 | LOG(ERROR) << "Failed to read EOCD of " << eocd_size << " bytes"; |
| 183 | return VERIFY_FAILURE; |
| 184 | } |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 185 | |
| 186 | // If this is really is the EOCD record, it will begin with the magic number $50 $4b $05 $06. |
| 187 | if (eocd[0] != 0x50 || eocd[1] != 0x4b || eocd[2] != 0x05 || eocd[3] != 0x06) { |
| 188 | LOG(ERROR) << "signature length doesn't match EOCD marker"; |
| 189 | return VERIFY_FAILURE; |
| 190 | } |
| 191 | |
| 192 | for (size_t i = 4; i < eocd_size-3; ++i) { |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 193 | if (eocd[i] == 0x50 && eocd[i+1] == 0x4b && eocd[i+2] == 0x05 && eocd[i+3] == 0x06) { |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 194 | // If the sequence $50 $4b $05 $06 appears anywhere after the real one, libziparchive will |
| 195 | // find the later (wrong) one, which could be exploitable. Fail the verification if this |
| 196 | // sequence occurs anywhere after the real one. |
| 197 | LOG(ERROR) << "EOCD marker occurs after start of EOCD"; |
| 198 | return VERIFY_FAILURE; |
| 199 | } |
| 200 | } |
| 201 | |
| 202 | bool need_sha1 = false; |
| 203 | bool need_sha256 = false; |
| 204 | for (const auto& key : keys) { |
| 205 | switch (key.hash_len) { |
| 206 | case SHA_DIGEST_LENGTH: need_sha1 = true; break; |
| 207 | case SHA256_DIGEST_LENGTH: need_sha256 = true; break; |
| 208 | } |
| 209 | } |
| 210 | |
| 211 | SHA_CTX sha1_ctx; |
| 212 | SHA256_CTX sha256_ctx; |
| 213 | SHA1_Init(&sha1_ctx); |
| 214 | SHA256_Init(&sha256_ctx); |
| 215 | |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 216 | std::vector<HasherUpdateCallback> hashers; |
| 217 | if (need_sha1) { |
| 218 | hashers.emplace_back( |
| 219 | std::bind(&SHA1_Update, &sha1_ctx, std::placeholders::_1, std::placeholders::_2)); |
| 220 | } |
| 221 | if (need_sha256) { |
| 222 | hashers.emplace_back( |
| 223 | std::bind(&SHA256_Update, &sha256_ctx, std::placeholders::_1, std::placeholders::_2)); |
| 224 | } |
| 225 | |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 226 | double frac = -1.0; |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 227 | uint64_t so_far = 0; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 228 | while (so_far < signed_len) { |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 229 | // On a Nexus 5X, experiment showed 16MiB beat 1MiB by 6% faster for a 1196MiB full OTA and |
| 230 | // 60% for an 89MiB incremental OTA. http://b/28135231. |
| 231 | uint64_t read_size = std::min<uint64_t>(signed_len - so_far, 16 * MiB); |
| 232 | package->UpdateHashAtOffset(hashers, so_far, read_size); |
| 233 | so_far += read_size; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 234 | |
xunchang | f07ed2e | 2019-02-25 14:14:01 -0800 | [diff] [blame] | 235 | double f = so_far / static_cast<double>(signed_len); |
| 236 | if (f > frac + 0.02 || read_size == so_far) { |
| 237 | package->SetProgress(f); |
| 238 | frac = f; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 239 | } |
| 240 | } |
| 241 | |
| 242 | uint8_t sha1[SHA_DIGEST_LENGTH]; |
| 243 | SHA1_Final(sha1, &sha1_ctx); |
| 244 | uint8_t sha256[SHA256_DIGEST_LENGTH]; |
| 245 | SHA256_Final(sha256, &sha256_ctx); |
| 246 | |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 247 | const uint8_t* signature = eocd + eocd_size - signature_start; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 248 | size_t signature_size = signature_start - FOOTER_SIZE; |
| 249 | |
| 250 | LOG(INFO) << "signature (offset: " << std::hex << (length - signature_start) << ", length: " |
| 251 | << signature_size << "): " << print_hex(signature, signature_size); |
| 252 | |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 253 | std::vector<uint8_t> sig_der; |
| 254 | if (!read_pkcs7(signature, signature_size, &sig_der)) { |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 255 | LOG(ERROR) << "Could not find signature DER block"; |
| 256 | return VERIFY_FAILURE; |
| 257 | } |
| 258 | |
| 259 | // Check to make sure at least one of the keys matches the signature. Since any key can match, |
| 260 | // we need to try each before determining a verification failure has happened. |
| 261 | size_t i = 0; |
| 262 | for (const auto& key : keys) { |
| 263 | const uint8_t* hash; |
| 264 | int hash_nid; |
| 265 | switch (key.hash_len) { |
| 266 | case SHA_DIGEST_LENGTH: |
| 267 | hash = sha1; |
| 268 | hash_nid = NID_sha1; |
| 269 | break; |
| 270 | case SHA256_DIGEST_LENGTH: |
| 271 | hash = sha256; |
| 272 | hash_nid = NID_sha256; |
| 273 | break; |
| 274 | default: |
| 275 | continue; |
| 276 | } |
| 277 | |
| 278 | // The 6 bytes is the "(signature_start) $ff $ff (comment_size)" that the signing tool appends |
| 279 | // after the signature itself. |
| 280 | if (key.key_type == Certificate::KEY_TYPE_RSA) { |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 281 | if (!RSA_verify(hash_nid, hash, key.hash_len, sig_der.data(), sig_der.size(), |
| 282 | key.rsa.get())) { |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 283 | LOG(INFO) << "failed to verify against RSA key " << i; |
| 284 | continue; |
| 285 | } |
| 286 | |
| 287 | LOG(INFO) << "whole-file signature verified against RSA key " << i; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 288 | return VERIFY_SUCCESS; |
| 289 | } else if (key.key_type == Certificate::KEY_TYPE_EC && key.hash_len == SHA256_DIGEST_LENGTH) { |
Tao Bao | 76fdb24 | 2017-03-20 17:09:13 -0700 | [diff] [blame] | 290 | if (!ECDSA_verify(0, hash, key.hash_len, sig_der.data(), sig_der.size(), key.ec.get())) { |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 291 | LOG(INFO) << "failed to verify against EC key " << i; |
| 292 | continue; |
| 293 | } |
| 294 | |
| 295 | LOG(INFO) << "whole-file signature verified against EC key " << i; |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 296 | return VERIFY_SUCCESS; |
| 297 | } else { |
| 298 | LOG(INFO) << "Unknown key type " << key.key_type; |
| 299 | } |
| 300 | i++; |
| 301 | } |
| 302 | |
| 303 | if (need_sha1) { |
| 304 | LOG(INFO) << "SHA-1 digest: " << print_hex(sha1, SHA_DIGEST_LENGTH); |
| 305 | } |
| 306 | if (need_sha256) { |
| 307 | LOG(INFO) << "SHA-256 digest: " << print_hex(sha256, SHA256_DIGEST_LENGTH); |
| 308 | } |
Tao Bao | 5e53501 | 2017-03-16 17:37:38 -0700 | [diff] [blame] | 309 | LOG(ERROR) << "failed to verify whole-file signature"; |
| 310 | return VERIFY_FAILURE; |
The Android Open Source Project | c24a8e6 | 2009-03-03 19:28:42 -0800 | [diff] [blame] | 311 | } |
Doug Zongker | 6c249f7 | 2012-11-02 15:04:05 -0700 | [diff] [blame] | 312 | |
Tianjie Xu | 0dd9685 | 2018-10-15 11:44:14 -0700 | [diff] [blame] | 313 | static std::vector<Certificate> IterateZipEntriesAndSearchForKeys(const ZipArchiveHandle& handle) { |
| 314 | void* cookie; |
| 315 | ZipString suffix("x509.pem"); |
| 316 | int32_t iter_status = StartIteration(handle, &cookie, nullptr, &suffix); |
| 317 | if (iter_status != 0) { |
| 318 | LOG(ERROR) << "Failed to iterate over entries in the certificate zipfile: " |
| 319 | << ErrorCodeString(iter_status); |
| 320 | return {}; |
| 321 | } |
| 322 | |
| 323 | std::vector<Certificate> result; |
| 324 | |
| 325 | ZipString name; |
| 326 | ZipEntry entry; |
| 327 | while ((iter_status = Next(cookie, &entry, &name)) == 0) { |
| 328 | std::vector<uint8_t> pem_content(entry.uncompressed_length); |
| 329 | if (int32_t extract_status = |
| 330 | ExtractToMemory(handle, &entry, pem_content.data(), pem_content.size()); |
| 331 | extract_status != 0) { |
| 332 | LOG(ERROR) << "Failed to extract " << std::string(name.name, name.name + name.name_length); |
| 333 | return {}; |
| 334 | } |
| 335 | |
| 336 | Certificate cert(0, Certificate::KEY_TYPE_RSA, nullptr, nullptr); |
| 337 | // Aborts the parsing if we fail to load one of the key file. |
| 338 | if (!LoadCertificateFromBuffer(pem_content, &cert)) { |
| 339 | LOG(ERROR) << "Failed to load keys from " |
| 340 | << std::string(name.name, name.name + name.name_length); |
| 341 | return {}; |
| 342 | } |
| 343 | |
| 344 | result.emplace_back(std::move(cert)); |
| 345 | } |
| 346 | |
| 347 | if (iter_status != -1) { |
| 348 | LOG(ERROR) << "Error while iterating over zip entries: " << ErrorCodeString(iter_status); |
| 349 | return {}; |
| 350 | } |
| 351 | |
| 352 | return result; |
| 353 | } |
| 354 | |
| 355 | std::vector<Certificate> LoadKeysFromZipfile(const std::string& zip_name) { |
| 356 | ZipArchiveHandle handle; |
| 357 | if (int32_t open_status = OpenArchive(zip_name.c_str(), &handle); open_status != 0) { |
| 358 | LOG(ERROR) << "Failed to open " << zip_name << ": " << ErrorCodeString(open_status); |
| 359 | return {}; |
| 360 | } |
| 361 | |
| 362 | std::vector<Certificate> result = IterateZipEntriesAndSearchForKeys(handle); |
| 363 | CloseArchive(handle); |
| 364 | return result; |
| 365 | } |
| 366 | |
Tianjie Xu | b5110de | 2018-10-23 23:31:43 -0700 | [diff] [blame] | 367 | bool CheckRSAKey(const std::unique_ptr<RSA, RSADeleter>& rsa) { |
| 368 | if (!rsa) { |
| 369 | return false; |
| 370 | } |
| 371 | |
| 372 | const BIGNUM* out_n; |
| 373 | const BIGNUM* out_e; |
| 374 | RSA_get0_key(rsa.get(), &out_n, &out_e, nullptr /* private exponent */); |
| 375 | auto modulus_bits = BN_num_bits(out_n); |
| 376 | if (modulus_bits != 2048) { |
| 377 | LOG(ERROR) << "Modulus should be 2048 bits long, actual: " << modulus_bits; |
| 378 | return false; |
| 379 | } |
| 380 | |
| 381 | BN_ULONG exponent = BN_get_word(out_e); |
| 382 | if (exponent != 3 && exponent != 65537) { |
| 383 | LOG(ERROR) << "Public exponent should be 3 or 65537, actual: " << exponent; |
| 384 | return false; |
| 385 | } |
| 386 | |
| 387 | return true; |
| 388 | } |
| 389 | |
| 390 | bool CheckECKey(const std::unique_ptr<EC_KEY, ECKEYDeleter>& ec_key) { |
| 391 | if (!ec_key) { |
| 392 | return false; |
| 393 | } |
| 394 | |
| 395 | const EC_GROUP* ec_group = EC_KEY_get0_group(ec_key.get()); |
| 396 | if (!ec_group) { |
| 397 | LOG(ERROR) << "Failed to get the ec_group from the ec_key"; |
| 398 | return false; |
| 399 | } |
| 400 | auto degree = EC_GROUP_get_degree(ec_group); |
| 401 | if (degree != 256) { |
| 402 | LOG(ERROR) << "Field size of the ec key should be 256 bits long, actual: " << degree; |
| 403 | return false; |
| 404 | } |
| 405 | |
| 406 | return true; |
| 407 | } |
| 408 | |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 409 | bool LoadCertificateFromBuffer(const std::vector<uint8_t>& pem_content, Certificate* cert) { |
| 410 | std::unique_ptr<BIO, decltype(&BIO_free)> content( |
| 411 | BIO_new_mem_buf(pem_content.data(), pem_content.size()), BIO_free); |
| 412 | |
| 413 | std::unique_ptr<X509, decltype(&X509_free)> x509( |
| 414 | PEM_read_bio_X509(content.get(), nullptr, nullptr, nullptr), X509_free); |
| 415 | if (!x509) { |
| 416 | LOG(ERROR) << "Failed to read x509 certificate"; |
| 417 | return false; |
| 418 | } |
| 419 | |
| 420 | int nid = X509_get_signature_nid(x509.get()); |
| 421 | switch (nid) { |
| 422 | // SignApk has historically accepted md5WithRSA certificates, but treated them as |
| 423 | // sha1WithRSA anyway. Continue to do so for backwards compatibility. |
| 424 | case NID_md5WithRSA: |
| 425 | case NID_md5WithRSAEncryption: |
| 426 | case NID_sha1WithRSA: |
| 427 | case NID_sha1WithRSAEncryption: |
| 428 | cert->hash_len = SHA_DIGEST_LENGTH; |
| 429 | break; |
| 430 | case NID_sha256WithRSAEncryption: |
| 431 | case NID_ecdsa_with_SHA256: |
| 432 | cert->hash_len = SHA256_DIGEST_LENGTH; |
| 433 | break; |
| 434 | default: |
| 435 | LOG(ERROR) << "Unrecognized signature nid " << OBJ_nid2ln(nid); |
| 436 | return false; |
| 437 | } |
| 438 | |
| 439 | std::unique_ptr<EVP_PKEY, decltype(&EVP_PKEY_free)> public_key(X509_get_pubkey(x509.get()), |
| 440 | EVP_PKEY_free); |
| 441 | if (!public_key) { |
| 442 | LOG(ERROR) << "Failed to extract the public key from x509 certificate"; |
| 443 | return false; |
| 444 | } |
| 445 | |
| 446 | int key_type = EVP_PKEY_id(public_key.get()); |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 447 | if (key_type == EVP_PKEY_RSA) { |
| 448 | cert->key_type = Certificate::KEY_TYPE_RSA; |
| 449 | cert->ec.reset(); |
| 450 | cert->rsa.reset(EVP_PKEY_get1_RSA(public_key.get())); |
Tianjie Xu | b5110de | 2018-10-23 23:31:43 -0700 | [diff] [blame] | 451 | if (!cert->rsa || !CheckRSAKey(cert->rsa)) { |
| 452 | LOG(ERROR) << "Failed to validate the rsa key info from public key"; |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 453 | return false; |
| 454 | } |
| 455 | } else if (key_type == EVP_PKEY_EC) { |
| 456 | cert->key_type = Certificate::KEY_TYPE_EC; |
| 457 | cert->rsa.reset(); |
| 458 | cert->ec.reset(EVP_PKEY_get1_EC_KEY(public_key.get())); |
Tianjie Xu | b5110de | 2018-10-23 23:31:43 -0700 | [diff] [blame] | 459 | if (!cert->ec || !CheckECKey(cert->ec)) { |
| 460 | LOG(ERROR) << "Failed to validate the ec key info from the public key"; |
Tianjie Xu | 8256698 | 2018-10-10 15:44:17 -0700 | [diff] [blame] | 461 | return false; |
| 462 | } |
| 463 | } else { |
| 464 | LOG(ERROR) << "Unrecognized public key type " << OBJ_nid2ln(key_type); |
| 465 | return false; |
| 466 | } |
| 467 | |
| 468 | return true; |
| 469 | } |