about summary refs log tree commit diff
path: root/src/libcore/str
diff options
context:
space:
mode:
authorScott McMurray <scottmcm@users.noreply.github.com>2017-06-04 11:08:25 -0700
committerScott McMurray <scottmcm@users.noreply.github.com>2017-06-04 11:08:25 -0700
commit808a08a363591abf754fafd93ec3f44c686486ec (patch)
tree5cd63c46768d830265ab663cf88844c6399ab2fb /src/libcore/str
parent18612b21d05ce0021f9ef995592b158e181ffd90 (diff)
Add overflow checking for str::get with inclusive ranges
Fixes #42401
Diffstat (limited to 'src/libcore/str')
-rw-r--r--src/libcore/str/mod.rs16
1 files changed, 12 insertions, 4 deletions
diff --git a/src/libcore/str/mod.rs b/src/libcore/str/mod.rs
index 76a5d0c0b50..34aca592b1e 100644
--- a/src/libcore/str/mod.rs
+++ b/src/libcore/str/mod.rs
@@ -1918,11 +1918,19 @@ mod traits {
         type Output = str;
         #[inline]
         fn get(self, slice: &str) -> Option<&Self::Output> {
-            (self.start..self.end+1).get(slice)
+            if let Some(end) = self.end.checked_add(1) {
+                (self.start..end).get(slice)
+            } else {
+                None
+            }
         }
         #[inline]
         fn get_mut(self, slice: &mut str) -> Option<&mut Self::Output> {
-            (self.start..self.end+1).get_mut(slice)
+            if let Some(end) = self.end.checked_add(1) {
+                (self.start..end).get_mut(slice)
+            } else {
+                None
+            }
         }
         #[inline]
         unsafe fn get_unchecked(self, slice: &str) -> &Self::Output {
@@ -1953,7 +1961,7 @@ mod traits {
         type Output = str;
         #[inline]
         fn get(self, slice: &str) -> Option<&Self::Output> {
-            if slice.is_char_boundary(self.end + 1) {
+            if self.end < usize::max_value() && slice.is_char_boundary(self.end + 1) {
                 Some(unsafe { self.get_unchecked(slice) })
             } else {
                 None
@@ -1961,7 +1969,7 @@ mod traits {
         }
         #[inline]
         fn get_mut(self, slice: &mut str) -> Option<&mut Self::Output> {
-            if slice.is_char_boundary(self.end + 1) {
+            if self.end < usize::max_value() && slice.is_char_boundary(self.end + 1) {
                 Some(unsafe { self.get_unchecked_mut(slice) })
             } else {
                 None