fix: read faststr should use bytes instead of bytesmut (#74)

cloudwego · Dec 8, 2022 · 136cfe8 · 136cfe8
1 parent d2e3759
commit 136cfe8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/pilota/src/thrift/binary.rs b/pilota/src/thrift/binary.rs
@@ -954,9 +954,9 @@ impl TInputProtocol for TBinaryProtocol<&mut BytesMut> {
     #[inline]
     fn read_faststr(&mut self) -> Result<FastStr, Error> {
         let len = self.trans.read_i32()? as usize;
-        let bytes = self.trans.split_to(len);
+        let bytes = self.trans.split_to(len).freeze();
         if len > INLINE_CAP {
-            unsafe { return Ok(FastStr::from_bytes_mut_unchecked(bytes)) };
+            unsafe { return Ok(FastStr::from_bytes_unchecked(bytes)) };
         }
         unsafe { Ok(FastStr::new_inline(str::from_utf8_unchecked(bytes.deref()))) }
     }