diff options
| author | Sebastian Dröge <sebastian@centricular.com> | 2018-01-04 11:34:05 +0200 |
|---|---|---|
| committer | Sebastian Dröge <sebastian@centricular.com> | 2018-01-04 11:34:05 +0200 |
| commit | b69c124255e25d5d1c16ee88c4f2a75964db9209 (patch) | |
| tree | 82cb568f71eecb77e012e6f4e28d66406790aba5 /src/libcore/slice | |
| parent | 3f29e2b495a6b875af835f1fff64256e2d58d2b6 (diff) | |
Fix potential overflow in TrustedRandomAccess impl for slice::{Chunks,ChunksMut}
Diffstat (limited to 'src/libcore/slice')
| -rw-r--r-- | src/libcore/slice/mod.rs | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/libcore/slice/mod.rs b/src/libcore/slice/mod.rs index 9b8334167f5..72036d6d3a2 100644 --- a/src/libcore/slice/mod.rs +++ b/src/libcore/slice/mod.rs @@ -2240,7 +2240,10 @@ impl<'a, T> FusedIterator for Chunks<'a, T> {} unsafe impl<'a, T> TrustedRandomAccess for Chunks<'a, T> { unsafe fn get_unchecked(&mut self, i: usize) -> &'a [T] { let start = i * self.chunk_size; - let end = cmp::min(start + self.chunk_size, self.v.len()); + let end = match start.checked_add(self.chunk_size) { + None => self.v.len(), + Some(end) => cmp::min(end, self.v.len()), + }; from_raw_parts(self.v.as_ptr().offset(start as isize), end - start) } fn may_have_side_effect() -> bool { false } @@ -2353,7 +2356,10 @@ impl<'a, T> FusedIterator for ChunksMut<'a, T> {} unsafe impl<'a, T> TrustedRandomAccess for ChunksMut<'a, T> { unsafe fn get_unchecked(&mut self, i: usize) -> &'a mut [T] { let start = i * self.chunk_size; - let end = cmp::min(start + self.chunk_size, self.v.len()); + let end = match start.checked_add(self.chunk_size) { + None => self.v.len(), + Some(end) => cmp::min(end, self.v.len()), + }; from_raw_parts_mut(self.v.as_mut_ptr().offset(start as isize), end - start) } fn may_have_side_effect() -> bool { false } |
