RFR 8207753 : Handle to process snapshot is leaked if Process32First() fails

Roger Riggs roger.riggs at oracle.com
Thu Jul 19 13:15:48 UTC 2018


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!
>



More information about the core-libs-dev mailing list