Skip to content
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

handshakecorruption crash #1942

Closed
larseggert opened this issue Jul 25, 2024 · 5 comments · Fixed by quinn-rs/quinn-interop#12
Closed

handshakecorruption crash #1942

larseggert opened this issue Jul 25, 2024 · 5 comments · Fixed by quinn-rs/quinn-interop#12

Comments

@larseggert
Copy link
Contributor

https://github.com/mozilla/neqo/actions/runs/10096708958/job/27920055350#step:6:735

client  | stack backtrace:
client  |    0: rust_begin_unwind
client  |              at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/std/src/panicking.rs:593:5
client  |    1: core::panicking::panic_fmt
client  |              at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/core/src/panicking.rs:67:14
client  |    2: core::result::unwrap_failed
client  |              at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/core/src/result.rs:1651:5
client  |    3: client::main::{{closure}}
client  |    4: tokio::runtime::park::CachedParkThread::block_on
client  |    5: tokio::runtime::runtime::Runtime::block_on
client  |    6: client::main
client  | note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
@djc
Copy link
Member

djc commented Jul 25, 2024

Looks like it might be related to #1938?

@Ralith
Copy link
Collaborator

Ralith commented Jul 25, 2024

The link seems to be a failure in the h3 test, not handshakecorruption? Scrolling up, that case ("C1") is reported as passing:

+-------+----------------------------------------------+
|       |                 neqo-latest                  |
+-------+----------------------------------------------+
| quinn | ✓(H,DC,LR,C20,M,S,R,Z,B,U,E,A,L1,L2,C1,C2,6) |
|       |                     ?()                      |
|       |                   ✕(3,V2)                    |
+-------+----------------------------------------------+

As for why it's failing, you clipped the key line:

client  | thread 'main' panicked at 'driver: h3::Error { timeout: true }', src/bin/client.rs:175:18

Looks like that reproduces. I'm not familiar with the h3 stack, but I'll take a quick look anyway.

@Ralith
Copy link
Collaborator

Ralith commented Jul 25, 2024

Fixed by quinn-rs/quinn-interop#12.

@Ralith
Copy link
Collaborator

Ralith commented Jul 25, 2024

Note that we do have a handshakecorruption failure (not crash) in the current version. This is fixed by #1937, which we'll include in a maintenance release soon.

@larseggert
Copy link
Contributor Author

Sorry I got getting the test name wrong, but glad you managed to fix it quickly in any case!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants