On Tue, 25 Jul 2023 23:50:07 GMT, Brian Burkhalter <b...@openjdk.org> 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 199:

> 197:     }
> 198: 
> 199:     if (buf != stackBuf)

Wouldn't this cause a leak when if-condition is not met and `free(buf)` is not 
called?

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

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

Reply via email to