fix(client): validate blocking receive lifecycle before socket I/O
Rust CI / cargo fmt (push) Successful in 15s
Rust CI / cargo check (x86_64-unknown-linux-gnu) (push) Successful in 28s
Rust CI / cargo clippy (push) Successful in 31s
Rust CI / cargo check (x86_64-unknown-linux-musl) (push) Failing after 9s
Rust CI / cargo test (x86_64-unknown-linux-musl) (push) Failing after 10s
Rust CI / cargo doc (push) Successful in 23s
Rust CI / cargo test (x86_64-unknown-linux-gnu) (push) Successful in 1m10s
Fuzz Smoke / cargo fuzz smoke (push) Failing after 29s
Supply chain / cargo-deny advisories (push) Successful in 6s
Supply chain / cargo-deny bans licenses sources (push) Successful in 5s
Supply chain / actionlint (push) Successful in 3s
Netem Interop / netem interop (push) Successful in 59s

Blocking receives previously checked lifecycle only after a datagram arrived, allowing invalid operations to block or return empty success after timeout. Reuse session validation before socket access and cover connected, closed, and no-test-completed states through public APIs.
This commit is contained in:
Nils Andreas Svee committed 2026-10-04 13:35:37 +07:00
1 parent ebc1f2d0b1
commit 596dd20bcf
5 files changed
+80 -4

No files matched your search

+33 -2
View File
@@ -1,7 +1,7 @@
use super::*;
#[test]
fn open_fails_after_close() {
fn operations_fail_after_local_close_without_datagrams() {
let params = default_params();
let server = start_fake_server(move |socket, tx| {
let (_, peer) = recv_request(&socket, &tx);
@@ -9,10 +9,26 @@ fn open_fails_after_close() {
socket.send_to(&reply, peer).unwrap();
let _ = recv_request(&socket, &tx);
});
let mut client = Client::connect(default_test_config(server.addr)).unwrap();
let mut config = default_test_config(server.addr);
config.socket_config.recv_timeout = Some(Duration::from_millis(50));
let mut client = Client::connect(config).unwrap();
assert_open_started(client.open().unwrap());
client.close().unwrap();
assert!(matches!(client.open(), Err(ClientError::AlreadyClosed)));
let results = [
client.recv_once(),
client.recv_available(RecvBudget { max_packets: 1 }),
];
assert!(
matches!(
results,
[
Err(ClientError::AlreadyClosed),
Err(ClientError::AlreadyClosed)
]
),
"unexpected receive results: {results:?}"
);
server.join();
}
@@ -473,6 +489,21 @@ fn close_flagged_echo_reply_emits_reply_then_closes_without_sending_close() {
));
assert_eq!(events.len(), 2);
let results = [
client.recv_once(),
client.recv_available(RecvBudget { max_packets: 1 }),
];
assert!(
matches!(
results,
[
Err(ClientError::AlreadyClosed),
Err(ClientError::AlreadyClosed)
]
),
"unexpected receive results: {results:?}"
);
assert!(matches!(
client.send_probe(),
Err(ClientError::AlreadyClosed)