Review request: JDK-8021245 (process) file leak in jdk/src/windows/native/java/lang/ProcessImpl_md.c

Alexey Utkin alexey.utkin at
Mon Jul 29 17:06:27 UTC 2013

Thanks, Alan

I did "if" reformatting and small refactoring for the code in function 
in accordance with
recommendations. I am sure that the presence of path separator has no 
dependence from search direction.

Here is the webrev:

The fix was tested in the debugger.


On 7/28/2013 9:40 PM, Alan Bateman wrote:
> On 26/07/2013 08:33, Alexey Utkin wrote:
>> Bug description:
>> Here is the suggested fix:
>> Summary:
>> In accordance with MS spec [-1] is THE ONLY error signal from the 
>> [_open] function call.
>> Any negative, except [-1], and zero are the valid handles that need 
>> to be closed.
>> The fix was tested in the debugger my manual switching to Win9x/WinMe 
>> brunch of execution.
> Checking if _open returns -1 and checking the return from _read and 
> _lseek/_read looks okay to me. My only comment is that the "if (" on 
> its own looks a bit odd, I just I would at least hoist the first part 
> of the expression onto the same line.
> -Alan.

More information about the core-libs-dev mailing list