diff options
| author | Per Lindgren <per.lindgren@ltu.se> | 2019-11-18 16:36:17 +0100 |
|---|---|---|
| committer | Per Lindgren <per.lindgren@ltu.se> | 2019-11-18 16:36:17 +0100 |
| commit | dfab15ed781fd780146d7592d4e0a86d6c31b4cc (patch) | |
| tree | 9d274803cce759e162cef9fc84d7267f12c93aff | |
| parent | 725d5e1aa9d0276783765dcabe5430f03ea13dd5 (diff) | |
Fixed internal overflow on subtraiton in elapsed and duration
| -rw-r--r-- | CHANGELOG.md | 5 | ||||
| -rw-r--r-- | src/cyccnt.rs | 6 |
2 files changed, 9 insertions, 2 deletions
diff --git a/CHANGELOG.md b/CHANGELOG.md index c96df3c..5a5cdec 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,11 @@ This project adheres to [Semantic Versioning](http://semver.org/). ## [Unreleased] +## v0.5.1 - 2019-11-18 +- Fixed arithmetic wrapping bug in src/cyccntr.rs + elapsed and duration could cause an internal overflow trap + on subtraction in debug mode. + ## v0.5.0 - 2019-11-14 ### Added diff --git a/src/cyccnt.rs b/src/cyccnt.rs index 86969cb..338bbbe 100644 --- a/src/cyccnt.rs +++ b/src/cyccnt.rs @@ -38,12 +38,14 @@ impl Instant { /// Returns the amount of time elapsed since this instant was created. pub fn elapsed(&self) -> Duration { - Instant::now() - *self + let diff = Instant::now().inner.wrapping_sub(self.inner); + assert!(diff >= 0, "instant now is earlier than self"); + Duration { inner: diff as u32 } } /// Returns the amount of time elapsed from another instant to this one. pub fn duration_since(&self, earlier: Instant) -> Duration { - let diff = self.inner - earlier.inner; + let diff = self.inner.wrapping_sub(earlier.inner); assert!(diff >= 0, "second instant is later than self"); Duration { inner: diff as u32 } } |
