https://issues.apache.org/bugzilla/show_bug.cgi?id=57173

--- Comment #2 from Konstantin Kolinko <knst.koli...@gmail.com> ---
What is the version of Java runtime that you are using?
(So that one could actually reproduce the issue)


Thoughts:
1) FastDataInputStream.fillNew() shall check that this.in.read() is not -1.

2) FastDataInputStream.readUnsignedShort() expects that fillNew() call will
read at least 2 bytes, but read() can actually return after reading 1 byte (I
think it may depend on internal implementation of zip inflater thus my question
on the version of Java that OP is using), so some looping is needed like in
FastDataInputStream.readFully(..)/skipBytes().

There is also a question of whether FastDataInputStream with such changes
remains faster than the original DataInputStream.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

Reply via email to