Skip to content

Commit 7d746bd

Browse files
author
MarcoFalke
committed
Merge bitcoin/bitcoin#23733: fuzz: Move ISO8601 to one place
fa72dd3 fuzz: Move ISO8601 to one place (MarcoFalke) Pull request description: Seems confusing to split this to two places. Also fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=42178 ACKs for top commit: fanquake: ACK fa72dd3 Tree-SHA512: 637b0671078848ea417fdf66b92715602040fad34d4ca5f7b843a519a1cfeebe5d992a79a399deba39926905125681d66ab0dc05f66f79a26f3bf555e12fb0ba
2 parents 50c502f + fa72dd3 commit 7d746bd

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

src/test/fuzz/integer.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,10 @@
2929
#include <util/strencodings.h>
3030
#include <util/string.h>
3131
#include <util/system.h>
32-
#include <util/time.h>
3332
#include <version.h>
3433

3534
#include <cassert>
3635
#include <chrono>
37-
#include <ctime>
3836
#include <limits>
3937
#include <set>
4038
#include <vector>
@@ -81,8 +79,6 @@ FUZZ_TARGET_INIT(integer, initialize_integer)
8179
(void)ComputeMerkleRoot(v256);
8280
(void)CountBits(u64);
8381
(void)DecompressAmount(u64);
84-
(void)FormatISO8601Date(i64);
85-
(void)FormatISO8601DateTime(i64);
8682
{
8783
if (std::optional<CAmount> parsed = ParseMoney(FormatMoney(i64))) {
8884
assert(parsed.value() == i64);

src/test/fuzz/parse_iso8601.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ FUZZ_TARGET(parse_iso8601)
1919
const std::string random_string = fuzzed_data_provider.ConsumeRemainingBytesAsString();
2020

2121
const std::string iso8601_datetime = FormatISO8601DateTime(random_time);
22+
(void)FormatISO8601Date(random_time);
2223
const int64_t parsed_time_1 = ParseISO8601DateTime(iso8601_datetime);
2324
if (random_time >= 0) {
2425
assert(parsed_time_1 >= 0);

0 commit comments

Comments
 (0)