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

Avoid panics in uses of Instant #2746

Merged
merged 1 commit into from Feb 1, 2022

Commits on Jan 31, 2022

  1. Avoid panics in uses of Instant

    We have reports of runtime panics (linkerd/linkerd2#7748) that sound a
    lot like rust-lang/rust#86470. We don't have any evidence that these
    panics originate in hyper, but hyperium#2385 reports a similar issue.
    
    Even though this is almost definitely a bug in Rust, it seems most
    prudent to actively avoid the uses of `Instant` that are prone to this
    bug.
    
    This change replaces uses of `Instant::elapsed` and `Instant::sub` with
    calls to `Instant::saturating_duration_since` to prevent this class of
    panic.
    olix0r committed Jan 31, 2022
    Copy the full SHA
    f9ec8cd View commit details
    Browse the repository at this point in the history