Fixed ByteArrayBuilder.TrimNewLine() to check array bounds properly

Fixes issue #1634
jstedfast · Aug 31, 2023 · 70d6482 · 70d6482
1 parent e84d570
commit 70d6482
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/MailKit/ByteArrayBuilder.cs b/MailKit/ByteArrayBuilder.cs
@@ -128,10 +128,10 @@ public bool Equals (string value, bool ignoreCase = false)
 		public void TrimNewLine ()
 		{
 			// Trim the <CR><LF> sequence from the end of the line.
-			if (buffer[length - 1] == (byte) '\n') {
+			if (length > 0 && buffer[length - 1] == (byte) '\n') {
 				length--;
 
-				if (buffer[length - 1] == (byte) '\r')
+				if (length > 0 && buffer[length - 1] == (byte) '\r')
 					length--;
 			}
 		}