Panic in BytesMut::split_to when out of bounds (#252) (#253)

CHANGELOG.md

@@ -1,4 +1,4 @@
-# 0.4.12 (March 6, 2018)
+# 0.4.12 (March 6, 2019)
 
 ### Added
 - Implement `FromIterator<&'a u8>` for `BytesMut`/`Bytes` (#244).

src/bytes.rs

@@ -1265,6 +1265,8 @@ impl BytesMut {
     ///
     /// Panics if `at > len`.
     pub fn split_to(&mut self, at: usize) -> BytesMut {
+        assert!(at <= self.len());
+
         BytesMut {
             inner: self.inner.split_to(at),
         }

tests/test_bytes.rs

@@ -258,15 +258,10 @@ fn split_to_oob_mut() {
 }
 
 #[test]
+#[should_panic]
 fn split_to_uninitialized() {
     let mut bytes = BytesMut::with_capacity(1024);
-    let other = bytes.split_to(128);
-
-    assert_eq!(bytes.len(), 0);
-    assert_eq!(bytes.capacity(), 896);
-
-    assert_eq!(other.len(), 0);
-    assert_eq!(other.capacity(), 128);
+    let _other = bytes.split_to(128);
 }
 
 #[test]