https://issues.apache.org/bugzilla/show_bug.cgi?id=54456
Konstantin Kolinko <knst.koli...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |NEW
--- Comment #4 from Konstantin Kolinko <knst.koli...@gmail.com> ---
The following lines in ChunkedInputFilter.java (162-164 in trunk)
if (pos >= lastValid) {
readBytes();
}
is the only place in this file where the return value of "readBytes()" is not
checked.
Looking at the code, if the method returns -1 then (lastValid - pos) will be -1
and it will proceed with "chunk.setBytes(buf, pos, result);", which seems a
wrong thing to do in such a case.
I have not tested what is the actual behaviour here, but an explicit "return
-1" or "throw new EOFException()" would be better.
--
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
