On Thu, 14 Dec 2023 23:04:25 GMT, Brian Burkhalter <b...@openjdk.org> wrote:

>> Sergey Tsypanov has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   8320971: Revert irrelevant changes
>
> src/java.base/share/classes/java/io/BufferedInputStream.java line 646:
> 
>> 644:             int avail = count - pos;
>> 645:             if (avail > 0) {
>> 646:                 if (isTrusted(out)) {
> 
> It might be cleaner for now to drop `isTrusted()` and just do the class check 
> explicitly here. That still takes care of the main intent of not passing the 
> buffer to an untrustworthy stream. Further cleanup and consolidation can be 
> done later.

I think this would be doing double job, wouldn't it?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16879#discussion_r1427749255

Reply via email to