| commit | dfffdd8726cbf7b6a309ab354c638ede92d2f215 | [log] [tgz] |
|---|---|---|
| author | Bryan Duxbury <bryanduxbury@apache.org> | Thu Aug 11 18:54:11 2011 +0000 |
| committer | Bryan Duxbury <bryanduxbury@apache.org> | Thu Aug 11 18:54:11 2011 +0000 |
| tree | 3bf939fb59ec572e1fb17f1ce7c95c30f1c39eba | |
| parent | 1121b47c09470a454a28a46825f92ef29c4dc273 [diff] |
THRIFT-1205. java: port server unduly fragile with arbitrary input Increase the default max frame size to 16MB. Patch: Ryan King git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1156731 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/lib/java/src/org/apache/thrift/transport/TFastFramedTransport.java b/lib/java/src/org/apache/thrift/transport/TFastFramedTransport.java index 2a1f1da..e32b7db 100644 --- a/lib/java/src/org/apache/thrift/transport/TFastFramedTransport.java +++ b/lib/java/src/org/apache/thrift/transport/TFastFramedTransport.java
@@ -59,9 +59,9 @@ */ public static final int DEFAULT_BUF_CAPACITY = 1024; /** - * How big is the largest allowable frame? Defaults to Integer.MAX_VALUE. + * How big is the largest allowable frame? Defaults to 16MB. */ - public static final int DEFAULT_MAX_LENGTH = Integer.MAX_VALUE; + public static final int DEFAULT_MAX_LENGTH = 16384000; private final TTransport underlying; private final AutoExpandingBufferWriteTransport writeBuffer;
diff --git a/lib/java/src/org/apache/thrift/transport/TFramedTransport.java b/lib/java/src/org/apache/thrift/transport/TFramedTransport.java index eb5b0c2..c948aa4 100644 --- a/lib/java/src/org/apache/thrift/transport/TFramedTransport.java +++ b/lib/java/src/org/apache/thrift/transport/TFramedTransport.java
@@ -27,7 +27,7 @@ */ public class TFramedTransport extends TTransport { - protected static final int DEFAULT_MAX_LENGTH = 0x7FFFFFFF; + protected static final int DEFAULT_MAX_LENGTH = 16384000; private int maxLength_;