Merge bitcoin/bitcoin#30725: fuzz: fix timeout in crypto_fschacha20poly1305

fanquake · fanquake · commit 128ade02e445 · 2024-08-28T09:42:43.000+01:00
8dec4e1 fuzz: fix timeout in `crypto_fschacha20poly1305` (brunoerg) Pull request description: Fixes #30505 This PR fixes a timeout in `crypto_fschacha20poly1305` by reducing the number of iterations. I left it running for a while and noticed it speeds up the target and do not impact coverage. ACKs for top commit: maflcko: lgtm ACK 8dec4e1 stratospher: ACK 8dec4e1. saw similar coverage stats (these are from different machines, saw more similar from same machine). Tree-SHA512: 66cd689f10f09cf880c1f2c77aa481336a95a7e10b655d156b2dc9cd38165670000019a6fb3775485ccb5cce248939a52a31ebd755a6befed6274cc841d21d97
diff --git a/src/test/fuzz/crypto_chacha20poly1305.cpp b/src/test/fuzz/crypto_chacha20poly1305.cpp
@@ -130,7 +130,7 @@ FUZZ_TARGET(crypto_fschacha20poly1305)
     // data).
     InsecureRandomContext rng(provider.ConsumeIntegral<uint64_t>());
 
-    LIMITED_WHILE(provider.ConsumeBool(), 10000)
+    LIMITED_WHILE(provider.ConsumeBool(), 100)
     {
         // Mode:
         // - Bit 0: whether to use single-plain Encrypt/Decrypt; otherwise use a split at prefix.

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ FUZZ_TARGET(crypto_fschacha20poly1305)`
`130`	`130`	`// data).`
`131`	`131`	`InsecureRandomContext rng(provider.ConsumeIntegral<uint64_t>());`
`132`	`132`
`133`		`- LIMITED_WHILE(provider.ConsumeBool(), 10000)`
	`133`	`+ LIMITED_WHILE(provider.ConsumeBool(), 100)`
`134`	`134`	`{`
`135`	`135`	`// Mode:`
`136`	`136`	`// - Bit 0: whether to use single-plain Encrypt/Decrypt; otherwise use a split at prefix.`