The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

I think it can be tested by replacing the ConsumeBool() to true which causes a timeout quickly.

--- 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(true, 10000)
     {
         // Mode:

Maybe this is a way to know whether a harness can have a possible timeout issue? cc: @maflcko

Please cross-link to the issue (#30505)

lgtm ACK 8dec4e1

https://cirrus-ci.com/task/6004238088667136?logs=ci#L5367

crypto_fschacha20poly1305                #83	DONE   cov: 3202 ft: 21189 corp: 80/12762b lim: 1368 exec/s: 2 rss: 119Mb

https://cirrus-ci.com/task/6249653828583424?logs=ci#L5217

crypto_fschacha20poly1305                #83	DONE   cov: 3202 ft: 21417 corp: 80/12762b lim: 1368 exec/s: 1 rss: 121Mb

cc: @stratospher

Concept ACK. (running it locally, will ACK after that!)
did you check if the timeout happens in crypto_aeadchacha20poly1305 too?

did you check if the timeout happens in crypto_aeadchacha20poly1305 too?

I think we did not have an issue with that yet, but anyway, I'll investigate it and follow-up.