RFR [XS] : 8205416 : windows: fix checking of CloseHandle return code in Java_java_io_FileCleanable_cleanupClose0
Baesken, Matthias
matthias.baesken at sap.com
Wed Jun 20 12:39:19 UTC 2018
Hi Thomas and Alan, thanks for the reviews.
I adjusted the expression like Thomas suggested and adjusted the file copyright year too :
http://cr.openjdk.java.net/~mbaesken/webrevs/8205416.1/
Best regards, Matthias
> -----Original Message-----
> From: Thomas Stüfe [mailto:thomas.stuefe at gmail.com]
> Sent: Mittwoch, 20. Juni 2018 14:17
> To: Baesken, Matthias <matthias.baesken at sap.com>
> Cc: core-libs-dev at openjdk.java.net
> Subject: Re: RFR [XS] : 8205416 : windows: fix checking of CloseHandle return
> code in Java_java_io_FileCleanable_cleanupClose0
>
> Good catch.
>
> Could you please change the expression to either
>
> CloseHandle() == FALSE (uppercase)
>
> or
>
> !CloseHandle().
>
> ? which would be the standard windows way of writing this.
>
> I am curious whether we now get a bunch of follow up errors, since
> before we never catched a failing CloseHandle. A typical reason why
> this could fail would be a double close.
>
> ..Thomas
>
>
> On Wed, Jun 20, 2018 at 2:09 PM, Baesken, Matthias
> <matthias.baesken at sap.com> wrote:
> > Please review this small fix for a return code handling of windows function
> CloseHandle .
> >
> > MSDN documents CloseHandle here : https://msdn.microsoft.com/de-
> de/library/windows/desktop/ms724211(v=vs.85).aspx
> > ....
> > Return value
> > If the function succeeds, the return value is nonzero.
> > If the function fails, the return value is zero. To get extended error
> information, call GetLastError<https://msdn.microsoft.com/de-
> de/library/windows/desktop/ms679360(v=vs.85).aspx>.
> >
> >
> > However until this patch, Java_java_io_FileCleanable_cleanupClose0
> > Checked for return code -1 of CloseHandle in error cases .
> >
> >
> >
> >
> > Bug:
> >
> > https://bugs.openjdk.java.net/browse/JDK-8205416
> >
> >
> > Webrev :
> >
> > http://cr.openjdk.java.net/~mbaesken/webrevs/8205416/
> >
> >
> >
> > Thanks, Matthias
More information about the core-libs-dev
mailing list