From e37f53648e3acc6aea75adafec4de2bdbd8cb293 Mon Sep 17 00:00:00 2001 From: practicalswift Date: Wed, 26 Feb 2020 20:04:23 +0000 Subject: Make lifetime correctness easier to see (avoid reference lifetime extension) --- src/test/fuzz/bloom_filter.cpp | 2 +- src/test/fuzz/rolling_bloom_filter.cpp | 2 +- src/test/fuzz/util.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/test/fuzz/bloom_filter.cpp b/src/test/fuzz/bloom_filter.cpp index b78744d9df..d1112f8e62 100644 --- a/src/test/fuzz/bloom_filter.cpp +++ b/src/test/fuzz/bloom_filter.cpp @@ -27,7 +27,7 @@ void test_one_input(const std::vector& buffer) while (fuzzed_data_provider.remaining_bytes() > 0) { switch (fuzzed_data_provider.ConsumeIntegralInRange(0, 6)) { case 0: { - const std::vector& b = ConsumeRandomLengthByteVector(fuzzed_data_provider); + const std::vector b = ConsumeRandomLengthByteVector(fuzzed_data_provider); (void)bloom_filter.contains(b); bloom_filter.insert(b); const bool present = bloom_filter.contains(b); diff --git a/src/test/fuzz/rolling_bloom_filter.cpp b/src/test/fuzz/rolling_bloom_filter.cpp index ce69c4e8da..3b37321977 100644 --- a/src/test/fuzz/rolling_bloom_filter.cpp +++ b/src/test/fuzz/rolling_bloom_filter.cpp @@ -24,7 +24,7 @@ void test_one_input(const std::vector& buffer) while (fuzzed_data_provider.remaining_bytes() > 0) { switch (fuzzed_data_provider.ConsumeIntegralInRange(0, 2)) { case 0: { - const std::vector& b = ConsumeRandomLengthByteVector(fuzzed_data_provider); + const std::vector b = ConsumeRandomLengthByteVector(fuzzed_data_provider); (void)rolling_bloom_filter.contains(b); rolling_bloom_filter.insert(b); const bool present = rolling_bloom_filter.contains(b); diff --git a/src/test/fuzz/util.h b/src/test/fuzz/util.h index a2b7d804ea..2f4aa9ad2b 100644 --- a/src/test/fuzz/util.h +++ b/src/test/fuzz/util.h @@ -27,7 +27,7 @@ NODISCARD inline std::vector ConsumeRandomLengthByteVector(FuzzedDataPr template NODISCARD inline Optional ConsumeDeserializable(FuzzedDataProvider& fuzzed_data_provider, size_t max_length = 4096) noexcept { - const std::vector& buffer = ConsumeRandomLengthByteVector(fuzzed_data_provider, max_length); + const std::vector buffer = ConsumeRandomLengthByteVector(fuzzed_data_provider, max_length); CDataStream ds{buffer, SER_NETWORK, INIT_PROTO_VERSION}; T obj; try { -- cgit v1.2.3