Avi Drissman | 6459548 | 2022-09-14 20:52:29 | [diff] [blame] | 1 | // Copyright 2020 The Chromium Authors |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include <stddef.h> |
| 6 | #include <stdint.h> |
| 7 | |
| 8 | #include <memory> |
| 9 | |
| 10 | #include "base/json/json_reader.h" |
Hans Wennborg | 725d043 | 2020-06-18 13:54:16 | [diff] [blame] | 11 | #include "base/logging.h" |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 12 | #include "base/strings/string_piece_forward.h" |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 13 | #include "base/time/tick_clock.h" |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 14 | #include "base/time/time.h" |
| 15 | #include "net/base/backoff_entry.h" |
| 16 | #include "net/base/backoff_entry_serializer.h" |
| 17 | #include "net/base/backoff_entry_serializer_fuzzer_input.pb.h" |
| 18 | #include "testing/libfuzzer/proto/json_proto_converter.h" |
| 19 | #include "testing/libfuzzer/proto/lpm_interface.h" |
Anton Bikineev | 068d291 | 2021-05-15 20:43:52 | [diff] [blame] | 20 | #include "third_party/abseil-cpp/absl/types/optional.h" |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 21 | |
| 22 | namespace net { |
| 23 | |
| 24 | namespace { |
| 25 | struct Environment { |
| 26 | Environment() { logging::SetMinLogLevel(logging::LOG_ERROR); } |
| 27 | }; |
| 28 | |
| 29 | class ProtoTranslator { |
| 30 | public: |
| 31 | explicit ProtoTranslator(const fuzz_proto::FuzzerInput& input) |
| 32 | : input_(input) {} |
| 33 | |
| 34 | BackoffEntry::Policy policy() const { |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 35 | return PolicyFromProto(input_->policy()); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 36 | } |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 37 | base::Time parse_time() const { |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 38 | return base::Time() + base::Microseconds(input_->parse_time()); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 39 | } |
Dan McArdle | 82d7097 | 2021-04-12 15:54:20 | [diff] [blame] | 40 | base::TimeTicks parse_time_ticks() const { |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 41 | return base::TimeTicks() + base::Microseconds(input_->parse_time()); |
Dan McArdle | 82d7097 | 2021-04-12 15:54:20 | [diff] [blame] | 42 | } |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 43 | base::Time serialize_time() const { |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 44 | return base::Time() + base::Microseconds(input_->serialize_time()); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 45 | } |
| 46 | base::TimeTicks now_ticks() const { |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 47 | return base::TimeTicks() + base::Microseconds(input_->now_ticks()); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 48 | } |
Anton Bikineev | 068d291 | 2021-05-15 20:43:52 | [diff] [blame] | 49 | absl::optional<base::Value> serialized_entry() const { |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 50 | json_proto::JsonProtoConverter converter; |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 51 | std::string json_array = converter.Convert(input_->serialized_entry()); |
Anton Bikineev | 068d291 | 2021-05-15 20:43:52 | [diff] [blame] | 52 | absl::optional<base::Value> value = base::JSONReader::Read(json_array); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 53 | return value; |
| 54 | } |
| 55 | |
| 56 | private: |
Ali Hijazi | f071ec0f | 2023-06-01 18:17:08 | [diff] [blame] | 57 | const raw_ref<const fuzz_proto::FuzzerInput> input_; |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 58 | |
| 59 | static BackoffEntry::Policy PolicyFromProto( |
| 60 | const fuzz_proto::BackoffEntryPolicy& policy) { |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 61 | BackoffEntry::Policy new_policy; |
| 62 | new_policy.num_errors_to_ignore = policy.num_errors_to_ignore(); |
| 63 | new_policy.initial_delay_ms = policy.initial_delay_ms(); |
| 64 | new_policy.multiply_factor = policy.multiply_factor(); |
| 65 | new_policy.jitter_factor = policy.jitter_factor(); |
| 66 | new_policy.maximum_backoff_ms = policy.maximum_backoff_ms(); |
| 67 | new_policy.entry_lifetime_ms = policy.entry_lifetime_ms(); |
| 68 | new_policy.always_use_initial_delay = policy.always_use_initial_delay(); |
| 69 | return new_policy; |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 70 | } |
| 71 | }; |
| 72 | |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 73 | class MockClock : public base::TickClock { |
| 74 | public: |
| 75 | MockClock() = default; |
| 76 | ~MockClock() override = default; |
| 77 | |
| 78 | void SetNow(base::TimeTicks now) { now_ = now; } |
| 79 | base::TimeTicks NowTicks() const override { return now_; } |
| 80 | |
| 81 | private: |
| 82 | base::TimeTicks now_; |
| 83 | }; |
| 84 | |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 85 | // Tests the "deserialize-reserialize" property. Deserializes a BackoffEntry |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 86 | // from JSON, reserializes it, then deserializes again. Holding time constant, |
| 87 | // we check that the parsed BackoffEntry values are equivalent. |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 88 | void TestDeserialize(const ProtoTranslator& translator) { |
| 89 | // Attempt to convert the json_proto.ArrayValue to a base::Value. |
Anton Bikineev | 068d291 | 2021-05-15 20:43:52 | [diff] [blame] | 90 | absl::optional<base::Value> value = translator.serialized_entry(); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 91 | if (!value) |
| 92 | return; |
| 93 | DCHECK(value->is_list()); |
| 94 | |
| 95 | BackoffEntry::Policy policy = translator.policy(); |
| 96 | |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 97 | MockClock clock; |
Dan McArdle | 82d7097 | 2021-04-12 15:54:20 | [diff] [blame] | 98 | clock.SetNow(translator.parse_time_ticks()); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 99 | |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 100 | // Attempt to deserialize a BackoffEntry. |
| 101 | std::unique_ptr<BackoffEntry> entry = |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 102 | BackoffEntrySerializer::DeserializeFromList( |
| 103 | value->GetList(), &policy, &clock, translator.parse_time()); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 104 | if (!entry) |
| 105 | return; |
| 106 | |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 107 | base::Value::List reserialized = |
| 108 | BackoffEntrySerializer::SerializeToList(*entry, translator.parse_time()); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 109 | |
| 110 | // Due to fuzzy interpretation in BackoffEntrySerializer:: |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 111 | // DeserializeFromList, we cannot assert that |*reserialized == *value|. |
Daniel McArdle | a8799f0 | 2020-08-04 15:44:16 | [diff] [blame] | 112 | // Rather, we can deserialize |reserialized| and check that some weaker |
| 113 | // properties are preserved. |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 114 | std::unique_ptr<BackoffEntry> entry_reparsed = |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 115 | BackoffEntrySerializer::DeserializeFromList(reserialized, &policy, &clock, |
| 116 | translator.parse_time()); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 117 | CHECK(entry_reparsed); |
Daniel McArdle | a8799f0 | 2020-08-04 15:44:16 | [diff] [blame] | 118 | CHECK_EQ(entry_reparsed->failure_count(), entry->failure_count()); |
| 119 | CHECK_LE(entry_reparsed->GetReleaseTime(), entry->GetReleaseTime()); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 120 | } |
| 121 | |
| 122 | // Tests the "serialize-deserialize" property. Serializes an arbitrary |
| 123 | // BackoffEntry to JSON, deserializes to another BackoffEntry, and checks |
| 124 | // equality of the two entries. Our notion of equality is *very weak* and needs |
| 125 | // improvement. |
| 126 | void TestSerialize(const ProtoTranslator& translator) { |
| 127 | BackoffEntry::Policy policy = translator.policy(); |
| 128 | |
| 129 | // Serialize the BackoffEntry. |
| 130 | BackoffEntry native_entry(&policy); |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 131 | base::Value::List serialized = BackoffEntrySerializer::SerializeToList( |
Maks Orlovich | 691496c | 2021-05-18 15:03:21 | [diff] [blame] | 132 | native_entry, translator.serialize_time()); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 133 | |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 134 | MockClock clock; |
| 135 | clock.SetNow(translator.now_ticks()); |
| 136 | |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 137 | // Deserialize it. |
| 138 | std::unique_ptr<BackoffEntry> deserialized_entry = |
Jiacheng Guo | 9dfe9f6 | 2022-09-21 11:35:47 | [diff] [blame] | 139 | BackoffEntrySerializer::DeserializeFromList(serialized, &policy, &clock, |
| 140 | translator.parse_time()); |
| 141 | // Even though SerializeToList was successful, we're not guaranteed to have a |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 142 | // |deserialized_entry|. One reason deserialization may fail is if the parsed |
| 143 | // |absolute_release_time_us| is below zero. |
| 144 | if (!deserialized_entry) |
| 145 | return; |
| 146 | |
| 147 | // TODO(dmcardle) Develop a stronger equality check for BackoffEntry. |
| 148 | |
| 149 | // Note that while |BackoffEntry::GetReleaseTime| looks like an accessor, it |
| 150 | // returns a |value that is computed based on a random double, so it's not |
| 151 | // suitable for CHECK_EQ here. See |BackoffEntry::CalculateReleaseTime|. |
| 152 | |
| 153 | CHECK_EQ(native_entry.failure_count(), deserialized_entry->failure_count()); |
| 154 | } |
| 155 | } // namespace |
| 156 | |
| 157 | DEFINE_PROTO_FUZZER(const fuzz_proto::FuzzerInput& input) { |
| 158 | static Environment env; |
| 159 | |
| 160 | // Print the entire |input| protobuf if asked. |
| 161 | if (getenv("LPM_DUMP_NATIVE_INPUT")) { |
| 162 | std::cout << "input: " << input.DebugString(); |
| 163 | } |
| 164 | |
| 165 | ProtoTranslator translator(input); |
Dan McArdle | 22c2f64 | 2021-03-18 23:40:10 | [diff] [blame] | 166 | // Skip this input if any of the time values are infinite. |
| 167 | if (translator.now_ticks().is_inf() || translator.parse_time().is_inf() || |
| 168 | translator.serialize_time().is_inf()) { |
| 169 | return; |
| 170 | } |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 171 | TestDeserialize(translator); |
Daniel McArdle | bbfc145 | 2020-05-18 15:11:55 | [diff] [blame] | 172 | TestSerialize(translator); |
Daniel McArdle | 72692a1 | 2020-05-11 16:47:16 | [diff] [blame] | 173 | } |
| 174 | |
| 175 | } // namespace net |