Author: markt Date: Wed May 20 09:33:23 2015 New Revision: 1680502 URL: http://svn.apache.org/r1680502 Log: Fix the limit. Add a TODO. May need to switch to long.
Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java?rev=1680502&r1=1680501&r2=1680502&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java (original) +++ tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettings.java Wed May 20 09:33:23 2015 @@ -28,7 +28,8 @@ public class ConnectionSettings { private final StringManager sm = StringManager.getManager(ConnectionSettings.class); public static final int DEFAULT_WINDOW_SIZE = (1 << 16) - 1; - private static final int UNLIMITED = 1 << 32; // Use the maximum possible + // TODO: The maximum allowed in a settings frame as 2^32 (unsigned) + private static final int UNLIMITED = (1 << 31) -1; // Use the maximum possible private static final int MAX_WINDOW_SIZE = (1 << 31) - 1; private static final int MIN_MAX_FRAME_SIZE = 1 << 14; private static final int MAX_MAX_FRAME_SIZE = (1 << 24) - 1; --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org