On Tue, 25 Jul 2023 23:50:07 GMT, Brian Burkhalter <[email protected]> wrote:
>> Limit native memory allocation and move write loop from the native layer
>> into Java. This change should make the OOME reported in the issue much less
>> likely.
>
> 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 ?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14981#discussion_r1274390203
