Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // |
| 2 | // Copyright (C) 2012 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 | // |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 16 | |
Alex Deymo | 14c0da8 | 2016-07-20 16:45:45 -0700 | [diff] [blame] | 17 | #include "update_engine/certificate_checker.h" |
Alex Deymo | 8427b4a | 2014-11-05 14:00:32 -0800 | [diff] [blame] | 18 | |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 19 | #include <string> |
| 20 | |
Kelvin Zhang | b9a9aa2 | 2024-10-15 10:38:35 -0700 | [diff] [blame] | 21 | #include <android-base/stringprintf.h> |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 22 | #include <gmock/gmock.h> |
| 23 | #include <gtest/gtest.h> |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 24 | |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 25 | #include "update_engine/common/constants.h" |
Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 26 | #include "update_engine/common/mock_prefs.h" |
Alex Deymo | 14c0da8 | 2016-07-20 16:45:45 -0700 | [diff] [blame] | 27 | #include "update_engine/mock_certificate_checker.h" |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 28 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 29 | using std::string; |
| 30 | using ::testing::_; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 31 | using ::testing::DoAll; |
| 32 | using ::testing::Return; |
Ben Chan | 672c1f5 | 2017-10-23 15:41:39 -0700 | [diff] [blame] | 33 | using ::testing::SetArgPointee; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 34 | using ::testing::SetArrayArgument; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 35 | |
| 36 | namespace chromeos_update_engine { |
| 37 | |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 38 | class MockCertificateCheckObserver : public CertificateChecker::Observer { |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 39 | public: |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 40 | MOCK_METHOD2(CertificateChecked, |
| 41 | void(ServerToCheck server_to_check, |
| 42 | CertificateCheckResult result)); |
| 43 | }; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 44 | |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 45 | class CertificateCheckerTest : public testing::Test { |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 46 | protected: |
Alex Deymo | 610277e | 2014-11-11 21:18:11 -0800 | [diff] [blame] | 47 | void SetUp() override { |
Kelvin Zhang | b9a9aa2 | 2024-10-15 10:38:35 -0700 | [diff] [blame] | 48 | cert_key_ = android::base::StringPrintf("%s-%d-%d", |
| 49 | cert_key_prefix_.c_str(), |
| 50 | static_cast<int>(server_to_check_), |
| 51 | depth_); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 52 | cert_checker.Init(); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 53 | cert_checker.SetObserver(&observer_); |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 54 | } |
| 55 | |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 56 | void TearDown() override { cert_checker.SetObserver(nullptr); } |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 57 | |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 58 | MockPrefs prefs_; |
Alex Deymo | 8427b4a | 2014-11-05 14:00:32 -0800 | [diff] [blame] | 59 | MockOpenSSLWrapper openssl_wrapper_; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 60 | // Parameters of our mock certificate digest. |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 61 | int depth_{0}; |
| 62 | unsigned int length_{4}; |
| 63 | uint8_t digest_[4]{0x17, 0x7D, 0x07, 0x5F}; |
| 64 | string digest_hex_{"177D075F"}; |
| 65 | string diff_digest_hex_{"1234ABCD"}; |
| 66 | string cert_key_prefix_{kPrefsUpdateServerCertificate}; |
| 67 | ServerToCheck server_to_check_{ServerToCheck::kUpdate}; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 68 | string cert_key_; |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 69 | |
| 70 | testing::StrictMock<MockCertificateCheckObserver> observer_; |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 71 | CertificateChecker cert_checker{&prefs_, &openssl_wrapper_}; |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 72 | }; |
| 73 | |
| 74 | // check certificate change, new |
| 75 | TEST_F(CertificateCheckerTest, NewCertificate) { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 76 | EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 77 | .WillOnce(DoAll(SetArgPointee<1>(depth_), |
| 78 | SetArgPointee<2>(length_), |
| 79 | SetArrayArgument<3>(digest_, digest_ + 4), |
| 80 | Return(true))); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 81 | EXPECT_CALL(prefs_, GetString(cert_key_, _)).WillOnce(Return(false)); |
| 82 | EXPECT_CALL(prefs_, SetString(cert_key_, digest_hex_)).WillOnce(Return(true)); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 83 | EXPECT_CALL( |
| 84 | observer_, |
| 85 | CertificateChecked(server_to_check_, CertificateCheckResult::kValid)); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 86 | ASSERT_TRUE( |
| 87 | cert_checker.CheckCertificateChange(1, nullptr, server_to_check_)); |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 88 | } |
| 89 | |
| 90 | // check certificate change, unchanged |
| 91 | TEST_F(CertificateCheckerTest, SameCertificate) { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 92 | EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 93 | .WillOnce(DoAll(SetArgPointee<1>(depth_), |
| 94 | SetArgPointee<2>(length_), |
| 95 | SetArrayArgument<3>(digest_, digest_ + 4), |
| 96 | Return(true))); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 97 | EXPECT_CALL(prefs_, GetString(cert_key_, _)) |
Ben Chan | 672c1f5 | 2017-10-23 15:41:39 -0700 | [diff] [blame] | 98 | .WillOnce(DoAll(SetArgPointee<1>(digest_hex_), Return(true))); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 99 | EXPECT_CALL(prefs_, SetString(_, _)).Times(0); |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 100 | EXPECT_CALL( |
| 101 | observer_, |
| 102 | CertificateChecked(server_to_check_, CertificateCheckResult::kValid)); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 103 | ASSERT_TRUE( |
| 104 | cert_checker.CheckCertificateChange(1, nullptr, server_to_check_)); |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 105 | } |
| 106 | |
| 107 | // check certificate change, changed |
| 108 | TEST_F(CertificateCheckerTest, ChangedCertificate) { |
Alex Vakulenko | 88b591f | 2014-08-28 16:48:57 -0700 | [diff] [blame] | 109 | EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 110 | .WillOnce(DoAll(SetArgPointee<1>(depth_), |
| 111 | SetArgPointee<2>(length_), |
| 112 | SetArrayArgument<3>(digest_, digest_ + 4), |
| 113 | Return(true))); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 114 | EXPECT_CALL(prefs_, GetString(cert_key_, _)) |
Ben Chan | 672c1f5 | 2017-10-23 15:41:39 -0700 | [diff] [blame] | 115 | .WillOnce(DoAll(SetArgPointee<1>(diff_digest_hex_), Return(true))); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 116 | EXPECT_CALL(observer_, |
| 117 | CertificateChecked(server_to_check_, |
| 118 | CertificateCheckResult::kValidChanged)); |
| 119 | EXPECT_CALL(prefs_, SetString(cert_key_, digest_hex_)).WillOnce(Return(true)); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 120 | ASSERT_TRUE( |
| 121 | cert_checker.CheckCertificateChange(1, nullptr, server_to_check_)); |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 122 | } |
| 123 | |
| 124 | // check certificate change, failed |
| 125 | TEST_F(CertificateCheckerTest, FailedCertificate) { |
Amin Hassani | 7cc8bb0 | 2019-01-14 16:29:47 -0800 | [diff] [blame] | 126 | EXPECT_CALL( |
| 127 | observer_, |
| 128 | CertificateChecked(server_to_check_, CertificateCheckResult::kFailed)); |
Alex Deymo | c1c17b4 | 2015-11-23 03:53:15 -0300 | [diff] [blame] | 129 | EXPECT_CALL(prefs_, GetString(_, _)).Times(0); |
Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 130 | EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(_, _, _, _)).Times(0); |
Alex Deymo | 33e91e7 | 2015-12-01 18:26:08 -0300 | [diff] [blame] | 131 | ASSERT_FALSE( |
| 132 | cert_checker.CheckCertificateChange(0, nullptr, server_to_check_)); |
Bruno Rocha | 7f9aea2 | 2011-09-12 14:31:24 -0700 | [diff] [blame] | 133 | } |
| 134 | |
| 135 | } // namespace chromeos_update_engine |