github-actions[bot] opened a new issue, #3429:
URL: https://github.com/apache/iceberg-rust/issues/3429

   
   > Peer minimum ACK delay can panic quinn-proto when ACK_FREQUENCY 
transmission is enabled
   
   | Details             |                                                |
   | ------------------- | ---------------------------------------------- |
   | Package             | `quinn-proto`                      |
   | Version             | `0.11.16`                   |
   | URL                 | 
[https://github.com/quinn-rs/quinn/pull/2868](https://github.com/quinn-rs/quinn/pull/2868)
 |
   | Date                | 2026-09-14                         |
   | Patched versions    | `>=0.11.18`                  |
   | Unaffected versions | `<0.11.0`               |
   
   When sending ACK_FREQUENCY frames is enabled through
   `TransportConfig::ack_frequency_config(Some(..))`, the peer&#39;s transport 
parameters could cause
   `AckFrequencyState::candidate_max_ack_delay` to call `Duration::clamp` with 
a minimum greater
   than its maximum, which panics.
   
   For example, a peer advertising a `min_ack_delay` of 100,000 microseconds 
and a `max_ack_delay`
   of 100 milliseconds passes transport parameter validation. At a 5 ms RTT, 
the lower bound of the
   clamp becomes 100 ms while the upper bound is 25 ms, violating the 
precondition of `clamp` and
   panicking while driving the connection. Both automatically derived and 
explicitly configured
   maximum ACK delays use this calculation.
   
   Sending ACK_FREQUENCY frames is disabled by default, so only endpoints that 
opt in are affected.
   
   The flaw was corrected in 0.11.18 by computing the delay with explicit 
minimum and maximum bounds
   instead of `clamp`.
   
   Thanks to [@zerox80](https://github.com/zerox80) for reporting this issue.
   
   See [advisory page](https://rustsec.org/advisories/RUSTSEC-2026-0340.html) 
for additional details.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to