fix extra boolean read

Author: Gerrygames

proxy/src/main/java/com/velocitypowered/proxy/connection/client/AuthSessionHandler.java

@@ -151,7 +151,7 @@ private void loadAppliedResourcePacks(ConnectedPlayer player, byte[] cookieData)
       player.resourcePackHandler().loadAppliedResourcePacks(appliedResourcePacks);
     } catch (SignatureException e) {
       logger.warn("Signature error while loading applied resource packs of {}", player.getUsername(), e);
-    } catch (CodecException e) {
+    } catch (IndexOutOfBoundsException | CodecException e) {
       logger.warn("Error while decoding applied resource packs of {}", player.getUsername(), e);
     }
   }

proxy/src/main/java/com/velocitypowered/proxy/connection/player/resourcepack/ResourcePackTransfer.java

@@ -17,6 +17,7 @@
 
 package com.velocitypowered.proxy.connection.player.resourcepack;
 
+import com.google.common.base.Preconditions;
 import com.velocitypowered.api.network.ProtocolVersion;
 import com.velocitypowered.api.proxy.player.ResourcePackInfo;
 import com.velocitypowered.proxy.protocol.ProtocolUtils;
@@ -104,6 +105,7 @@ public static byte[] createCookieData(final byte[] secret, final Collection<Reso
         byte[] hash = appliedResourcePack.getHash();
         buffer.writeBoolean(hash != null);
         if (hash != null) {
+          Preconditions.checkArgument(hash.length == 20, "Hash length is not 20");
           buffer.writeBytes(hash);
         }
         buffer.writeBoolean(appliedResourcePack.getShouldForce());
@@ -194,7 +196,6 @@ public static Collection<ResourcePackInfo> decodeAndValidateCookieData(final byt
         VelocityResourcePackInfo appliedResourcePack = builder.build();
         appliedResourcePack.setOriginalOrigin(ORIGINS[ProtocolUtils.readVarInt(buffer)]);
         appliedResourcePacks.add(appliedResourcePack);
-        buffer.readBoolean();
       }
       return appliedResourcePacks;
     } finally {