Fix underlying assumption of SessionTrack that not work well for MariaDB

Signed-off-by: elemount <[email protected]>

Author: elemount

src/MySqlConnector/Protocol/Payloads/OkPayload.cs

@@ -43,22 +43,25 @@ public static OkPayload Create(ReadOnlySpan<byte> span, bool deprecateEof)
 			{
 				reader.ReadLengthEncodedByteString(); // human-readable info
 
-				// implies ProtocolCapabilities.SessionTrack
-				var sessionStateChangeDataLength = checked((int) reader.ReadLengthEncodedInteger());
-				var endOffset = reader.Offset + sessionStateChangeDataLength;
-				while (reader.Offset < endOffset)
+				// If ProtocolCapabilities.SessionTrack is enabled, it will contains the information. Otherwise, it will contains Nothing.
+				if (reader.BytesRemaining > 0)
 				{
-					var kind = (SessionTrackKind) reader.ReadByte();
-					var dataLength = (int) reader.ReadLengthEncodedInteger();
-					switch (kind)
+					var sessionStateChangeDataLength = checked((int) reader.ReadLengthEncodedInteger());
+					var endOffset = reader.Offset + sessionStateChangeDataLength;
+					while (reader.Offset < endOffset)
 					{
-					case SessionTrackKind.Schema:
-						newSchema = Encoding.UTF8.GetString(reader.ReadLengthEncodedByteString());
-						break;
+						var kind = (SessionTrackKind) reader.ReadByte();
+						var dataLength = (int) reader.ReadLengthEncodedInteger();
+						switch (kind)
+						{
+						case SessionTrackKind.Schema:
+							newSchema = Encoding.UTF8.GetString(reader.ReadLengthEncodedByteString());
+							break;
 
-					default:
-						reader.Offset += dataLength;
-						break;
+						default:
+							reader.Offset += dataLength;
+							break;
+						}
 					}
 				}
 			}