-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
quic fuzzer hang #22412
Comments
If you fix this, can I suggest that you run the fuzzer for a minute or so. I've found both those hangs in a few seconds. Use libfuzzer, and add -timeout=5 to the command line, so it gives you a nice file instead of just hanging for a very long time.
|
This hang is caused because we receive a very large token in a retry packet. Later we try to create a PING frame but, because the token is so large, we fail to insert the PING or anything else into the datagram. Since we have failed to generate a PING, but the ping deadline has expired we immediately try to create another PING - and so on in an infinite loop. |
If when generating a datagram we are unable to output anything then we are not making progress and we should fail, otherwise we may get into an infinite loop (i.e. continually trying to output a datagram and continually failing to do so in an infinite loop). Fixes openssl#22412
I did that and immediately ran into #22427 |
The original hang was fixed. |
The fuzzer from #22368 combined with #22410 and #22411, hangs with
hang2.gz
The text was updated successfully, but these errors were encountered: