Skip to content

Commit abfbd51

Browse files
committed
Assert that the buffer has free space before reading
1 parent 773b7c5 commit abfbd51

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/imap_stream.rs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,6 +270,13 @@ impl<R: Read + Write + Unpin> Stream for ImapStream<R> {
270270
this.buffer.ensure_capacity(this.decode_needs)?;
271271
let buf = this.buffer.free_as_mut_slice();
272272

273+
// The buffer should have at least one byte free
274+
// before we try reading into it
275+
// so we can treat 0 bytes read as EOF.
276+
// This is guaranteed by `ensure_capacity()` above
277+
// even if it is called with 0 as an argument.
278+
debug_assert!(buf.len() > 0);
279+
273280
#[cfg(feature = "runtime-async-std")]
274281
let num_bytes_read = ready!(Pin::new(&mut this.inner).poll_read(cx, buf))?;
275282

0 commit comments

Comments
 (0)