On Fri, 4 Mar 2022 13:25:12 GMT, Zhengyu Gu <z...@openjdk.org> wrote:

> Please review this small patch that fixes a potential memory leak that 
> exception return fails to release allocated `cacheDirs`
> 
> Test:
> 
> - [x] jdk_desktop on Linux x86_64

I would have just inlined JNU_CHECK_EXCEPTION and added the cleanup code 
directly. The additional macro wasn't really necessary and would not really be 
usable for any kind of general cleanup actions beyond a one-liner.

But it is okay.

Thanks.

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/7691

Reply via email to