Author: markt
Date: Fri Aug 12 07:47:10 2016
New Revision: 1756101

URL: http://svn.apache.org/viewvc?rev=1756101&view=rev
Log:
Align ActionCode.NB_READ_INTEREST

Modified:
    tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java
    tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java
    tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java

Modified: tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java?rev=1756101&r1=1756100&r2=1756101&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/ajp/AjpProcessor.java Fri Aug 12 
07:47:10 2016
@@ -546,8 +546,8 @@ public class AjpProcessor extends Abstra
             break;
         }
         case NB_READ_INTEREST: {
-            if (!endOfStream) {
-                socketWrapper.registerReadInterest();
+            if (!isRequestBodyFullyRead()) {
+                registerReadInterest();
             }
             break;
         }
@@ -1512,6 +1512,11 @@ public class AjpProcessor extends Abstra
     }
     
     
+    private void registerReadInterest() {
+        socketWrapper.registerReadInterest();
+    }
+    
+    
     /**
      * Read at least the specified amount of bytes, and place them
      * in the input buffer. Note that if any data is available to read then 
this

Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java?rev=1756101&r1=1756100&r2=1756101&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Fri Aug 12 
07:47:10 2016
@@ -839,7 +839,9 @@ public class Http11Processor extends Abs
             break;
         }
         case NB_READ_INTEREST: {
-            socketWrapper.registerReadInterest();
+            if (!isRequestBodyFullyRead()) {
+                registerReadInterest();
+            }
             break;
         }
         case NB_WRITE_INTEREST: {
@@ -1835,6 +1837,11 @@ public class Http11Processor extends Abs
     }
     
     
+    private void registerReadInterest() {
+        socketWrapper.registerReadInterest();
+    }
+    
+    
     /**
      * Checks to see if the keep-alive loop should be broken, performing any
      * processing (e.g. sendfile handling) that may have an impact on whether

Modified: tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java?rev=1756101&r1=1756100&r2=1756101&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http2/StreamProcessor.java Fri Aug 12 
07:47:10 2016
@@ -293,7 +293,9 @@ public class StreamProcessor extends Abs
             break;
         }
         case NB_READ_INTEREST: {
-            stream.getInputBuffer().registerReadInterest();
+            if (!isRequestBodyFullyRead()) {
+                registerReadInterest();
+            }
             break;
         }
         case NB_WRITE_INTEREST: {
@@ -447,6 +449,11 @@ public class StreamProcessor extends Abs
     }
     
     
+    private void registerReadInterest() {
+        stream.getInputBuffer().registerReadInterest();
+    }
+    
+    
     @Override
     public void recycle() {
         // StreamProcessor instances are not re-used.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to