🐛 Fix Package Null padding

Thanks @ExusAltimus
Partial-Fix: #56
Discovered-in: #57

Author: xobust

src/CoreRCON/Constants.cs

@@ -18,6 +18,13 @@ internal class Constants
         /// </summary>
         internal const int PACKET_HEADER_SIZE = 12;
 
+        /// <summary>
+        /// The size of the padding of an RCON packet.
+        /// The packet is padded with two null bytes.
+        /// One to terminate the string and one to terminate the packet.
+        /// </summary>
+        internal const int PACKET_PADDING_SIZE = 2;
+
         /// <summary>
         /// Special response value when you send a Response.Response to the server.
         /// Used to finde the end of a multi packet response.

src/CoreRCON/PacketFormats/RCONPacket.cs

@@ -68,8 +68,8 @@ internal static RCONPacket FromBytes(byte[] buffer)
         /// <returns>Byte array with each field.</returns>
         internal byte[] ToBytes()
         {
-            int bodyLength = Encoding.UTF8.GetByteCount(Body) + 1;
-            int totalLength = Constants.PACKET_HEADER_SIZE + bodyLength;
+            int bodyLength = Encoding.UTF8.GetByteCount(Body);
+            int totalLength = Constants.PACKET_HEADER_SIZE + bodyLength + Constants.PACKET_PADDING_SIZE;
             byte[] packetBytes = new byte[totalLength];
             Span<byte> packetSpan = packetBytes;
 
@@ -88,7 +88,7 @@ internal byte[] ToBytes()
 
             // Write body
             Encoding.UTF8.GetBytes(Body, 0, Body.Length, packetBytes, Constants.PACKET_HEADER_SIZE);
-            packetSpan[bodyLength - 1] = 0; // Null terminator for the body
+            packetSpan[bodyLength] = 0; // Null terminator for the body
             packetBytes[totalLength - 1] = 0; // Null terminator for the package
 
             return packetBytes;