2012/2/24  <ma...@apache.org>:
> Author: markt
> Date: Thu Feb 23 22:28:32 2012
> New Revision: 1292995
>
> URL: http://svn.apache.org/viewvc?rev=1292995&view=rev
> Log:
> Review comments on r1292776 from kkolinko
>
> Modified:
>    tomcat/trunk/java/org/apache/catalina/websocket/WsOutbound.java
>
> @@ -166,7 +166,7 @@ public class WsOutbound {
>
>
>     /**
> -     * Respond to a client close by sending a close that echos the status 
> code
> +     * Respond to a client close by sending a close that echoes the status 
> code
>      * and message.
>      *
>      * @param frame The close frame received from a client
> @@ -274,7 +274,8 @@ public class WsOutbound {
>             upgradeOutbound.write(0);
>         } else {
>             upgradeOutbound.write(data.limit());

The above line has to be changed as well.
BTW, what happens if we are passed more than 125 bytes of data? IAE?

> -            upgradeOutbound.write(data.array(), 0, data.limit());
> +            upgradeOutbound.write(data.array(), data.position(),
> +                    data.limit() - data.position());
>         }
>
>         upgradeOutbound.flush();
>

Best regards,
Konstantin Kolinko

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

Reply via email to