Merge bitcoin/bitcoin#21890: fuzz: Limit ParseISO8601DateTime fuzzing to 32-bit

fanquake · fanquake · commit 86e04741dd7f · 2021-05-10T14:20:01.000+08:00
fa1aa6c fuzz: Limit ParseISO8601DateTime fuzzing to 32-bit (MarcoFalke) Pull request description: 2038 is more than 10 years in the future, so no need for us to waste time fuzzing a 3rd party lib that will be EOL by then. Hopefully fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=34092 ACKs for top commit: theStack: Concept and code review ACK fa1aa6c Tree-SHA512: fdd2fbc7b5c7ce33ad23b2e5431bb97eaf6ae8c2d2a55990a3ab73be79282c584b704dcd1471ba288de75283732970c70c9a03ddad059b97b66ba8b3de39effe
diff --git a/src/test/fuzz/parse_iso8601.cpp b/src/test/fuzz/parse_iso8601.cpp
@@ -15,7 +15,7 @@ FUZZ_TARGET(parse_iso8601)
 {
     FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size());
 
-    const int64_t random_time = fuzzed_data_provider.ConsumeIntegral<int64_t>();
+    const int64_t random_time = fuzzed_data_provider.ConsumeIntegral<int32_t>();
     const std::string random_string = fuzzed_data_provider.ConsumeRemainingBytesAsString();
 
     const std::string iso8601_datetime = FormatISO8601DateTime(random_time);

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ FUZZ_TARGET(parse_iso8601)`
`15`	`15`	`{`
`16`	`16`	`FuzzedDataProvider fuzzed_data_provider(buffer.data(), buffer.size());`
`17`	`17`
`18`		`- const int64_t random_time = fuzzed_data_provider.ConsumeIntegral<int64_t>();`
	`18`	`+ const int64_t random_time = fuzzed_data_provider.ConsumeIntegral<int32_t>();`
`19`	`19`	`const std::string random_string = fuzzed_data_provider.ConsumeRemainingBytesAsString();`
`20`	`20`
`21`	`21`	`const std::string iso8601_datetime = FormatISO8601DateTime(random_time);`