Looks good.

Thanks for catching and fixing this.

Roger

On 7/18/18 11:19 PM, Ivan Gerasimov wrote:
Hello!

In native functions java.lang.ProcessHandleImpl.getProcessPids0() and java.lang.ProcessHandleImpl.parent0() a handle to a snapshot of all processes in the system is obtained with CreateToolhelp32Snapshot().

This handle isn't releases if the subsequent call to Process32First() fails.

Would you please help review a trivial fix?

BUGURL: https://bugs.openjdk.java.net/browse/JDK-8207753
WEBREV: http://cr.openjdk.java.net/~igerasim/8207753/00/webrev/

Thanks in advance!


Reply via email to