On Thu, 7 Jul 2022 07:34:32 GMT, Сергей Цыпанов <d...@openjdk.org> wrote:

>> Markus KARG has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fixed missing BufferedInputStream
>
> src/java.base/share/classes/java/io/BufferedInputStream.java line 495:
> 
>> 493:         int avail = count - pos;
>> 494:         if (avail > 0) {
>> 495:             out.write(buf, pos, avail);
> 
> I think `buf` should not be accessed directly but via `getBufIfOpen()` 
> because we need to throw IOE in case `this` is closed.

Thank you, good catch! I just fixed this in 
https://github.com/openjdk/jdk/pull/6935/commits/2c415d0c9c39cec372c1edb189ac140af2755fa4.
 👍

-------------

PR: https://git.openjdk.org/jdk/pull/6935

Reply via email to