don't try to decode packets before they have fully arrived

Author: MrNavaStar

common/src/main/java/me/mrnavastar/protoweaver/core/netty/ProtoPacketHandler.java

@@ -53,11 +53,18 @@ public void channelInactive(ChannelHandlerContext ctx) {
 
     @Override
     protected void decode(ChannelHandlerContext ctx, ByteBuf byteBuf, List<Object> list) {
-        if (byteBuf.readableBytes() == 0) return;
-        Object packet = null;
+        // Ensure the whole packet has arrived before trying to decode
+        if (byteBuf.readableBytes() < 4) return;
+        byteBuf.markReaderIndex();
+        int packetLen = byteBuf.readInt();
+        if (byteBuf.readableBytes() < packetLen) {
+            byteBuf.resetReaderIndex();
+            return;
+        }
 
+        Object packet = null;
         try {
-            byte[] bytes = new byte[byteBuf.readInt()];
+            byte[] bytes = new byte[packetLen];
             byteBuf.readBytes(bytes);
             packet = connection.getProtocol().deserialize(bytes);
             handler.handlePacket(connection, packet);

gradle.properties

@@ -30,7 +30,7 @@ waterfall_version=1.21-R0.1-SNAPSHOT
 
 # Dependencies
 # https://github.com/apache/fury/releases
-fury_version=0.9.0
+fury_version=0.7.0
 r_version=1.0.8
 # should always match the netty version in (latest) minecraft release
 netty_version=4.1.97.Final