RFR: 6478546: FileInputStream.read() throws OutOfMemoryError when there is plenty available [v2]

Brian Burkhalter bpb at openjdk.org
Wed Jul 26 15:11:51 UTC 2023


On Wed, 26 Jul 2023 05:26:52 GMT, Vyom Tewari <vtewari at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - 6478546: Decrease malloc limit to 1.5 MB
>>  - 6478546: Minor refactoring
>>  - 6478546: Prevent short read
>
> src/java.base/share/native/libjava/io_util.c line 163:
> 
>> 161:     }
>> 162: 
>> 163:     if (outOfBounds(env, off, len, bytes)) {
> 
> we are already performing this check at Java level, do we need the perform this check again ?

Please refer to my prior [comment](https://github.com/openjdk/jdk/pull/14981#issuecomment-1646312987).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14981#discussion_r1275120710



More information about the security-dev mailing list