Increase timeout for quic-srtm fuzzer#14938
Merged
DavidKorczynski merged 1 commit intogoogle:masterfrom Feb 18, 2026
Merged
Conversation
The quic-srtm fuzzer can run for quite some time in a single call to FuzzerTestOneInput. This occurs for two reasons: 1) The input buffer is consumed only a few bytes at a time, using a single byte to determine a command to run, and just a few more as input data. 2) The test mutates a hash table that can grow quite large based on the input data, and some commands iterate over the hash table multiple times. with each of two hash tables often growing to 4000-5000 entries. The sum total of this is that a single call to FuzzerTestOneInput may result in the function running for more than 60 secnods, triggering a timeout So lets address that by...waiting longer. Up the timeout of the test to 120 seconds, which should be sufficient to complete an iteration. This should address: https://oss-fuzz.com/testcase-detail/4785309438246912
|
nhorman is either the primary contact or is in the CCs list of projects/openssl. |
esyr
approved these changes
Feb 11, 2026
Sashan
approved these changes
Feb 12, 2026
Contributor
Sashan
left a comment
Sashan
left a comment
There was a problem hiding this comment.
two minutes should be enough for everyone...
DavidKorczynski
approved these changes
Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The quic-srtm fuzzer can run for quite some time in a single call to FuzzerTestOneInput. This occurs for two reasons:
The input buffer is consumed only a few bytes at a time, using a
single byte to determine a command to run, and just a few more as
input data.
The test mutates a hash table that can grow quite large based on the
input data, and some commands iterate over the hash table multiple
times. with each of two hash tables often growing to 4000-5000
entries.
The sum total of this is that a single call to FuzzerTestOneInput may result in the function running for more than 60 secnods, triggering a timeout
So lets address that by...waiting longer. Up the timeout of the test to 120 seconds, which should be sufficient to complete an iteration.
This should address:
https://oss-fuzz.com/testcase-detail/4785309438246912