ee10: DefaultServlet: Replace checks for isStreaming() by !isWriting()

Checking for isStreaming() will return false even when no data has been written yet. This erroneously triggers double-encoding of binary data in a Writer. If the Writer is not set to ISO-8859-1 or similar, the output will be corrupted, causing errors like "java.io.IOException: written 25746 > 12014 content-length", and interrupted transmissions. Replace the isStreaming() check by !isWriting(). #9134 Signed-off-by: Christian Kohlschütter <[email protected]>
jetty · Jan 9, 2023 · 984a082 · 984a082
1 parent 6d075a4
commit 984a082
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...-ee10/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/DefaultServlet.java b/...-ee10/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/DefaultServlet.java
@@ -802,10 +802,10 @@ public boolean isWritingOrStreaming()
             return servletContextResponse.isWritingOrStreaming();
         }
 
-        public boolean isStreaming()
+        public boolean isWriting()
         {
             ServletContextResponse servletContextResponse = Response.as(_coreResponse, ServletContextResponse.class);
-            return servletContextResponse.isStreaming();
+            return servletContextResponse.isWriting();
         }
 
         @Override
@@ -815,7 +815,7 @@ public void write(boolean last, ByteBuffer byteBuffer, Callback callback)
             {
                 if (BufferUtil.hasContent(byteBuffer))
                 {
-                    if (isStreaming())
+                    if (!isWriting())
                     {
                         BufferUtil.writeTo(byteBuffer, _response.getOutputStream());
                         if (last)