You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2008/08/14 11:32:27 UTC
svn commit: r685833 - in /tomcat:
container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java
container/tc5.5.x/webapps/docs/changelog.xml current/tc5.5.x/STATUS.txt
Author: markt
Date: Thu Aug 14 02:32:26 2008
New Revision: 685833
URL: http://svn.apache.org/viewvc?rev=685833&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42727
Correctly handle request lines that are exact multiples of 4096 in length.
Patch provided by Will Pugh.
Modified:
tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java
tomcat/container/tc5.5.x/webapps/docs/changelog.xml
tomcat/current/tc5.5.x/STATUS.txt
Modified: tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java?rev=685833&r1=685832&r2=685833&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java (original)
+++ tomcat/container/tc5.5.x/catalina/src/share/org/apache/catalina/connector/CoyoteReader.java Thu Aug 14 02:32:26 2008
@@ -153,7 +153,7 @@
while ((pos < MAX_LINE_LENGTH) && (end < 0)) {
int nRead = read(lineBuffer, pos, MAX_LINE_LENGTH - pos);
if (nRead < 0) {
- if (pos == 0) {
+ if (pos == 0 && aggregator == null) {
return null;
}
end = pos;
Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?rev=685833&r1=685832&r2=685833&view=diff
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Thu Aug 14 02:32:26 2008
@@ -110,6 +110,10 @@
mod_jk has hung up the phone. (billbarker)
</update>
<fix>
+ <bug>42727</bug>: Handle request lines that are exact multiples of 4096
+ in length. Patch provided by Will Pugh. (markt)
+ </fix>
+ <fix>
<bug>43191</bug>: Compression could not be disabled for some file types.
Based on a patch by Len Popp. (markt)
</fix>
Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=685833&r1=685832&r2=685833&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Thu Aug 14 02:32:26 2008
@@ -49,13 +49,6 @@
+1: markt, yoavs, fhanik
-1:
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42727
- Correctly handle request lines that are exact multiples of 4096 in length.
- Patch provided by Will Pugh.
- http://svn.apache.org/viewvc?rev=677759&view=rev
- +1: markt, yoavs, fhanik
- -1:
-
* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45453
Add required sync to race condition
Based on a patch by Santtu Hyrkk
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org