diff options
| author | beetrees <b@beetr.ee> | 2022-10-05 22:27:17 +0100 |
|---|---|---|
| committer | beetrees <b@beetr.ee> | 2022-10-14 15:13:20 +0100 |
| commit | 5def7534e4f86165fd5f984e542c17921dc9fd08 (patch) | |
| tree | af6313250749536ee84d8dc77a820db63fadb062 /library/std/src/sys/hermit | |
| parent | 9b0a099dfc9a97ecd10adb319396c731c4b2d169 (diff) | |
Fix `checked_{add,sub}_duration` incorrectly returning `None` when `other` has more than `i64::MAX` seconds
Diffstat (limited to 'library/std/src/sys/hermit')
| -rw-r--r-- | library/std/src/sys/hermit/time.rs | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/library/std/src/sys/hermit/time.rs b/library/std/src/sys/hermit/time.rs index c17e6c8af62..d6d2b1a3027 100644 --- a/library/std/src/sys/hermit/time.rs +++ b/library/std/src/sys/hermit/time.rs @@ -39,11 +39,7 @@ impl Timespec { } fn checked_add_duration(&self, other: &Duration) -> Option<Timespec> { - let mut secs = other - .as_secs() - .try_into() // <- target type would be `libc::time_t` - .ok() - .and_then(|secs| self.t.tv_sec.checked_add(secs))?; + let mut secs = self.tv_sec.checked_add_unsigned(other.as_secs())?; // Nano calculations can't overflow because nanos are <1B which fit // in a u32. @@ -56,11 +52,7 @@ impl Timespec { } fn checked_sub_duration(&self, other: &Duration) -> Option<Timespec> { - let mut secs = other - .as_secs() - .try_into() // <- target type would be `libc::time_t` - .ok() - .and_then(|secs| self.t.tv_sec.checked_sub(secs))?; + let mut secs = self.tv_sec.checked_sub_unsigned(other.as_secs())?; // Similar to above, nanos can't overflow. let mut nsec = self.t.tv_nsec as i32 - other.subsec_nanos() as i32; |
