Skip to content

dtls13: bound ACK tracking during handshake replacement - #120

Merged
algesten merged 2 commits into
algesten:mainfrom
zRedShift:fix/dtls13-ack-replacement-overflow
May 25, 2026
Merged

algesten merged 2 commits into
algesten:mainfrom
zRedShift:fix/dtls13-ack-replacement-overflow

Conversation

@zRedShift

Copy link
Copy Markdown
Contributor

Summary

  • use bounded ACK tracking when replacing handled DTLS 1.3 handshake fragments
  • return a parser/state error instead of panicking if the ACK tracking array is full
  • add a regression that fills ACK tracking and exercises handled-handshake replacement

Tests

  • cargo fmt --manifest-path extern/scratch/dimpl/workspaces/DIMP-021/Cargo.toml --check
  • cargo test --manifest-path extern/scratch/dimpl/workspaces/DIMP-021/Cargo.toml --all-targets --features rcgen
  • cargo clippy --manifest-path extern/scratch/dimpl/workspaces/DIMP-021/Cargo.toml --all-targets --features rcgen -- -D warnings

@zRedShift
zRedShift force-pushed the fix/dtls13-ack-replacement-overflow branch from c19a23c to 59c49c4 Compare May 25, 2026 13:08
@algesten
algesten merged commit ce7d646 into algesten:main May 25, 2026
46 checks passed
@algesten

Copy link
Copy Markdown
Owner

Seems legit!

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 this pull request may close these issues.

2 participants