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

[Ivan Gerasimov]

Thank you Roger for review!


On 7/19/18 6:15 AM, Roger Riggs wrote:
> 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!
>>
>
>

-- 
With kind regards,
Ivan Gerasimov