From f6d88ca5bf213d8ae528ba1104652200dc77e6e5 Mon Sep 17 00:00:00 2001 From: Luke Wood Date: Fri, 22 Feb 2013 15:23:47 +0000 Subject: [PATCH] Use Short.MAX_VALUE, as it accurately represents what we're trying to limit ourselves to --- .../handler/codec/compression/SnappyFramedEncoder.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/codec/src/main/java/io/netty/handler/codec/compression/SnappyFramedEncoder.java b/codec/src/main/java/io/netty/handler/codec/compression/SnappyFramedEncoder.java index c4f9f91f20..eab82803d1 100644 --- a/codec/src/main/java/io/netty/handler/codec/compression/SnappyFramedEncoder.java +++ b/codec/src/main/java/io/netty/handler/codec/compression/SnappyFramedEncoder.java @@ -67,12 +67,12 @@ public class SnappyFramedEncoder extends ByteToByteEncoder { } out.writeInt(0); - if (dataLength >= 32768) { - ByteBuf slice = in.readSlice(32767); + if (dataLength > Short.MAX_VALUE) { + ByteBuf slice = in.readSlice(Short.MAX_VALUE); calculateAndWriteChecksum(slice, out); - snappy.encode(slice, out, 32767); + snappy.encode(slice, out, Short.MAX_VALUE); setChunkLength(out, lengthIdx); - dataLength -= 32767; + dataLength -= Short.MAX_VALUE; } else { ByteBuf slice = in.readSlice(dataLength); calculateAndWriteChecksum(slice, out);