You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2002/11/22 16:55:33 UTC

cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 InternalOutputBuffer.java

remm        2002/11/22 07:55:32

  Modified:    http11/src/java/org/apache/coyote/http11
                        InternalOutputBuffer.java
  Log:
  - If an IOE happens during the commit, the response is never flagged as
    committed (so would get generated multiple times in the OB depending on how much
    commit is used in the rest of the processing).
  - Fix bug 14760 (ok, there really was a bug, although it has almost zero actual impact).
  
  Revision  Changes    Path
  1.16      +4 -4      jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11/InternalOutputBuffer.java
  
  Index: InternalOutputBuffer.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11/InternalOutputBuffer.java,v
  retrieving revision 1.15
  retrieving revision 1.16
  diff -u -r1.15 -r1.16
  --- InternalOutputBuffer.java	10 Oct 2002 13:14:55 -0000	1.15
  +++ InternalOutputBuffer.java	22 Nov 2002 15:55:32 -0000	1.16
  @@ -536,15 +536,15 @@
       protected void commit()
           throws IOException {
   
  +        // The response is now committed
  +        committed = true;
  +        response.setCommitted(true);
  +
           if (pos > 0) {
               // Sending the response header buffer
               outputStream.write(buf, 0, pos);
               outputStream.flush(); // Is it really necessary ?
           }
  -
  -        // The response is now committed
  -        committed = true;
  -        response.setCommitted(true);
   
       }
   
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>