Merge pull request #23 from leshleekor/develop

Fix: ByteBuffer allocating
MrGraversen · Feb 14, 2024 · 651eee4 · 651eee4
2 parents c7dcbeb + ae869fc
commit 651eee4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/io/graversen/minecraft/rcon/MinecraftClient.java b/src/main/java/io/graversen/minecraft/rcon/MinecraftClient.java
@@ -147,10 +147,10 @@ private ByteBuffer readData(int bytes) {
 
     private ByteBuffer createRconByteBuffer(int requestCount, int requestType, String command) {
         // In accordance with the RCON format: Length + Request ID + Type + Payload + Two nil bytes
-        ByteBuffer byteBuffer = ByteBuffer.allocate((3 * Integer.BYTES) + command.length() + (2 * Byte.BYTES));
+        ByteBuffer byteBuffer = ByteBuffer.allocate((3 * Integer.BYTES) + command.getBytes().length + (2 * Byte.BYTES));
         byteBuffer.order(ByteOrder.LITTLE_ENDIAN);
 
-        byteBuffer.putInt((2 * Integer.BYTES) + command.length() + (2 * Byte.BYTES));
+        byteBuffer.putInt((2 * Integer.BYTES) + command.getBytes().length + (2 * Byte.BYTES));
         byteBuffer.putInt(requestCount);
         byteBuffer.putInt(requestType);
         byteBuffer.put(command.getBytes());